diff options
| author | Kenneth Anthony Giusti <kgiusti@apache.org> | 2010-09-24 15:24:38 +0000 |
|---|---|---|
| committer | Kenneth Anthony Giusti <kgiusti@apache.org> | 2010-09-24 15:24:38 +0000 |
| commit | aa3b721d0ea89abe89ba61f9b4eb17768d9ea17c (patch) | |
| tree | 5ae20a2e5c370bfecc92c7cd208c427655b2e96d /qpid/cpp | |
| parent | 5f4a31f34404ac321414dd8d02f5d1b90213b6d9 (diff) | |
| download | qpid-python-aa3b721d0ea89abe89ba61f9b4eb17768d9ea17c.tar.gz | |
QPID-2885: clean up memory leaks on C++ qmf console shutdown.
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1000907 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'qpid/cpp')
| -rw-r--r-- | qpid/cpp/src/qpid/console/Broker.cpp | 5 | ||||
| -rw-r--r-- | qpid/cpp/src/qpid/console/SessionManager.cpp | 9 |
2 files changed, 14 insertions, 0 deletions
diff --git a/qpid/cpp/src/qpid/console/Broker.cpp b/qpid/cpp/src/qpid/console/Broker.cpp index d2ff8f819e..86a17d4a10 100644 --- a/qpid/cpp/src/qpid/console/Broker.cpp +++ b/qpid/cpp/src/qpid/console/Broker.cpp @@ -59,6 +59,11 @@ Broker::~Broker() { connThreadBody.shutdown(); connThread.join(); + resetAgents(); + // resetAgents() does not delete the broker agent... + for (AgentMap::iterator iter = agents.begin(); iter != agents.end(); iter++) { + delete iter->second; + } } string Broker::getUrl() const diff --git a/qpid/cpp/src/qpid/console/SessionManager.cpp b/qpid/cpp/src/qpid/console/SessionManager.cpp index 4f39095a54..80c5959417 100644 --- a/qpid/cpp/src/qpid/console/SessionManager.cpp +++ b/qpid/cpp/src/qpid/console/SessionManager.cpp @@ -46,6 +46,15 @@ SessionManager::~SessionManager() for (vector<Broker*>::iterator iter = brokers.begin(); iter != brokers.end(); iter++) delete *iter; + + for (map<string, Package*>::iterator iter = packages.begin(); + iter != packages.end(); iter++) { + for (Package::ClassMap::iterator citer = iter->second->classes.begin(); + citer != iter->second->classes.end(); + citer++) + delete citer->second; + delete iter->second; + } } Broker* SessionManager::addBroker(client::ConnectionSettings& settings) |