From bcc62cc9e34ce7536a05f0bdccda7ada628ba220 Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Tue, 28 Mar 2006 08:03:25 +0000 Subject: [PATCH] #1451929 (http://curl.haxx.se/bug/view.cgi?id=1451929) detailed a bug that occurred when asking libcurl to follow HTTP redirects and the original URL had more than one question mark (?). Added test case 276 to verify. --- CHANGES | 5 +++ RELEASE-NOTES | 1 + lib/transfer.c | 2 +- tests/data/Makefile.am | 2 +- tests/data/test276 | 72 ++++++++++++++++++++++++++++++++++++++++++ 5 files changed, 80 insertions(+), 2 deletions(-) create mode 100644 tests/data/test276 diff --git a/CHANGES b/CHANGES index 62296efa0..e55edd4d0 100644 --- a/CHANGES +++ b/CHANGES @@ -6,6 +6,11 @@ Changelog +Daniel (28 March 2006) +- #1451929 (http://curl.haxx.se/bug/view.cgi?id=1451929) detailed a bug that + occurred when asking libcurl to follow HTTP redirects and the original URL + had more than one question mark (?). Added test case 276 to verify. + Daniel (27 March 2006) - David Byron found a problem multiple -d options when libcurl was built with --enable-debug, as then curl used free() on memory allocated both with diff --git a/RELEASE-NOTES b/RELEASE-NOTES index 73d5165b0..e7bfc8010 100644 --- a/RELEASE-NOTES +++ b/RELEASE-NOTES @@ -16,6 +16,7 @@ This release includes the following changes: This release includes the following bugfixes: + o following redirects with more than one question mark in source URL o fixed debug build crash with -d o TFTP works on more systems o generates a fine AIX Toolbox RPM spec diff --git a/lib/transfer.c b/lib/transfer.c index 65c899934..6fa8f6334 100644 --- a/lib/transfer.c +++ b/lib/transfer.c @@ -1827,7 +1827,7 @@ CURLcode Curl_follow(struct SessionHandle *data, /* First we need to find out if there's a ?-letter in the URL, and cut it and the right-side of that off */ - pathsep = strrchr(protsep, '?'); + pathsep = strchr(protsep, '?'); if(pathsep) *pathsep=0; diff --git a/tests/data/Makefile.am b/tests/data/Makefile.am index 818257198..c28a57250 100644 --- a/tests/data/Makefile.am +++ b/tests/data/Makefile.am @@ -34,5 +34,5 @@ EXTRA_DIST = test1 test108 test117 test127 test20 test27 test34 test46 \ test250 test251 test252 test253 test254 test255 test521 test522 test523 \ test256 test257 test258 test259 test260 test261 test262 test263 test264 \ test265 test266 test267 test268 test269 test270 test271 test272 test273 \ - test274 test275 test524 + test274 test275 test524 test276 diff --git a/tests/data/test276 b/tests/data/test276 new file mode 100644 index 000000000..7090ec581 --- /dev/null +++ b/tests/data/test276 @@ -0,0 +1,72 @@ + + +HTTP +HTTP GET +followlocation + + +# Server-side + + +HTTP/1.1 301 This is a weirdo text message swsclose +Date: Thu, 09 Nov 2010 14:49:00 GMT +Server: test-server/fake +Location: data/2760002.txt?coolsite=http://anotherurl/?a_second/2760002 +Connection: close + +This server reply is for testing a simple Location: following + + + +HTTP/1.1 200 Followed here fine swsclose +Date: Thu, 09 Nov 2010 14:49:00 GMT +Server: test-server/fake + +If this is received, the location following worked + + + +HTTP/1.1 301 This is a weirdo text message swsclose +Date: Thu, 09 Nov 2010 14:49:00 GMT +Server: test-server/fake +Location: data/2760002.txt?coolsite=http://anotherurl/?a_second/2760002 +Connection: close + +HTTP/1.1 200 Followed here fine swsclose +Date: Thu, 09 Nov 2010 14:49:00 GMT +Server: test-server/fake + +If this is received, the location following worked + + + + +# Client-side + + +http + + +HTTP Location: following with multiple question marks in URLs + + +"http://%HOSTIP:%HTTPPORT/want?uri=http://anything/276?secondq/276" -L + + + +# Verify data after the test has been "shot" + + +^User-Agent:.* + + +GET /want?uri=http://anything/276?secondq/276 HTTP/1.1 +Host: 127.0.0.1:%HTTPPORT +Accept: */* + +GET /data/2760002.txt?coolsite=http://anotherurl/?a_second/2760002 HTTP/1.1 +Host: 127.0.0.1:%HTTPPORT +Accept: */* + + +