]> git.ipfire.org Git - thirdparty/openvpn.git/commitdiff
Un-break --auth-user-pass on windows
authorGert Doering <gert@greenie.muc.de>
Tue, 4 Aug 2015 12:53:16 +0000 (14:53 +0200)
committerGert Doering <gert@greenie.muc.de>
Tue, 4 Aug 2015 15:43:54 +0000 (17:43 +0200)
Commit b131c7b974d9d4d3f0 introduced a check to create a meaningful
warning if we try to read a password after daemon()izing (by checking
whether stdin or stderr is connected to a tty).  For some reason this
breaks on Windows builds if run under GUI control - but since Windows
doesn't have this particular daemon() issue anyway, just #ifndef WIN32
the offending code.

Signed-off-by: Gert Doering <gert@greenie.muc.de>
Acked-by: Steffan Karger <steffan.karger@fox-it.com>
Message-Id: <1438692796-14663-1-git-send-email-gert@greenie.muc.de>
URL: http://article.gmane.org/gmane.network.openvpn.devel/10000

src/openvpn/misc.c

index c4438b6de9ae5fe52d6eaddb3e72f146b69bf2f2..895e9facf94a23b6ef389b1fee4219d1a60a0bc0 100644 (file)
@@ -1056,9 +1056,11 @@ get_user_pass_cr (struct user_pass *up,
        */
       else if (from_stdin)
        {
+#ifndef WIN32
          /* did we --daemon'ize before asking for passwords? */
          if ( !isatty(0) && !isatty(2) )
            { msg(M_FATAL, "neither stdin nor stderr are a tty device, can't ask for %s password.  If you used --daemon, you need to use --askpass to make passphrase-protected keys work, and you can not use --auth-nocache.", prefix ); }
+#endif
 
 #ifdef ENABLE_CLIENT_CR
          if (auth_challenge && (flags & GET_USER_PASS_DYNAMIC_CHALLENGE))