From: Michael Kaufmann <mail@michael-kaufmann.ch>
Date: Sun, 24 Sep 2017 13:21:44 +0000 (+0200)
Subject: proxy: read the "no_proxy" variable only if necessary
X-Git-Tag: curl-7_56_0~5
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c4ebd8b46de09d45e251bed7e9bfc2a0ddf21d98;p=thirdparty%2Fcurl.git

proxy: read the "no_proxy" variable only if necessary

Reviewed-by: Daniel Stenberg

Closes #1919
---

diff --git a/lib/url.c b/lib/url.c
index df12cbf6d0..132dbb432c 100644
--- a/lib/url.c
+++ b/lib/url.c
@@ -5352,22 +5352,21 @@ static CURLcode create_conn_helper_init_proxy(struct connectdata *conn)
     }
   }
 
-  no_proxy = curl_getenv("no_proxy");
-  if(!no_proxy)
-    no_proxy = curl_getenv("NO_PROXY");
+  if(!data->set.str[STRING_NOPROXY]) {
+    no_proxy = curl_getenv("no_proxy");
+    if(!no_proxy)
+      no_proxy = curl_getenv("NO_PROXY");
+  }
 
-  if(check_noproxy(conn->host.name, data->set.str[STRING_NOPROXY]) ||
-     (!data->set.str[STRING_NOPROXY] &&
-      check_noproxy(conn->host.name, no_proxy))) {
+  if(check_noproxy(conn->host.name, data->set.str[STRING_NOPROXY] ?
+      data->set.str[STRING_NOPROXY] : no_proxy)) {
     Curl_safefree(proxy);
     Curl_safefree(socksproxy);
   }
-  else if(!proxy && !socksproxy)
 #ifndef CURL_DISABLE_HTTP
+  else if(!proxy && !socksproxy)
     /* if the host is not in the noproxy list, detect proxy. */
     proxy = detect_proxy(conn);
-#else  /* !CURL_DISABLE_HTTP */
-    proxy = NULL;
 #endif /* CURL_DISABLE_HTTP */
 
   Curl_safefree(no_proxy);