lkml.org 
[lkml]   [2022]   [Jan]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 5.4 7/8] vhost/test: fix memory leak of vhost virtqueues
    Date
    From: Xianting Tian <xianting.tian@linux.alibaba.com>

    [ Upstream commit 080063920777af65105e5953e2851e036376e3ea ]

    We need free the vqs in .release(), which are allocated in .open().

    Signed-off-by: Xianting Tian <xianting.tian@linux.alibaba.com>
    Link: https://lore.kernel.org/r/20211228030924.3468439-1-xianting.tian@linux.alibaba.com
    Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
    Acked-by: Jason Wang <jasowang@redhat.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/vhost/test.c | 1 +
    1 file changed, 1 insertion(+)

    diff --git a/drivers/vhost/test.c b/drivers/vhost/test.c
    index 056308008288c..fd8e9f70b06d3 100644
    --- a/drivers/vhost/test.c
    +++ b/drivers/vhost/test.c
    @@ -166,6 +166,7 @@ static int vhost_test_release(struct inode *inode, struct file *f)
    /* We do an extra flush before freeing memory,
    * since jobs can re-queue themselves. */
    vhost_test_flush(n);
    + kfree(n->dev.vqs);
    kfree(n);
    return 0;
    }
    --
    2.34.1
    \
     
     \ /
      Last update: 2022-01-23 01:15    [W:3.213 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site