qemu-bridge-helper: Fix fd leak in main()

initialize fd and ctlfd, and close them at the end Signed-off-by: Gonglei <[email protected]> Signed-off-by: Stefan Hajnoczi <[email protected]>
my-unikernel · Jun 27, 2014 · eb3f45c · eb3f45c
1 parent ff4873c
commit eb3f45c
Showing 1 changed file with 7 additions and 2 deletions.
diff --git 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);