Re: [Qemu-devel] [PATCH] iscsi: fix segfault in url parsing

[ronnie sahlberg]

Acked-By: address@hidden

On Thu, Nov 15, 2012 at 6:42 AM, Peter Lieven <address@hidden> wrote:
> If an invalid URL is specified iscsi_get_error(iscsi) is called
> with iscsi == NULL.
>
> Signed-off-by: Peter Lieven <address@hidden>
> ---
>  block/iscsi.c |    3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/block/iscsi.c b/block/iscsi.c
> index d0b1a10..b5c3161 100644
> --- a/block/iscsi.c
> +++ b/block/iscsi.c
> @@ -947,8 +947,7 @@ static int iscsi_open(BlockDriverState *bs, const char
> *filename, int flags)
>
>      iscsi_url = iscsi_parse_full_url(iscsi, filename);
>      if (iscsi_url == NULL) {
> -        error_report("Failed to parse URL : %s %s", filename,
> -                     iscsi_get_error(iscsi));
> +        error_report("Failed to parse URL : %s", filename);
>          ret = -EINVAL;
>          goto out;
>      }
> --
> 1.7.9.5
>