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:29:46 +0200 |
| commit | bce656d27fe06ed364a4acebd3c3d6d996750613 (patch) | |
| tree | deff298e4d5e243e4d18947fb287de9d1c0e19cc /src/openvpn/openvpn.c | |
| parent | e5664524d625b51f6c3e844e13d3f2c0066d7607 (diff) | |
| download | openvpn-bce656d27fe06ed364a4acebd3c3d6d996750613.tar.gz openvpn-bce656d27fe06ed364a4acebd3c3d6d996750613.tar.xz openvpn-bce656d27fe06ed364a4acebd3c3d6d996750613.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>
(cherry picked from commit 659eae7b79e5565bb0c93f6d6d04e2163fea1141)
Diffstat (limited to 'src/openvpn/openvpn.c')
| -rw-r--r-- | src/openvpn/openvpn.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/openvpn/openvpn.c b/src/openvpn/openvpn.c index 2f327f3..00bd570 100644 --- a/src/openvpn/openvpn.c +++ b/src/openvpn/openvpn.c @@ -231,7 +231,10 @@ openvpn_main (int argc, char *argv[]) /* become a daemon if --daemon */ if (c.first_time) - c.did_we_daemonize = possibly_become_daemon (&c.options); + { + c.did_we_daemonize = possibly_become_daemon (&c.options); + write_pid (c.options.writepid); + } #ifdef ENABLE_MANAGEMENT /* open management subsystem */ |