- djm@cvs.openbsd.org 2010/01/28 00:21:18

     [clientloop.c]
     downgrade an error() to a debug() - this particular case can be hit in
     normal operation for certain sequences of mux slave vs session closure
     and is harmless

diff --git a/ChangeLog b/ChangeLog
index 8e427cc41a8bf379d26428dd0aa23a88b5333e25..f9a84fddf2030bc6a0952ad836c00d52371b7ce6 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+20100130
+ - (djm) OpenBSD CVS Sync
+   - djm@cvs.openbsd.org 2010/01/28 00:21:18
+     [clientloop.c]
+     downgrade an error() to a debug() - this particular case can be hit in
+     normal operation for certain sequences of mux slave vs session closure
+     and is harmless
+
 20100129
  - (dtucker) [openbsd-compat/openssl-compat.c] Bug #1707: Call OPENSSL_config()
    after registering the hardware engines, which causes the openssl.cnf file to

diff --git a/clientloop.c b/clientloop.c
index 05f4720a1d5b7fb0678bac3a75e7795c6c2cab1d..6ffef95a2d775a217cedd4292023bd570c5d6008 100644 (file)
--- a/clientloop.c
+++ b/clientloop.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: clientloop.c,v 1.217 2010/01/26 01:28:35 djm Exp $ */
+/* $OpenBSD: clientloop.c,v 1.218 2010/01/28 00:21:18 djm Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -1857,8 +1857,9 @@ client_input_channel_req(int type, u_int32_t seq, void *ctxt)
                        success = 1;
                        exit_status = exitval;
                } else {
-                       error("client_input_channel_req: unexpected channel %d",
-                           session_ident);
+                       /* Probably for a mux channel that has already closed */
+                       debug("%s: no sink for exit-status on channel %d",
+                           __func__, id);
                }
                packet_check_eom();
        }