Skip to content

Commit

Permalink
As identified in bug report #495290, the last "name=value" pair in a
Browse files Browse the repository at this point in the history
Set-Cookie: line was ignored if they didn't end with a trailing
semicolon. This is indeed wrong syntax, but there are high-profile web sites
out there sending cookies like that so we must make a best-effort to parse
them.
  • Loading branch information
bagder committed Jan 7, 2002
1 parent 2182e37 commit 8703713
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions lib/cookie.c
Expand Up @@ -194,6 +194,11 @@ Curl_cookie_add(struct CookieInfo *c,
while(ptr && *ptr && isspace((int)*ptr))
ptr++;
semiptr=strchr(ptr, ';'); /* now, find the next semicolon */

if(!semiptr && *ptr)
/* There are no more semicolons, but there's a final name=value pair
coming up */
semiptr=ptr;
} while(semiptr);

if(NULL == co->domain)
Expand Down

0 comments on commit 8703713

Please sign in to comment.