You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by gs...@apache.org on 2009/08/06 22:04:13 UTC

svn commit: r801802 - /qpid/trunk/qpid/cpp/src/qpid/broker/Broker.cpp

Author: gsim
Date: Thu Aug  6 20:04:13 2009
New Revision: 801802

URL: http://svn.apache.org/viewvc?rev=801802&view=rev
Log:
Stop timer thread before users are deleted (as they don't all clean up prior to being destroyed)


Modified:
    qpid/trunk/qpid/cpp/src/qpid/broker/Broker.cpp

Modified: qpid/trunk/qpid/cpp/src/qpid/broker/Broker.cpp
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/src/qpid/broker/Broker.cpp?rev=801802&r1=801801&r2=801802&view=diff
==============================================================================
--- qpid/trunk/qpid/cpp/src/qpid/broker/Broker.cpp (original)
+++ qpid/trunk/qpid/cpp/src/qpid/broker/Broker.cpp Thu Aug  6 20:04:13 2009
@@ -334,6 +334,10 @@
 Broker::~Broker() {
     shutdown();
     queueEvents.shutdown();
+    //TODO: timer clients should really remove any registered tasks
+    //before the are destroyed; until that is the case, this prevents
+    //their failure to do from crashing the broker
+    timer.stop();
     finalize();                 // Finalize any plugins.
     if (config.auth)
         SaslAuthenticator::fini();



---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:commits-subscribe@qpid.apache.org