Skip to content

Commit

Permalink
Dylan Salisbury's fix to prevent us from accepting cookies from TLD only
Browse files Browse the repository at this point in the history
  • Loading branch information
bagder committed Aug 4, 2004
1 parent 5e1eb58 commit de6ab3d
Showing 1 changed file with 2 additions and 14 deletions.
16 changes: 2 additions & 14 deletions lib/cookie.c
Expand Up @@ -223,14 +223,9 @@ Curl_cookie_add(struct SessionHandle *data,

const char *domptr=whatptr;
int dotcount=1;
unsigned int i;

static const char *seventhree[]= {
"com", "edu", "net", "org", "gov", "mil", "int"
};

/* Count the dots, we need to make sure that there are THREE dots
in the normal domains, or TWO in the seventhree-domains. */
/* Count the dots, we need to make sure that there are enough
of them. */

if('.' == whatptr[0])
/* don't count the initial dot, assume it */
Expand All @@ -244,13 +239,6 @@ Curl_cookie_add(struct SessionHandle *data,
}
} while(domptr);

for(i=0;
i<sizeof(seventhree)/sizeof(seventhree[0]); i++) {
if(tailmatch(seventhree[i], whatptr)) {
dotcount++; /* we allow one dot less for these */
break;
}
}
/* The original Netscape cookie spec defined that this domain name
MUST have three dots (or two if one of the seven holy TLDs),
but it seems that these kinds of cookies are in use "out there"
Expand Down

0 comments on commit de6ab3d

Please sign in to comment.