[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] libvhost-user: Fix resource leak
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH] libvhost-user: Fix resource leak |
Date: |
Thu, 18 Jan 2018 16:53:41 +0100 |
Hi
On Thu, Jan 18, 2018 at 4:41 PM, Yongji Xie <address@hidden> 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 <address@hidden>
> ---
> contrib/libvhost-user/libvhost-user.c | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/contrib/libvhost-user/libvhost-user.c
> b/contrib/libvhost-user/libvhost-user.c
> index f409bd3..cadced7 100644
> --- a/contrib/libvhost-user/libvhost-user.c
> +++ b/contrib/libvhost-user/libvhost-user.c
> @@ -405,6 +405,15 @@ vu_set_mem_table_exec(VuDev *dev, VhostUserMsg *vmsg)
> {
> int i;
> VhostUserMemory *memory = &vmsg->payload.memory;
> +
> + for (i = 0; i < dev->nregions; i++) {
> + VuDevRegion *r = &dev->regions[i];
> + void *m = (void *) (uintptr_t) r->mmap_addr;
> +
> + if (m) {
> + munmap(m, r->size + r->mmap_offset);
> + }
> + }
> dev->nregions = memory->nregions;
>
> DPRINT("Nregions: %d\n", memory->nregions);
> @@ -470,9 +479,14 @@ vu_set_log_base_exec(VuDev *dev, VhostUserMsg *vmsg)
>
> rc = mmap(0, log_mmap_size, PROT_READ | PROT_WRITE, MAP_SHARED, fd,
> log_mmap_offset);
> + close(fd);
or call vmsg_close_fds() ?
> if (rc == MAP_FAILED) {
> perror("log mmap error");
> }
> +
> + if (dev->log_table) {
> + munmap(dev->log_table, dev->log_size);
> + }
> dev->log_table = rc;
> dev->log_size = log_mmap_size;
>
> --
> 1.7.9.5
>
>
Reviewed-by: Marc-André Lureau <address@hidden>
--
Marc-André Lureau