aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFrank Lichtenheld2023-05-12 17:50:23 +0200
committerGert Doering2023-05-13 11:14:58 +0200
commit26ea58f94ce10e108aa7c607b0a7dcb3419449fc (patch)
tree77e8f54bdef19f7dfd3d52ab825be3dd77806aba
parentb4f749f14a8edc75cefedda276ac6a38ecbc1595 (diff)
downloadopenvpn-26ea58f94ce10e108aa7c607b0a7dcb3419449fc.zip
openvpn-26ea58f94ce10e108aa7c607b0a7dcb3419449fc.tar.gz
dco_linux: properly close dco version file
Since we only call this once, it is not a bad leak, but still. Change-Id: Id85766738c3ece4f2d1860f7d101e4446a894aed Signed-off-by: Frank Lichtenheld <frank@lichtenheld.com> Acked-by: Gert Doering <gert@greenie.muc.de> Message-Id: <20230512155023.444406-1-frank@lichtenheld.com> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg26650.html Signed-off-by: Gert Doering <gert@greenie.muc.de> (cherry picked from commit cf496476b364f8613bacd48e10d6a1bbbf0aceda)
-rw-r--r--src/openvpn/dco_linux.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/openvpn/dco_linux.c b/src/openvpn/dco_linux.c
index 41540c0..796e6f2 100644
--- a/src/openvpn/dco_linux.c
+++ b/src/openvpn/dco_linux.c
@@ -1020,6 +1020,7 @@ dco_version_string(struct gc_arena *gc)
if (!fgets(BSTR(&out), BCAP(&out), fp))
{
+ fclose(fp);
return "ERR";
}
@@ -1031,6 +1032,7 @@ dco_version_string(struct gc_arena *gc)
*nl = '\0';
}
+ fclose(fp);
return BSTR(&out);
}