cookie: only count accepted cookies in Curl_cookie_add

The counter used to stop accepting cookies after a certain amount has
been received in a single response would previously also count some
cookies that were not actually accepted as they were discarded after the
counter was increased.

Starting now, the counter is increased only for cookies that were
accepted.

Pointed out by ZeroPath
This commit is contained in:
Daniel Stenberg 2025-10-20 11:34:25 +02:00
parent 4be9db7bc8
commit 7db69ba375
No known key found for this signature in database
GPG key ID: 5CC908FDB71E12C2

View file

@ -750,7 +750,6 @@ parse_cookie_header(struct Curl_easy *data,
if(!co->name)
return CERR_BAD;
data->req.setcookies++;
return CERR_OK;
}
@ -1140,6 +1139,9 @@ Curl_cookie_add(struct Curl_easy *data,
if(co->expires && (co->expires < ci->next_expiration))
ci->next_expiration = co->expires;
if(httpheader)
data->req.setcookies++;
return co;
fail:
freecookie(co);