Patchwork use qemu_free() instead of free()

login
register
mail settings
Submitter Bruce Rogers
Date May 13, 2010, 9:14 p.m.
Message ID <4BEC175902000048000957DF@sinclair.provo.novell.com>
Download mbox | patch
Permalink /patch/52521/
State New
Headers show

Comments

Bruce Rogers - May 13, 2010, 9:14 p.m.
There is a call to free() where qemu_free() should instead be used.

Signed-off-by: Bruce Rogers <brogers@novell.com>
Kevin Wolf - May 14, 2010, 11:44 a.m.
Am 13.05.2010 23:14, schrieb Bruce Rogers:
> There is a call to free() where qemu_free() should instead be used.
> 
> Signed-off-by: Bruce Rogers <brogers@novell.com>
> 
> diff --git a/block.c b/block.c
> index 48305b7..1a9d72d 100644
> --- a/block.c
> +++ b/block.c
> @@ -2073,7 +2073,7 @@ int bdrv_aio_multiwrite(BlockDriverState *bs, BlockRequest
>  *reqs, int num_reqs)
>      return 0;
>  
>  fail:
> -    free(mcb);
> +    qemu_free(mcb);
>      return -1;
>  }

Fixed the line wrap and applied to the block branch. Thanks.

Kevin

Patch

diff --git a/block.c b/block.c
index 48305b7..1a9d72d 100644
--- a/block.c
+++ b/block.c
@@ -2073,7 +2073,7 @@  int bdrv_aio_multiwrite(BlockDriverState *bs, BlockRequest
 *reqs, int num_reqs)
     return 0;
 
 fail:
-    free(mcb);
+    qemu_free(mcb);
     return -1;
 }