- deraadt@cvs.openbsd.org 2005/11/12 18:38:15

     [scp.c]
     avoid close(-1), as in rcp; ok cloder

1 files changed, 10 insertions, 4 deletions

diff --git a/scp.c b/scp.c
index 58c00442f..59285abca 100644
--- a/scp.c
+++ b/scp.c
@@ -71,7 +71,7 @@
  */
 
 #include "includes.h"
-RCSID("$OpenBSD: scp.c,v 1.126 2005/09/13 23:40:07 djm Exp $");
+RCSID("$OpenBSD: scp.c,v 1.127 2005/11/12 18:38:15 deraadt Exp $");
 
 #include "xmalloc.h"
 #include "atomicio.h"
@@ -571,7 +571,10 @@ syserr:			run_err("%s: %s", name, strerror(errno));
                 if (response() < 0)
                         goto next;
                 if ((bp = allocbuf(&buffer, fd, 2048)) == NULL) {
-next:                   (void) close(fd);
+next:                   if (fd != -1) {
+                                (void) close(fd);
+                                fd = -1;
+                        }
                         continue;
                 }
                 if (showprogress)
@@ -600,8 +603,11 @@ next:			(void) close(fd);
                 if (showprogress)
                         stop_progress_meter();
 
-                if (close(fd) < 0 && !haderr)
-                        haderr = errno;
+                if (fd != -1) {
+                        if (close(fd) < 0 && !haderr)
+                                haderr = errno;
+                        fd = -1;
+                }
                 if (!haderr)
                         (void) atomicio(vwrite, remout, "", 1);
                 else