Re: [Qemu-devel] [PATCH] vhost: fix a memory leak

2017-07-04 Thread Michael S. Tsirkin
On Tue, Jul 04, 2017 at 11:59:54PM +0200, Marc-André Lureau wrote: > Hi > > On Wed, Jul 5, 2017 at 12:33 AM, Peng Hao wrote: > > vhost exists a call for g_file_get_contents, but not call g_free. > > > > Signed-off-by: Peng Hao > > Reviewed-by:

Re: [Qemu-devel] [PATCH] vhost: fix a memory leak

2017-07-04 Thread Marc-André Lureau
Hi On Wed, Jul 5, 2017 at 12:33 AM, Peng Hao wrote: > vhost exists a call for g_file_get_contents, but not call g_free. > > Signed-off-by: Peng Hao Reviewed-by: Marc-André Lureau mst: is this better? > --- >

Re: [Qemu-devel] [PATCH] vhost: fix a memory leak

2017-07-04 Thread Michael S. Tsirkin
On Tue, Jul 04, 2017 at 02:21:08PM +, Marc-André Lureau wrote: > Hi > > On Tue, Jul 4, 2017 at 4:16 PM Peng Hao wrote: > > vhost exists a call for g_file_get_contents, but not call g_free. > > Signed-off-by: Peng Hao > > >  Reviewed-by:

Re: [Qemu-devel] [PATCH] vhost: fix a memory leak

2017-07-04 Thread Marc-André Lureau
Hi On Tue, Jul 4, 2017 at 4:16 PM Peng Hao wrote: > vhost exists a call for g_file_get_contents, but not call g_free. > > Signed-off-by: Peng Hao > Reviewed-by: Marc-André Lureau --- > hw/virtio/vhost-backend.c | 2 ++

[Qemu-devel] [PATCH] vhost: fix a memory leak

2017-07-04 Thread Peng Hao
vhost exists a call for g_file_get_contents, but not call g_free. Signed-off-by: Peng Hao --- hw/virtio/vhost-backend.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/virtio/vhost-backend.c b/hw/virtio/vhost-backend.c index 4e31de1..2c481d6 100644 ---