]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
singlesocket: remove dead code
authorDaniel Stenberg <daniel@haxx.se>
Tue, 12 Jun 2012 21:04:04 +0000 (23:04 +0200)
committerDaniel Stenberg <daniel@haxx.se>
Tue, 12 Jun 2012 21:04:04 +0000 (23:04 +0200)
No need to check if 'entry' is non-NULL in a spot where it is already checked
and guaranteed to be non-NULL.

(Spotted by a Coverity scan)

lib/multi.c

index b0104310bda67f400c66d821bb554152bd9f0d89..f4e15c4138bcbdc1108efe561a04324b76bb898a 100644 (file)
@@ -5,7 +5,7 @@
  *                            | (__| |_| |  _ <| |___
  *                             \___|\___/|_| \_\_____|
  *
- * Copyright (C) 1998 - 2011, Daniel Stenberg, <daniel@haxx.se>, et al.
+ * Copyright (C) 1998 - 2012, Daniel Stenberg, <daniel@haxx.se>, et al.
  *
  * This software is licensed as described in the file COPYING, which
  * you should have received as part of this distribution. The terms
@@ -2018,12 +2018,13 @@ static void singlesocket(struct Curl_multi *multi,
         remove_sock_from_hash = FALSE;
 
       if(remove_sock_from_hash) {
+        /* in this case 'entry' is always non-NULL */
         if(multi->socket_cb)
           multi->socket_cb(easy->easy_handle,
                            s,
                            CURL_POLL_REMOVE,
                            multi->socket_userp,
-                           entry ? entry->socketp : NULL);
+                           entry->socketp);
         sh_delentry(multi->sockhash, s);
       }