From 90abb74ff0e3134d8647722cee36b1815c14143d Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Fri, 3 Nov 2017 15:04:53 +0200 Subject: curl: pass through [] in URLs instead of calling globbing error Assisted-by: Per Lundberg Fixes #2044 Closes #2046 Closes #2048 --- src/tool_urlglob.c | 8 +++++--- tests/data/Makefile.inc | 2 +- tests/data/test1290 | 48 ++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 54 insertions(+), 4 deletions(-) create mode 100644 tests/data/test1290 diff --git a/src/tool_urlglob.c b/src/tool_urlglob.c index df85d7129..f78d058cd 100644 --- a/src/tool_urlglob.c +++ b/src/tool_urlglob.c @@ -367,9 +367,11 @@ static CURLcode glob_parse(URLGlob *glob, char *pattern, size_t sublen = 0; while(*pattern && *pattern != '{') { if(*pattern == '[') { - /* Skip over potential IPv6 literals. */ - size_t skip; - if(peek_ipv6(pattern, &skip)) { + /* skip over IPv6 literals and [] */ + size_t skip = 0; + if(!peek_ipv6(pattern, &skip) && (pattern[1] == ']')) + skip = 2; + if(skip) { memcpy(buf, pattern, skip); buf += skip; pattern += skip; diff --git a/tests/data/Makefile.inc b/tests/data/Makefile.inc index e3eb80e4d..35c41a5c1 100644 --- a/tests/data/Makefile.inc +++ b/tests/data/Makefile.inc @@ -137,7 +137,7 @@ test1252 test1253 test1254 test1255 test1256 test1257 test1258 test1259 \ test1260 test1261 test1262 \ \ test1280 test1281 test1282 test1283 test1284 test1285 test1286 test1287 \ -test1288 test1289 \ +test1288 test1289 test1290 \ test1298 test1299 \ test1300 test1301 test1302 test1303 test1304 test1305 test1306 test1307 \ test1308 test1309 test1310 test1311 test1312 test1313 test1314 test1315 \ diff --git a/tests/data/test1290 b/tests/data/test1290 new file mode 100644 index 000000000..e556303bf --- /dev/null +++ b/tests/data/test1290 @@ -0,0 +1,48 @@ + + + +HTTP +HTTP GET +globbing + + + +# +# Server-side + + +HTTP/1.1 200 OK +Content-Length: 6 +Connection: close +Content-Type: text/html + +-foo- + + + +# Client-side + + +http + + +Verify URL globbing ignores [] + + +"http://%HOSTIP:%HTTPPORT/we/want/[]/page/1290" + + + +# Verify data after the test has been "shot" + + +^User-Agent:.* + + +GET /we/want/[]/page/1290 HTTP/1.1 +Host: %HOSTIP:%HTTPPORT +Accept: */* + + + + -- cgit v1.2.1