up: Notice errors on both file handles

Report errors on both filehandles in up's Copy routine, rather
than just on the second one that we close.

Caught by clang-analyzer

Change-Id: Ia3aa9be91ccfca2993a4f27c291332e0b0d350f5
Reviewed-on: http://gerrit.openafs.org/9192
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>

diff --git a/src/venus/up.c b/src/venus/up.c
index 522dc08b44ce68b3ba65b348217047f5af35238e..f253875a9e4db9aa01cefa28fe4e7a197b7a46b6 100644 (file)
--- a/src/venus/up.c
+++ b/src/venus/up.c
@@ -296,6 +296,10 @@ Copy(char *file1, char *file2, short recursive, int level)
 
            /* Close the files */
            code = close(f1);
+           if (code < 0) {
+               perror("close ");
+               rcode = 1;
+           }
            code = close(f2);
            if (code < 0) {
                perror("close ");