[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qapi: Fix memory leak
From: |
Michael Tokarev |
Subject: |
Re: [Qemu-devel] [PATCH] qapi: Fix memory leak |
Date: |
Sun, 19 Aug 2012 01:06:07 +0400 |
User-agent: |
Mozilla/5.0 (X11; Linux i686 on x86_64; rv:10.0.5) Gecko/20120624 Icedove/10.0.5 |
On 19.08.2012 00:51, Stefan Weil wrote:
> +++ b/qapi/opts-visitor.c
> @@ -416,7 +416,7 @@ opts_visitor_cleanup(OptsVisitor *ov)
> g_hash_table_destroy(ov->unprocessed_opts);
> }
> g_free(ov->fake_id_opt);
> - memset(ov, '\0', sizeof *ov);
> + g_free(ov);
Shouldn't the function be named opts_visitor_free() or .._destroy()
in this case? Or should maybe the caller free "ov" instead of
this function? To me it looks like either both free+rename shoud
be made, or none.
Thanks,
/mjt