]> granicus.if.org Git - curl/commitdiff
cookies: fix NULL dereference if flushing cookies with no CookieInfo set
authorMichael Wallner <mike@php.net>
Mon, 25 Feb 2019 18:05:02 +0000 (19:05 +0100)
committerDaniel Stenberg <daniel@haxx.se>
Tue, 26 Feb 2019 09:07:58 +0000 (10:07 +0100)
Regression brought by a52e46f3900fb0 (shipped in 7.63.0)

Closes #3613

lib/cookie.c

index 3a3f45de3855d1d7e21f4d6c26fb4ad3d91eeec7..a58ce2b35b56cb7800d79f3a6f64e66babdfd8ad 100644 (file)
@@ -1542,7 +1542,8 @@ static int cookie_output(struct CookieInfo *c, const char *dumphere)
   struct Cookie **array;
 
   /* at first, remove expired cookies */
-  remove_expired(c);
+  if(c)
+    remove_expired(c);
 
   if(!strcmp("-", dumphere)) {
     /* use stdout */
@@ -1561,7 +1562,7 @@ static int cookie_output(struct CookieInfo *c, const char *dumphere)
         "# This file was generated by libcurl! Edit at your own risk.\n\n",
         out);
 
-  if(c->numcookies) {
+  if(c && c->numcookies) {
     array = malloc(sizeof(struct Cookie *) * c->numcookies);
     if(!array) {
       if(!use_stdout)