diff options
author | Daniel Stenberg <daniel@haxx.se> | 2012-02-10 16:26:20 +0100 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2012-02-10 16:26:20 +0100 |
commit | ecc93caaebe4d7c0168cedd99c3a6c42f7db9666 (patch) | |
tree | d5f398bcb5c1e427f5d52a2990f25ddc9fadb3d8 /lib | |
parent | ebf31389927dd1f514c0a7092a6ba52ad003ad95 (diff) |
parse_proxy: bail out on zero-length proxy names!
The proxy parser function strips off trailing slashes off the proxy name
which could lead to a mistaken zero length proxy name which would be
treated as no proxy at all by subsequent functions!
This is now detected and an error is returned. Verified by the new test
1329.
Reported by: Chandrakant Bagul
Bug: http://curl.haxx.se/mail/lib-2012-02/0000.html
Diffstat (limited to 'lib')
-rw-r--r-- | lib/url.c | 11 |
1 files changed, 10 insertions, 1 deletions
@@ -4271,11 +4271,20 @@ static CURLcode parse_proxy(struct SessionHandle *data, conn->port = strtol(prox_portno, NULL, 10); } else { + if(proxyptr[0]=='/') { + /* If the first character in the proxy string is a slash, fail + immediately. The following code will otherwise clear the string which + will lead to code running as if no proxy was set! */ + free(proxy); /* free the former proxy string */ + return CURLE_COULDNT_RESOLVE_PROXY; + } + /* without a port number after the host name, some people seem to use a slash so we strip everything from the first slash */ atsign = strchr(proxyptr, '/'); - if(atsign) + if(atsign) { *atsign = 0x0; /* cut off path part from host name */ + } if(data->set.proxyport) /* None given in the proxy string, then get the default one if it is |