Coverity: Remove unreachable code.
authorRichard W.M. Jones <rjones@redhat.com>
Wed, 8 Jun 2011 20:53:07 +0000 (21:53 +0100)
committerRichard W.M. Jones <rjones@redhat.com>
Thu, 9 Jun 2011 09:53:12 +0000 (10:53 +0100)
daemon/base64.c
daemon/debug.c
daemon/tar.c
daemon/upload.c

index f55e1f5..8aca5b7 100644 (file)
@@ -86,8 +86,6 @@ do_base64_in (const char *file)
   }
 
   if (pclose (fp) != 0) {
-    if (r == -1)                /* if r == 0, file transfer ended already */
-      cancel_receive ();
     reply_with_error ("base64 subcommand failed on file: %s", file);
     return -1;
   }
index cd3e8a5..0ade2a4 100644 (file)
@@ -549,10 +549,6 @@ do_debug_upload (const char *filename, int mode)
   }
 
   if (close (fd) == -1) {
-    int err = errno;
-    if (r == -1)                /* if r == 0, file transfer ended already */
-      cancel_receive ();
-    errno = err;
     reply_with_perror ("close: %s", filename);
     return -1;
   }
index 39fce8c..74d7b05 100644 (file)
@@ -127,8 +127,6 @@ do_tXz_in (const char *dir, const char *filter)
   }
 
   if (pclose (fp) != 0) {
-    if (r == -1)                /* if r == 0, file transfer ended already */
-      r = cancel_receive ();
     char *errstr = read_error_file ();
     reply_with_error ("tar subcommand failed on directory: %s: %s",
                       dir, errstr);
index 2c42969..a1a37ae 100644 (file)
@@ -103,10 +103,6 @@ upload (const char *filename, int flags, int64_t offset)
   }
 
   if (close (data.fd) == -1) {
-    err = errno;
-    if (r == -1)                /* if r == 0, file transfer ended already */
-      r = cancel_receive ();
-    errno = err;
     reply_with_perror ("close: %s", filename);
     return -1;
   }