[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 06/18] vhost-vdpa: fix leaking of vhost_net in vhost_vdpa_add
From: |
Eli Cohen |
Subject: |
Re: [PATCH 06/18] vhost-vdpa: fix leaking of vhost_net in vhost_vdpa_add() |
Date: |
Thu, 24 Jun 2021 10:32:20 +0300 |
User-agent: |
Mutt/1.9.5 (bf161cf53efb) (2018-04-13) |
On Thu, Jun 24, 2021 at 03:10:46PM +0800, Jason Wang wrote:
>
> 在 2021/6/24 下午3:06, Eli Cohen 写道:
> > On Wed, Jun 23, 2021 at 05:00:16PM +0200, Stefano Garzarella wrote:
> > > On Mon, Jun 21, 2021 at 12:16:38PM +0800, Jason Wang wrote:
> > > > Fixes: 1e0a84ea49b68 ("vhost-vdpa: introduce vhost-vdpa net client")
> > > > Signed-off-by: Jason Wang <jasowang@redhat.com>
> > > > ---
> > > > net/vhost-vdpa.c | 1 +
> > > > 1 file changed, 1 insertion(+)
> > > >
> > > > diff --git a/net/vhost-vdpa.c b/net/vhost-vdpa.c
> > > > index f5689a7c32..21f09c546f 100644
> > > > --- a/net/vhost-vdpa.c
> > > > +++ b/net/vhost-vdpa.c
> > > > @@ -111,6 +111,7 @@ static int vhost_vdpa_add(NetClientState *ncs, void
> > > > *be)
> > > > err:
> > > > if (net) {
> > This check is redundant. net is not null.
>
>
> Actually, it can:
>
> net = vhost_net_init(&options);
> if (!net) {
> error_report("failed to init vhost_net for queue");
> goto err;
> }
Hmmm... right.
>
> Thanks
>
>
> > > > vhost_net_cleanup(net);
> > > > + g_free(net);
> > > > }
> > > > return -1;
> > > > }
> > > > --
> > > > 2.25.1
> > > >
> > > >
> > > Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>
> > >
>
- Re: [PATCH 03/18] vhost_net: do not assume nvqs is always 2, (continued)
- [PATCH 04/18] vhost-vdpa: remove the unnecessary check in vhost_vdpa_add(), Jason Wang, 2021/06/21
- [PATCH 05/18] vhost-vdpa: don't cleanup twice in vhost_vdpa_add(), Jason Wang, 2021/06/21
- [PATCH 06/18] vhost-vdpa: fix leaking of vhost_net in vhost_vdpa_add(), Jason Wang, 2021/06/21
- Re: [PATCH 06/18] vhost-vdpa: fix leaking of vhost_net in vhost_vdpa_add(), Eli Cohen, 2021/06/24
- Re: [PATCH 06/18] vhost-vdpa: fix leaking of vhost_net in vhost_vdpa_add(), Jason Wang, 2021/06/24
[PATCH 07/18] vhost-vdpa: tweak the error label in vhost_vdpa_add(), Jason Wang, 2021/06/21
[PATCH 08/18] vhost-vdpa: fix the wrong assertion in vhost_vdpa_init(), Jason Wang, 2021/06/21
[PATCH 09/18] vhost-vdpa: remove the unncessary queue_index assignment, Jason Wang, 2021/06/21
[PATCH 10/18] vhost-vdpa: open device fd in net_init_vhost_vdpa(), Jason Wang, 2021/06/21