Unverified Commit 299d9660 authored by Daniel Stenberg's avatar Daniel Stenberg
Browse files

Revert "cookies: extend domain checks to non psl builds" 

This reverts commit 3773de37.

Regression shipped in 7.64.0
Fixes #3649
parent 76b63489

lib/cookie.c

+5 −8
Original line number Diff line number Diff line
@@ -816,8 +816,6 @@ Curl_cookie_add(struct Curl_easy *data, 
        co->domain = strdup(ptr);

        if(!co->domain)

          badcookie = TRUE;

        else if(bad_domain(co->domain))

          badcookie = TRUE;

        break;

      case 1:

        /* This field got its explanation on the 23rd of May 2001 by
@@ -946,20 +944,18 @@ Curl_cookie_add(struct Curl_easy *data, 
  if(!noexpire)

    remove_expired(c);



  if(domain && co->domain && !isip(co->domain)) {

    int acceptable;

#ifdef USE_LIBPSL

  /* Check if the domain is a Public Suffix and if yes, ignore the cookie. */

  if(domain && co->domain && !isip(co->domain)) {

    const psl_ctx_t *psl = Curl_psl_use(data);

    int acceptable;



    /* Check if the domain is a Public Suffix and if yes, ignore the cookie. */

    if(psl) {

      acceptable = psl_is_cookie_domain_acceptable(psl, domain, co->domain);

      Curl_psl_release(data);

    }

    else

#endif

    /* Without libpsl, do the best we can. */

    acceptable = !bad_domain(co->domain);

      acceptable = !bad_domain(domain);



    if(!acceptable) {

      infof(data, "cookie '%s' dropped, domain '%s' must not "
@@ -968,6 +964,7 @@ Curl_cookie_add(struct Curl_easy *data, 
      return NULL;

    }

  }

#endif



  myhash = cookiehash(co->domain);

  clist = c->cookies[myhash];

tests/data/test8

+0 −1
Original line number Diff line number Diff line
@@ -46,7 +46,6 @@ Set-Cookie: trailingspace = removed; path=/we/want; 
Set-Cookie: nocookie=yes; path=/WE;

Set-Cookie: blexp=yesyes; domain=%HOSTIP; domain=%HOSTIP; expiry=totally bad;

Set-Cookie: partialip=nono; domain=.0.0.1;

Set-Cookie: chocolate=chip; domain=curl; path=/we/want;



</file>

<precheck>