Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/crypto/external/bsd/openssh/dist CID 1129615: close argument...



details:   https://anonhg.NetBSD.org/src/rev/060159fdf953
branches:  trunk
changeset: 791468:060159fdf953
user:      christos <christos%NetBSD.org@localhost>
date:      Tue Nov 19 16:58:16 2013 +0000

description:
CID 1129615: close argument can't be negative

diffstat:

 crypto/external/bsd/openssh/dist/sftp-client.c |  7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diffs (28 lines):

diff -r 7e3bc7774cbb -r 060159fdf953 crypto/external/bsd/openssh/dist/sftp-client.c
--- a/crypto/external/bsd/openssh/dist/sftp-client.c    Tue Nov 19 16:56:21 2013 +0000
+++ b/crypto/external/bsd/openssh/dist/sftp-client.c    Tue Nov 19 16:58:16 2013 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: sftp-client.c,v 1.10 2013/11/11 16:46:20 christos Exp $        */
+/*     $NetBSD: sftp-client.c,v 1.11 2013/11/19 16:58:16 christos Exp $        */
 /* $OpenBSD: sftp-client.c,v 1.101.2.1 2013/11/08 01:33:56 djm Exp $ */
 /*
  * Copyright (c) 2001-2004 Damien Miller <djm%openbsd.org@localhost>
@@ -22,7 +22,7 @@
 /* XXX: copy between two remote sites */
 
 #include "includes.h"
-__RCSID("$NetBSD: sftp-client.c,v 1.10 2013/11/11 16:46:20 christos Exp $");
+__RCSID("$NetBSD: sftp-client.c,v 1.11 2013/11/19 16:58:16 christos Exp $");
 #include <sys/types.h>
 #include <sys/poll.h>
 #include <sys/queue.h>
@@ -1069,7 +1069,8 @@
                        do_close(conn, handle, handle_len);
                        buffer_free(&msg);
                        free(handle);
-                       close(local_fd);
+                       if (local_fd != -1)
+                               close(local_fd);
                        return -1;
                }
                offset = highwater = st.st_size;



Home | Main Index | Thread Index | Old Index