[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] vhost-user: fix memory leak
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH v2] vhost-user: fix memory leak |
Date: |
Tue, 13 Feb 2018 18:01:09 +0100 |
On Tue, Feb 13, 2018 at 6:08 AM, linzhecheng <address@hidden> wrote:
> g_free() was moved from vhost_net_cleanup in commit e6bcb1b, so we should
> free net after vhost_net_cleanup
>
> Signed-off-by: linzhecheng <address@hidden>
This can be reproduced and catched by ASAN too:
==11833==ERROR: LeakSanitizer: detected memory leaks
Direct leak of 784 byte(s) in 1 object(s) allocated from:
#0 0x7fa1aed89a38 in __interceptor_calloc (/lib64/libasan.so.4+0xdea38)
#1 0x7fa1adadff75 in g_malloc0 ../glib/gmem.c:124
#2 0x56082a195515 in vhost_net_init
/home/elmarco/src/qemu/hw/net/vhost_net.c:147
#3 0x61500000bbff (<unknown module>)
good catch,
Reviewed-by: Marc-André Lureau < address@hidden>
>
> diff --git a/net/vhost-user.c b/net/vhost-user.c
> index cb45512506..d024573e45 100644
> --- a/net/vhost-user.c
> +++ b/net/vhost-user.c
> @@ -109,6 +109,7 @@ static int vhost_user_start(int queues, NetClientState
> *ncs[], CharBackend *be)
> err:
> if (net) {
> vhost_net_cleanup(net);
> + g_free(net);
> }
> vhost_user_stop(i, ncs);
> return -1;
> --
> 2.12.2.windows.2
>
>
>
--
Marc-André Lureau