Andrés García's netscape cookie file parser fix

diff --git a/lib/cookie.c b/lib/cookie.c
index d1c308304ecf7bf51b1df8ad4e6d31b8e4701e3f..3c4034c423765ed78cefc020295d1457176eb541 100644 (file)
--- a/lib/cookie.c
+++ b/lib/cookie.c
@@ -233,8 +233,18 @@ Curl_cookie_add(struct CookieInfo *c,
         /* what _is_ this field for? */
         break;
       case 2:
-        co->path = strdup(ptr);
-        break;
+        /* It turns out, that sometimes the file format allows the path
+           field to remain not filled in, we try to detect this and work
+           around it! Andrés García made us aware of this... */
+        if (strcmp("TRUE", ptr) && strcmp("FALSE", ptr)) {
+          /* only if the path doesn't look like a boolean option! */
+          co->path = strdup(ptr);
+          break;
+        }
+        /* this doesn't look like a path, make one up! */
+        co->path = strdup("/");
+        fields++; /* add a field and fall down to secure */
+        /* FALLTHROUGH */
       case 3:
         co->secure = strequal(ptr, "TRUE");
         break;