You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ac...@apache.org on 2012/07/17 23:18:15 UTC

svn commit: r1362653 - /qpid/branches/0.18/qpid/cpp/src/qpid/broker/Link.cpp

Author: aconway
Date: Tue Jul 17 21:18:14 2012
New Revision: 1362653

URL: http://svn.apache.org/viewvc?rev=1362653&view=rev
Log:
QPID-4133: (Jason Dillaman) Non-responsive peer in federated link can result in entire cluster shutdown

Fix is to not start link processing until the connection is open.

Modified:
    qpid/branches/0.18/qpid/cpp/src/qpid/broker/Link.cpp

Modified: qpid/branches/0.18/qpid/cpp/src/qpid/broker/Link.cpp
URL: http://svn.apache.org/viewvc/qpid/branches/0.18/qpid/cpp/src/qpid/broker/Link.cpp?rev=1362653&r1=1362652&r2=1362653&view=diff
==============================================================================
--- qpid/branches/0.18/qpid/cpp/src/qpid/broker/Link.cpp (original)
+++ qpid/branches/0.18/qpid/cpp/src/qpid/broker/Link.cpp Tue Jul 17 21:18:14 2012
@@ -498,7 +498,9 @@ void Link::maintenanceVisit ()
             }
         }
     }
-    else if (state == STATE_OPERATIONAL && (!active.empty() || !created.empty() || !cancellations.empty()) && connection != 0)
+    else if (state == STATE_OPERATIONAL &&
+             (!active.empty() || !created.empty() || !cancellations.empty()) &&
+             connection && connection->isOpen())
         connection->requestIOProcessing (boost::bind(&Link::ioThreadProcessing, this));
 }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org