Fix temp file leak. Closes: #764651

parent 9573f42d
openvpn (2.3.4-3) unstable; urgency=medium
* Apply patch by Samuel Thibault to clean up temporary files.
(Closes: #764651). Thanks Samuel!
-- Alberto Gonzalez Iniesta <agi@inittab.org> Mon, 13 Oct 2014 18:24:03 +0200
openvpn (2.3.4-2) unstable; urgency=medium
* openvpn.service. Remove ExecStop, add ExecReload.
......
Index: openvpn/src/openvpn/multi.c
===================================================================
--- openvpn.orig/src/openvpn/multi.c 2014-10-13 18:23:19.340637895 +0200
+++ openvpn/src/openvpn/multi.c 2014-10-13 18:23:19.336637895 +0200
@@ -1452,10 +1452,6 @@
option_types_found,
mi->context.c2.es);
- if (!platform_unlink (dc_file))
- msg (D_MULTI_ERRORS, "MULTI: problem deleting temporary file: %s",
- dc_file);
-
/*
* If the --client-connect script generates a config file
* with an --ifconfig-push directive, it will override any
@@ -1698,6 +1694,11 @@
multi_client_connect_post (m, mi, dc_file, option_permissions_mask, &option_types_found);
++cc_succeeded_count;
}
+
+ if (!platform_unlink (dc_file))
+ msg (D_MULTI_ERRORS, "MULTI: problem deleting temporary file: %s",
+ dc_file);
+
script_depr_failed:
argv_reset (&argv);
}
@@ -1751,6 +1752,11 @@
}
else
cc_succeeded = false;
+
+ if (!platform_unlink (dc_file))
+ msg (D_MULTI_ERRORS, "MULTI: problem deleting temporary file: %s",
+ dc_file);
+
script_failed:
argv_reset (&argv);
}
......@@ -7,3 +7,4 @@ kfreebsd_support.patch
accommodate_typo.patch
manpage_fixes.patch
better_systemd_detection.patch
client_connect_tmp_files.patch
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment