Re: [Qemu-devel] [PATCH] libvhost-user: Fix resource leak

2018-02-08 Thread Maxime Coquelin
On 01/18/2018 04:41 PM, Yongji Xie wrote: Free the mmaped memory when we need to mmap new memory space on vu_set_mem_table_exec() and vu_set_log_base_exec() to avoid memory leak. Also close the corresponding fd after mmap() on vu_set_log_base_exec() to avoid fd leak. Signed-off-by: Yongji Xie

Re: [Qemu-devel] [PATCH] libvhost-user: Fix resource leak

2018-01-18 Thread Marc-André Lureau
Hi On Thu, Jan 18, 2018 at 4:41 PM, Yongji Xie wrote: > Free the mmaped memory when we need to mmap new memory > space on vu_set_mem_table_exec() and vu_set_log_base_exec() to > avoid memory leak. > > Also close the corresponding fd after mmap() on > vu_set_log_base_exec() to avoid fd leak. > > S

[Qemu-devel] [PATCH] libvhost-user: Fix resource leak

2018-01-18 Thread Yongji Xie
Free the mmaped memory when we need to mmap new memory space on vu_set_mem_table_exec() and vu_set_log_base_exec() to avoid memory leak. Also close the corresponding fd after mmap() on vu_set_log_base_exec() to avoid fd leak. Signed-off-by: Yongji Xie --- contrib/libvhost-user/libvhost-user.c |