]> granicus.if.org Git - curl/commitdiff
cookies: Remove redundant expired check
authorDaniel Gustafsson <daniel@yesql.se>
Sun, 9 Sep 2018 06:52:04 +0000 (08:52 +0200)
committerDaniel Stenberg <daniel@haxx.se>
Mon, 10 Sep 2018 06:30:24 +0000 (08:30 +0200)
Expired cookies have already been purged at a later expiration time
before this check, so remove the redundant check.

closes #2962

lib/cookie.c

index fd7341f0bd857d1bce77c5e39c2fbd4f2be7df88..1668f02769eb3449f0b64a50356ad8b1d488a4b1 100644 (file)
@@ -1218,7 +1218,6 @@ struct Cookie *Curl_cookie_getlist(struct CookieInfo *c,
 {
   struct Cookie *newco;
   struct Cookie *co;
-  time_t now = time(NULL);
   struct Cookie *mainco = NULL;
   size_t matches = 0;
   bool is_ip;
@@ -1236,11 +1235,8 @@ struct Cookie *Curl_cookie_getlist(struct CookieInfo *c,
   co = c->cookies[myhash];
 
   while(co) {
-    /* only process this cookie if it is not expired or had no expire
-       date AND that if the cookie requires we're secure we must only
-       continue if we are! */
-    if((!co->expires || (co->expires > now)) &&
-       (co->secure?secure:TRUE)) {
+    /* if the cookie requires we're secure we must only continue if we are! */
+    if(co->secure?secure:TRUE) {
 
       /* now check if the domain is correct */
       if(!co->domain ||