[geode-native] branch develop updated: GEODE-3645: Minimal fix of SEGV in cache destructor

[dg...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

dgkimura pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode-native.git


The following commit(s) were added to refs/heads/develop by this push:
     new 1212c66  GEODE-3645: Minimal fix of SEGV in cache destructor
1212c66 is described below

commit 1212c66f9bea583447d94e0b86ee9329b019e301
Author: David Kimura <dk...@pivotal.io>
AuthorDate: Mon Dec 11 09:29:26 2017 -0800

    GEODE-3645: Minimal fix of SEGV in cache destructor
---
 cppcache/src/PoolManagerImpl.cpp  | 1 -
 cppcache/src/ThinClientPoolDM.cpp | 3 +++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/cppcache/src/PoolManagerImpl.cpp b/cppcache/src/PoolManagerImpl.cpp
index c3fbeb4..a91ef56 100644
--- a/cppcache/src/PoolManagerImpl.cpp
+++ b/cppcache/src/PoolManagerImpl.cpp
@@ -44,7 +44,6 @@ void PoolManagerImpl::close(bool keepAlive) {
 
   for (const auto& iter : poolsList) {
     iter->destroy(keepAlive);
-    m_connectionPools.erase(iter->getName().c_str());
   }
 }
 
diff --git a/cppcache/src/ThinClientPoolDM.cpp b/cppcache/src/ThinClientPoolDM.cpp
index 60d8d04..cc26b8f 100644
--- a/cppcache/src/ThinClientPoolDM.cpp
+++ b/cppcache/src/ThinClientPoolDM.cpp
@@ -850,6 +850,9 @@ void ThinClientPoolDM::destroy(bool keepAlive) {
       GF_SAFE_DELETE(m_clientMetadataService);
     }
 
+    m_connManager.getCacheImpl()->getPoolManager().removePool(
+        m_poolName.c_str());
+
     stopChunkProcessor();
     m_manager->closeAllStickyConnections();
     m_isDestroyed = true;

-- 
To stop receiving notification emails like this one, please contact
['"commits@geode.apache.org" <co...@geode.apache.org>'].