]> www.infradead.org Git - users/dwmw2/openconnect.git/commitdiff
Close config_fd before returning from write_new_config()
authorDavid Woodhouse <David.Woodhouse@intel.com>
Wed, 26 Sep 2012 19:24:52 +0000 (20:24 +0100)
committerDavid Woodhouse <David.Woodhouse@intel.com>
Wed, 26 Sep 2012 19:34:40 +0000 (20:34 +0100)
Spotted by Coverity.

Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
main.c

diff --git a/main.c b/main.c
index a93e10ae2c09dc42f2d2603fd168cd58dd12262a..4d0d9d042462c01c6ed8757c17b28534ebd0eceb 100644 (file)
--- a/main.c
+++ b/main.c
@@ -885,10 +885,11 @@ static int write_new_config(void *_vpninfo, char *buf, int buflen)
                err = errno;
                fprintf(stderr, _("Failed to write config to %s: %s\n"),
                        vpninfo->xmlconfig, strerror(err));
-
+               close(config_fd);
                return -err;
        }
 
+       close(config_fd);
        return 0;
 }