You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by pm...@apache.org on 2015/03/05 12:37:36 UTC

svn commit: r1664313 - /qpid/trunk/qpid/cpp/src/qpid/management/ManagementAgent.cpp

Author: pmoravec
Date: Thu Mar  5 11:37:35 2015
New Revision: 1664313

URL: http://svn.apache.org/r1664313
Log:
QPID-6397: [C++ broker] segfault when processing QMF method during periodic processing

Modified:
    qpid/trunk/qpid/cpp/src/qpid/management/ManagementAgent.cpp

Modified: qpid/trunk/qpid/cpp/src/qpid/management/ManagementAgent.cpp
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/src/qpid/management/ManagementAgent.cpp?rev=1664313&r1=1664312&r2=1664313&view=diff
==============================================================================
--- qpid/trunk/qpid/cpp/src/qpid/management/ManagementAgent.cpp (original)
+++ qpid/trunk/qpid/cpp/src/qpid/management/ManagementAgent.cpp Thu Mar  5 11:37:35 2015
@@ -706,8 +706,8 @@ void ManagementAgent::moveNewObjects()
 void ManagementAgent::periodicProcessing (void)
 {
 #define HEADROOM  4096
-    debugSnapshot("Management agent periodic processing");
     sys::Mutex::ScopedLock lock (userLock);
+    debugSnapshot("Management agent periodic processing");
     string              routingKey;
     string sBuf;
 
@@ -2704,6 +2704,8 @@ string ManagementAgent::summarizeAgents(
 
 
 void ManagementAgent::debugSnapshot(const char* title) {
+    sys::Mutex::ScopedLock lock(addLock);
+    sys::Mutex::ScopedLock objLock (objectLock);
     QPID_LOG(debug, title << ": management snapshot: "
              << packages.size() << " packages, "
              << summarizeMap("objects", managementObjects)



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