]> granicus.if.org Git - curl/commitdiff
do_sec_send: remove dead code
authorDaniel Stenberg <daniel@haxx.se>
Wed, 8 Oct 2014 10:48:06 +0000 (12:48 +0200)
committerDaniel Stenberg <daniel@haxx.se>
Wed, 8 Oct 2014 10:48:06 +0000 (12:48 +0200)
Coverity CID 1241951. The condition 'len >= 0' would always be true at
that point and thus not necessary to check for.

lib/security.c

index 25cfa21706ce7c7c3e1592c33e303b8e242314b4..d1c0369f9b8400023bbe6fda98ac09a731cc13dc 100644 (file)
@@ -332,7 +332,6 @@ static void do_sec_send(struct connectdata *conn, curl_socket_t fd,
 static ssize_t sec_write(struct connectdata *conn, curl_socket_t fd,
                          const char *buffer, size_t length)
 {
-  /* FIXME: Check for overflow */
   ssize_t tx = 0, len = conn->buffer_size;
 
   len -= conn->mech->overhead(conn->app_data, conn->data_prot,
@@ -340,10 +339,9 @@ static ssize_t sec_write(struct connectdata *conn, curl_socket_t fd,
   if(len <= 0)
     len = length;
   while(length) {
-    if(len >= 0 || length < (size_t)len) {
-      /* FIXME: Check for overflow. */
+    if(length < (size_t)len)
       len = length;
-    }
+
     do_sec_send(conn, fd, buffer, curlx_sztosi(len));
     length -= len;
     buffer += len;