diff options
| author | Steffan Karger <steffan@karger.me> | 2015-06-19 00:08:45 +0200 |
|---|---|---|
| committer | Gert Doering <gert@greenie.muc.de> | 2015-06-21 14:27:30 +0200 |
| commit | 659eae7b79e5565bb0c93f6d6d04e2163fea1141 (patch) | |
| tree | dc1b379ff9f41d30fb766b9b49128461966052b0 /src/openvpn/misc.h | |
| parent | 60fd44e501f2002459a49c6c9bc64370ea26ca87 (diff) | |
| download | openvpn-659eae7b79e5565bb0c93f6d6d04e2163fea1141.tar.gz openvpn-659eae7b79e5565bb0c93f6d6d04e2163fea1141.tar.xz openvpn-659eae7b79e5565bb0c93f6d6d04e2163fea1141.zip | |
write pid file immediately after daemonizing
Since we split daemonizing from changing directory in commit da9b292
(f025de005d719201a69ad0313d545a1ddd244752 in release/2.3), we can
now simply write the pid file immediately after daemonizing.
This not only fixes the bug reported in trac #563, but also further
simplifies the code.
trac #563
Signed-off-by: Steffan Karger <steffan@karger.me>
Acked-by: Gert Doering <gert@greenie.muc.de>
Message-Id: <1434665325-3225-1-git-send-email-steffan@karger.me>
URL: http://article.gmane.org/gmane.network.openvpn.devel/9793
Signed-off-by: Gert Doering <gert@greenie.muc.de>
Diffstat (limited to 'src/openvpn/misc.h')
| -rw-r--r-- | src/openvpn/misc.h | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/src/openvpn/misc.h b/src/openvpn/misc.h index 5fe085e..7c26912 100644 --- a/src/openvpn/misc.h +++ b/src/openvpn/misc.h @@ -73,14 +73,7 @@ void run_up_down (const char *command, const char *script_type, struct env_set *es); -/* workspace for get_pid_file/write_pid */ -struct pid_state { - FILE *fp; - const char *filename; -}; - -void get_pid_file (const char* filename, struct pid_state *state); -void write_pid (const struct pid_state *state); +void write_pid (const char *filename); /* check file protections */ void warn_if_group_others_accessible(const char* filename); |