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 ed541a12fd3bbc8ff0129d89ee1f077b852018d5..982bed063af3eeb537e76ba82983491ce0eb9bed 100644 (file)
--- 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)