- djm@cvs.openbsd.org 2009/01/01 21:14:35

     [channels.c]
     call channel destroy callbacks on receipt of open failure messages.
     fixes client hangs when connecting to a server that has MaxSessions=0
     set spotted by imorgan AT nas.nasa.gov; ok markus@

diff --git a/ChangeLog b/ChangeLog
index f974cd9e94334eb3fe6ed3da70daf7ccf2277515..dea2f6015c9c5e7b8c1db4d8e268f4a17bd9c866 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -31,6 +31,11 @@
      [sshd_config.5]
      add AllowAgentForwarding to available Match keywords list
      ok djm
+   - djm@cvs.openbsd.org 2009/01/01 21:14:35
+     [channels.c]
+     call channel destroy callbacks on receipt of open failure messages.
+     fixes client hangs when connecting to a server that has MaxSessions=0
+     set spotted by imorgan AT nas.nasa.gov; ok markus@
 
 20090107
  - (djm) [uidswap.c] bz#1412: Support >16 supplemental groups in OS X.

diff --git a/channels.c b/channels.c
index c0d335921c56d3642702b8863edc38513fc7cbac..d138882b300b6cc0b7400df46a82e83177fd48f4 100644 (file)
--- a/channels.c
+++ b/channels.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: channels.c,v 1.290 2008/12/09 03:20:42 stevesk Exp $ */
+/* $OpenBSD: channels.c,v 1.291 2009/01/01 21:14:35 djm Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -2311,8 +2311,8 @@ channel_input_open_failure(int type, u_int32_t seq, void *ctxt)
                        xfree(lang);
        }
        packet_check_eom();
-       /* Free the channel.  This will also close the socket. */
-       channel_free(c);
+       /* Schedule the channel for cleanup/deletion. */
+       chan_mark_dead(c);
 }
 
 /* ARGSUSED */