redirect from a bad url such as "www.com?moo=foo" to an absolute path didn't

work, as reported by John McGowan

diff --git a/lib/transfer.c b/lib/transfer.c
index 56cc46cfab057c6debd2f19b91b3d97dda48f984..74f03611b34fe67f7f905c99d90313c05b53f885 100644 (file)
--- a/lib/transfer.c
+++ b/lib/transfer.c
@@ -1666,6 +1666,15 @@ CURLcode Curl_follow(struct SessionHandle *data,
       pathsep = strchr(protsep, '/');
       if(pathsep)
         *pathsep=0;
+      else {
+        /* There was no slash. Now, since we might be operating on a badly
+           formatted URL, such as "http://www.url.com?id=2380" which doesn't
+           use a slash separator as it is supposed to, we need to check for a
+           ?-letter as well! */
+        pathsep = strchr(protsep, '?');
+        if(pathsep)
+          *pathsep=0;
+      }
     }
 
     /* If the new part contains a space, this is a mighty stupid redirect