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 2008/06/09 15:08:44 UTC
svn commit: r664695 -
/incubator/qpid/trunk/qpid/cpp/src/qpid/broker/SessionManager.cpp
Author: aconway
Date: Mon Jun 9 06:08:44 2008
New Revision: 664695
URL: http://svn.apache.org/viewvc?rev=664695&view=rev
Log:
Missing lock in SessionManager::forget()
Modified:
incubator/qpid/trunk/qpid/cpp/src/qpid/broker/SessionManager.cpp
Modified: incubator/qpid/trunk/qpid/cpp/src/qpid/broker/SessionManager.cpp
URL: http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/cpp/src/qpid/broker/SessionManager.cpp?rev=664695&r1=664694&r2=664695&view=diff
==============================================================================
--- incubator/qpid/trunk/qpid/cpp/src/qpid/broker/SessionManager.cpp (original)
+++ incubator/qpid/trunk/qpid/cpp/src/qpid/broker/SessionManager.cpp Mon Jun 9 06:08:44 2008
@@ -81,7 +81,10 @@
}
}
-void SessionManager::forget(const SessionId& id) { attached.erase(id); }
+void SessionManager::forget(const SessionId& id) {
+ Mutex::ScopedLock l(lock);
+ attached.erase(id);
+}
void SessionManager::eraseExpired() {
// Called with lock held.