- djm@cvs.openbsd.org 2008/01/19 22:04:57
[sftp-client.c] fix remote handle leak in do_download() local file open error path; report and fix from sworley AT chkno.net
This commit is contained in:
parent
d39a3cffc9
commit
6b0c818568
|
@ -19,6 +19,10 @@
|
||||||
ignore SIGPIPE in multiplex client mode - we can receive this if the
|
ignore SIGPIPE in multiplex client mode - we can receive this if the
|
||||||
server runs out of fds on us midway. Report and patch from
|
server runs out of fds on us midway. Report and patch from
|
||||||
gregory_shively AT fanniemae.com
|
gregory_shively AT fanniemae.com
|
||||||
|
- djm@cvs.openbsd.org 2008/01/19 22:04:57
|
||||||
|
[sftp-client.c]
|
||||||
|
fix remote handle leak in do_download() local file open error path;
|
||||||
|
report and fix from sworley AT chkno.net
|
||||||
|
|
||||||
20080119
|
20080119
|
||||||
- (djm) Silence noice from expr in ssh-copy-id; patch from
|
- (djm) Silence noice from expr in ssh-copy-id; patch from
|
||||||
|
@ -3547,4 +3551,4 @@
|
||||||
OpenServer 6 and add osr5bigcrypt support so when someone migrates
|
OpenServer 6 and add osr5bigcrypt support so when someone migrates
|
||||||
passwords between UnixWare and OpenServer they will still work. OK dtucker@
|
passwords between UnixWare and OpenServer they will still work. OK dtucker@
|
||||||
|
|
||||||
$Id: ChangeLog,v 1.4823 2008/02/10 11:23:18 djm Exp $
|
$Id: ChangeLog,v 1.4824 2008/02/10 11:23:41 djm Exp $
|
||||||
|
|
|
@ -1,4 +1,4 @@
|
||||||
/* $OpenBSD: sftp-client.c,v 1.78 2008/01/11 07:22:27 chl Exp $ */
|
/* $OpenBSD: sftp-client.c,v 1.79 2008/01/19 22:04:57 djm Exp $ */
|
||||||
/*
|
/*
|
||||||
* Copyright (c) 2001-2004 Damien Miller <djm@openbsd.org>
|
* Copyright (c) 2001-2004 Damien Miller <djm@openbsd.org>
|
||||||
*
|
*
|
||||||
|
@ -823,6 +823,7 @@ do_download(struct sftp_conn *conn, char *remote_path, char *local_path,
|
||||||
if (local_fd == -1) {
|
if (local_fd == -1) {
|
||||||
error("Couldn't open local file \"%s\" for writing: %s",
|
error("Couldn't open local file \"%s\" for writing: %s",
|
||||||
local_path, strerror(errno));
|
local_path, strerror(errno));
|
||||||
|
do_close(conn, handle, handle_len);
|
||||||
buffer_free(&msg);
|
buffer_free(&msg);
|
||||||
xfree(handle);
|
xfree(handle);
|
||||||
return(-1);
|
return(-1);
|
||||||
|
|
Loading…
Reference in New Issue