diff options
author | Steffan Karger <steffan@karger.me> | 2015-07-09 23:35:59 +0200 |
---|---|---|
committer | Gert Doering <gert@greenie.muc.de> | 2015-07-13 21:12:10 +0200 |
commit | 315f6fbc7f657a7f1127628bd714f468709d5185 (patch) | |
tree | 1ee80a051f914227a278ae8abcaed33e7492586f /src/openvpn/openvpn.c | |
parent | 9884e20810bda737c7708ff587e09cc0bb8475c7 (diff) | |
download | openvpn-315f6fbc7f657a7f1127628bd714f468709d5185.tar.gz openvpn-315f6fbc7f657a7f1127628bd714f468709d5185.tar.xz openvpn-315f6fbc7f657a7f1127628bd714f468709d5185.zip |
fix regression: query password before becoming daemon
The init sequence was changed to daemonize before the crypto init to fix
issues on FreeBSD some commits ago. This introduced a regression where
we would no longer query for passwords before daemonizing, as described
in trac #574 and #576.
This commit restores the correct order, and adds a bit of const
correctness since we're touching this now code anyway.
Signed-off-by: Steffan Karger <steffan@karger.me>
Acked-by: Gert Doering <gert@greenie.muc.de>
Message-Id: <1436477759-5884-1-git-send-email-steffan@karger.me>
URL: http://article.gmane.org/gmane.network.openvpn.devel/9901
Signed-off-by: Gert Doering <gert@greenie.muc.de>
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 00bd570..d05acde 100644 --- a/src/openvpn/openvpn.c +++ b/src/openvpn/openvpn.c @@ -228,7 +228,10 @@ openvpn_main (int argc, char *argv[]) /* test crypto? */ if (do_test_crypto (&c.options)) break; - + + /* Query passwords before becoming a daemon */ + init_query_passwords (&c); + /* become a daemon if --daemon */ if (c.first_time) { |