Fix logic error in rev.13930

 Using !=0 on both string compares means any login= value will permit
 40x responses through. Only PASS and PASSTHRU should be doing that.

 Detected by Coverity Scan. Issue 1364711

diff --git a/src/tunnel.cc b/src/tunnel.cc
index 43e74dd54f0b61bb002a00f0dd512287d6d925e1..73d64115af494a26b2ab955593949914ab37c7a8 100644 (file)
--- a/src/tunnel.cc
+++ b/src/tunnel.cc
@@ -476,7 +476,7 @@ TunnelStateData::handleConnectResponse(const size_t chunkSize)
 
     // we need to relay the 401/407 responses when login=PASS(THRU)
     const char *pwd = server.conn->getPeer()->login;
-    const bool relay = pwd && (strcmp(pwd, "PASS") != 0 || strcmp(pwd, "PASSTHRU") != 0) &&
+    const bool relay = pwd && (strcmp(pwd, "PASS") == 0 || strcmp(pwd, "PASSTHRU") == 0) &&
                        (*status_ptr == Http::scProxyAuthenticationRequired ||
                         *status_ptr == Http::scUnauthorized);