summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarkus Friedl <markus@cvs.openbsd.org>2000-04-19 09:24:40 +0000
committerMarkus Friedl <markus@cvs.openbsd.org>2000-04-19 09:24:40 +0000
commitbb18e8a9ae4fa55452517aba892b7ff6a73b321d (patch)
treee249534d7e8c72788ad2021c624930affd34db27
parent58e8d0ad965cc816de8a0ed682edb13b60d62a2f (diff)
remove bogus chan_read_failed. this could cause data
corruption (missing data) at end of a SSH2 session.
-rw-r--r--usr.bin/ssh/session.c10
1 files changed, 7 insertions, 3 deletions
diff --git a/usr.bin/ssh/session.c b/usr.bin/ssh/session.c
index e21988509c5..3b549c7ef4d 100644
--- a/usr.bin/ssh/session.c
+++ b/usr.bin/ssh/session.c
@@ -8,7 +8,7 @@
*/
#include "includes.h"
-RCSID("$OpenBSD: session.c,v 1.4 2000/04/14 10:30:33 markus Exp $");
+RCSID("$OpenBSD: session.c,v 1.5 2000/04/19 09:24:39 markus Exp $");
#include "xmalloc.h"
#include "ssh.h"
@@ -1321,8 +1321,12 @@ session_exit_message(Session *s, int status)
/* disconnect channel */
debug("session_exit_message: release channel %d", s->chanid);
channel_cancel_cleanup(s->chanid);
- if (c->istate == CHAN_INPUT_OPEN)
- chan_read_failed(c);
+ /*
+ * emulate a write failure with 'chan_write_failed', nobody will be
+ * interested in data we write.
+ * Note that we must not call 'chan_read_failed', since there could
+ * be some more data waiting in the pipe.
+ */
chan_write_failed(c);
s->chanid = -1;
}