[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH] vl: fix resource leak with monitor_fdset_add_fd
From: |
Paolo Bonzini |
Subject: |
[Qemu-devel] [PATCH] vl: fix resource leak with monitor_fdset_add_fd |
Date: |
Fri, 13 Mar 2015 13:55:34 +0100 |
monitor_fdset_add_fd returns an AddfdInfo struct (used by the QMP
command add_fd). Free it.
Signed-off-by: Paolo Bonzini <address@hidden>
---
v1->v2: line length [Fam], pass &error_abort [Shannon]
---
vl.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/vl.c b/vl.c
index eba5d4c..9eae8f9 100644
--- a/vl.c
+++ b/vl.c
@@ -1011,6 +1011,7 @@ static int parse_add_fd(QemuOpts *opts, void *opaque)
int fd, dupfd, flags;
int64_t fdset_id;
const char *fd_opaque = NULL;
+ AddfdInfo *fdinfo;
fd = qemu_opt_get_number(opts, "fd", -1);
fdset_id = qemu_opt_get_number(opts, "set", -1);
@@ -1060,8 +1061,10 @@ static int parse_add_fd(QemuOpts *opts, void *opaque)
}
/* add the duplicate fd, and optionally the opaque string, to the fd set */
- monitor_fdset_add_fd(dupfd, true, fdset_id, fd_opaque ? true : false,
- fd_opaque, NULL);
+ fdinfo = monitor_fdset_add_fd(dupfd, true, fdset_id,
+ fd_opaque ? true : false, fd_opaque,
+ &error_abort);
+ g_free(fdinfo);
return 0;
}
--
2.3.0
- [Qemu-devel] [PATCH] vl: fix resource leak with monitor_fdset_add_fd, Paolo Bonzini, 2015/03/12
- Re: [Qemu-devel] [PATCH] vl: fix resource leak with monitor_fdset_add_fd, Eric Blake, 2015/03/12
- Re: [Qemu-devel] [PATCH] vl: fix resource leak with monitor_fdset_add_fd, Fam Zheng, 2015/03/12
- Re: [Qemu-devel] [PATCH] vl: fix resource leak with monitor_fdset_add_fd, Patchew Tool, 2015/03/12
- Re: [Qemu-devel] [PATCH] vl: fix resource leak with monitor_fdset_add_fd, Shannon Zhao, 2015/03/13
- Re: [Qemu-devel] [PATCH] vl: fix resource leak with monitor_fdset_add_fd, Michael Tokarev, 2015/03/19
- [Qemu-devel] [PATCH] vl: fix resource leak with monitor_fdset_add_fd,
Paolo Bonzini <=