[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 5/6] qemu-bridge-helper: Fix fd leak in main()
From: |
arei.gonglei |
Subject: |
[Qemu-devel] [PATCH v2 5/6] qemu-bridge-helper: Fix fd leak in main() |
Date: |
Wed, 28 May 2014 21:21:39 +0800 |
From: Gonglei <address@hidden>
initialize fd and ctlfd, and close them at the end
Signed-off-by: Gonglei <address@hidden>
---
qemu-bridge-helper.c | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/qemu-bridge-helper.c b/qemu-bridge-helper.c
index 6a0974e..36eb3bc 100644
--- a/qemu-bridge-helper.c
+++ b/qemu-bridge-helper.c
@@ -229,7 +229,7 @@ int main(int argc, char **argv)
unsigned long ifargs[4];
#endif
int ifindex;
- int fd, ctlfd, unixfd = -1;
+ int fd = -1, ctlfd = -1, unixfd = -1;
int use_vnet = 0;
int mtu;
const char *bridge = NULL;
@@ -436,7 +436,12 @@ int main(int argc, char **argv)
/* profit! */
cleanup:
-
+ if (fd >= 0) {
+ close(fd);
+ }
+ if (ctlfd >= 0) {
+ close(ctlfd);
+ }
while ((acl_rule = QSIMPLEQ_FIRST(&acl_list)) != NULL) {
QSIMPLEQ_REMOVE_HEAD(&acl_list, entry);
g_free(acl_rule);
--
1.7.12.4
- [Qemu-devel] [PATCH v2 0/6] Fix some errors spotted by Coverity, arei.gonglei, 2014/05/28
- [Qemu-devel] [PATCH v2 6/6] audio: Fix using freed pointer in wav_fini_out(), arei.gonglei, 2014/05/28
- [Qemu-devel] [PATCH v2 2/6] vga: Fix divide-by-zero in vga_update_text, arei.gonglei, 2014/05/28
- [Qemu-devel] [PATCH v2 1/6] vnc-enc-tight: Fix divide-by-zero in tight_detect_smooth_image{16, 24, 32}, arei.gonglei, 2014/05/28
- [Qemu-devel] [PATCH v2 3/6] json-parser: Fix two coverity defects, arei.gonglei, 2014/05/28
- [Qemu-devel] [PATCH v2 5/6] qemu-bridge-helper: Fix fd leak in main(),
arei.gonglei <=
- [Qemu-devel] [PATCH v2 4/6] qcow2-cluster: Fix memory leak in copy_sectors, arei.gonglei, 2014/05/28