From: Daniel Stenberg <daniel@haxx.se>
Date: Mon, 13 Oct 2008 21:03:12 +0000 (+0000)
Subject: Removed superfluous check of clist->name, as in this code path that pointer
X-Git-Tag: curl-7_19_1~163
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=18be9882f7ed8eaac5052e096c3868906385c1f6;p=curl

Removed superfluous check of clist->name, as in this code path that pointer
has already been dereferenced so it is bound to be valid. Pointed out to us
by coverity.com
---

diff --git a/lib/cookie.c b/lib/cookie.c
index ed541a12f..982bed063 100644
--- a/lib/cookie.c
+++ b/lib/cookie.c
@@ -614,8 +614,7 @@ Curl_cookie_add(struct SessionHandle *data,
         co->next = clist->next; /* get the next-pointer first */
 
         /* then free all the old pointers */
-        if(clist->name)
-          free(clist->name);
+        free(clist->name);
         if(clist->value)
           free(clist->value);
         if(clist->domain)