diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2008-10-13 21:03:12 +0000 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2008-10-13 21:03:12 +0000 |
| commit | 18be9882f7ed8eaac5052e096c3868906385c1f6 (patch) | |
| tree | 12cc99f3e5afd9f3a0075047dab21330ab907559 | |
| parent | a102c2c22b2764359334f69a9f41174a0f303cab (diff) | |
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
| -rw-r--r-- | lib/cookie.c | 3 |
1 files changed, 1 insertions, 2 deletions
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) |