Re: [Qemu-devel] [PATCH] bochs: Fix memory leak in bochs_open() error path

[Laszlo Ersek <lersek@redhat.com>]

On 04/09/14 11:20, Kevin Wolf wrote:
> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
> ---
>  block/bochs.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/block/bochs.c b/block/bochs.c
> index 826ec12..50b84a9 100644
> --- a/block/bochs.c
> +++ b/block/bochs.c
> @@ -150,11 +150,13 @@ static int bochs_open(BlockDriverState *bs, QDict *options, int flags,
>      s->extent_size = le32_to_cpu(bochs.extent);
>      if (s->extent_size == 0) {
>          error_setg(errp, "Extent size may not be zero");
> -        return -EINVAL;
> +        ret = -EINVAL;
> +        goto fail;
>      } else if (s->extent_size > 0x800000) {
>          error_setg(errp, "Extent size %" PRIu32 " is too large",
>                     s->extent_size);
> -        return -EINVAL;
> +        ret = -EINVAL;
> +        goto fail;
>      }
>  
>      if (s->catalog_size < bs->total_sectors / s->extent_size) {
> 

Reviewed-by: Laszlo Ersek <lersek@redhat.com>