[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH/RFC] vl.c: fix memory leak spotted by valgrind
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-trivial] [PATCH/RFC] vl.c: fix memory leak spotted by valgrind |
Date: |
Wed, 28 Jan 2015 16:18:55 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 |
Reviewed-by: Paolo Bonzini <address@hidden>
and Ccing qemu-trivial. Funny that we do the g_slist_free before exiting!
Paolo
On 28/01/2015 16:07, Christian Borntraeger wrote:
> valgrind complains about:
> ==42062== 16 bytes in 1 blocks are definitely lost in loss record 387 of 1,048
> ==42062== at 0x402DCB2: malloc (vg_replace_malloc.c:299)
> ==42062== by 0x40C1BE3: g_malloc (in /usr/lib64/libglib-2.0.so.0.3800.2)
> ==42062== by 0x40DA133: g_slice_alloc (in
> /usr/lib64/libglib-2.0.so.0.3800.2)
> ==42062== by 0x40DB2E5: g_slist_prepend (in
> /usr/lib64/libglib-2.0.so.0.3800.2)
> ==42062== by 0x801637FF: object_class_get_list_tramp (object.c:690)
> ==42062== by 0x40A96C9: g_hash_table_foreach (in
> /usr/lib64/libglib-2.0.so.0.3800.2)
> ==42062== by 0x80164885: object_class_foreach (object.c:665)
> ==42062== by 0x80164975: object_class_get_list (object.c:698)
> ==42062== by 0x800100A5: machine_parse (vl.c:2447)
> ==42062== by 0x800100A5: main (vl.c:3756)
>
> Lets free machines in case of mc.
>
> Signed-off-by: Christian Borntraeger <address@hidden>
> ---
> vl.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/vl.c b/vl.c
> index 983259b..9d008ac 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -2450,6 +2450,7 @@ static gint machine_class_cmp(gconstpointer a,
> gconstpointer b)
> mc = find_machine(name);
> }
> if (mc) {
> + g_slist_free(machines);
> return mc;
> }
> if (name && !is_help_option(name)) {
>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-trivial] [PATCH/RFC] vl.c: fix memory leak spotted by valgrind,
Paolo Bonzini <=