[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] 答复: [PATCH] vhost-user: fix memory leak
From: |
linzhecheng |
Subject: |
[Qemu-devel] 答复: [PATCH] vhost-user: fix memory leak |
Date: |
Tue, 13 Feb 2018 04:16:02 +0000 |
> -----邮件原件-----
> 发件人: Philippe Mathieu-Daudé [mailto:address@hidden
> 代表 Philippe Mathieu-Daudé
> 发送时间: 2018年2月13日 11:54
> 收件人: linzhecheng <address@hidden>; address@hidden
> 抄送: address@hidden; wangxin (U) <address@hidden>;
> lidonglin <address@hidden>; address@hidden
> 主题: Re: [Qemu-devel] [PATCH] vhost-user: fix memory leak
>
> Hi Linzhecheng,
>
> On 02/12/2018 11:53 PM, linzhecheng wrote:
> > fix memory leak
> >
> > Signed-off-by: linzhecheng <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);
>
> I think this g_free() belongs to vhost_net_cleanup() in net/vhost_net.c:
I think your qemu version is out of date, g_free was moved from
vhost_net_cleanup in commit e6bcb1b
>
> void vhost_net_cleanup(struct vhost_net *net) {
> vhost_dev_cleanup(&net->dev);
> g_free(net);
> }
>
> Regards,
>
> Phil.
>
> > }
> > vhost_user_stop(i, ncs);
> > return -1;
> >