diff options
author | Ben Lindstrom <mouring@eviladmin.org> | 2002-02-26 18:02:43 +0000 |
---|---|---|
committer | Ben Lindstrom <mouring@eviladmin.org> | 2002-02-26 18:02:43 +0000 |
commit | 021fcd3a36169ae53b53e9fc7bd2340ee9a5b535 (patch) | |
tree | 3eaf67fa7152d6d1cb1fee896fde1a5801a0bd10 /sftp-client.c | |
parent | 13c5d3b37095a990e6ab836ebe6facddc0ac9cff (diff) |
- markus@cvs.openbsd.org 2002/02/24 16:57:19
[sftp-client.c]
early close(), missing free; ok stevesk@
Diffstat (limited to 'sftp-client.c')
-rw-r--r-- | sftp-client.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/sftp-client.c b/sftp-client.c index 481341c66..17ac14a95 100644 --- a/sftp-client.c +++ b/sftp-client.c | |||
@@ -28,7 +28,7 @@ | |||
28 | /* XXX: copy between two remote sites */ | 28 | /* XXX: copy between two remote sites */ |
29 | 29 | ||
30 | #include "includes.h" | 30 | #include "includes.h" |
31 | RCSID("$OpenBSD: sftp-client.c,v 1.23 2002/02/13 00:59:23 djm Exp $"); | 31 | RCSID("$OpenBSD: sftp-client.c,v 1.24 2002/02/24 16:57:19 markus Exp $"); |
32 | 32 | ||
33 | #if defined(HAVE_SYS_QUEUE_H) && !defined(HAVE_BOGUS_SYS_QUEUE_H) | 33 | #if defined(HAVE_SYS_QUEUE_H) && !defined(HAVE_BOGUS_SYS_QUEUE_H) |
34 | #include <sys/queue.h> | 34 | #include <sys/queue.h> |
@@ -799,7 +799,6 @@ do_download(struct sftp_conn *conn, char *remote_path, char *local_path, | |||
799 | handle = get_handle(conn->fd_in, id, &handle_len); | 799 | handle = get_handle(conn->fd_in, id, &handle_len); |
800 | if (handle == NULL) { | 800 | if (handle == NULL) { |
801 | buffer_free(&msg); | 801 | buffer_free(&msg); |
802 | close(local_fd); | ||
803 | return(-1); | 802 | return(-1); |
804 | } | 803 | } |
805 | 804 | ||
@@ -807,6 +806,8 @@ do_download(struct sftp_conn *conn, char *remote_path, char *local_path, | |||
807 | if (local_fd == -1) { | 806 | if (local_fd == -1) { |
808 | error("Couldn't open local file \"%s\" for writing: %s", | 807 | error("Couldn't open local file \"%s\" for writing: %s", |
809 | local_path, strerror(errno)); | 808 | local_path, strerror(errno)); |
809 | buffer_free(&msg); | ||
810 | xfree(handle); | ||
810 | return(-1); | 811 | return(-1); |
811 | } | 812 | } |
812 | 813 | ||