Merge pull request #402 from SlideRuleEarth/clustersoc-memleak

fixed memory leak in clustersocket
SlideRuleEarth · May 16, 2024 · dca5431 · dca5431
2 parents 6c928da + 5c29650
commit dca5431
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/packages/core/ClusterSocket.cpp b/packages/core/ClusterSocket.cpp
@@ -161,6 +161,8 @@ ClusterSocket::~ClusterSocket(void)
         SockLib::sockclose(fd);
         fd = write_connections.next( &writeCon );
     }
+
+    /* Table desctructor will free connections memory */
 }
 
 /*----------------------------------------------------------------------------
@@ -193,7 +195,7 @@ int ClusterSocket::writeBuffer(const void *buf, int len, int timeout)
     {
         return TIMEOUT_RC;
     }
-    
+
     if(len <= MAX_MSG_SIZE)
     {
         int status = pubsockq->postCopy(buf, len, timeout);

diff --git a/packages/core/Table.h b/packages/core/Table.h
@@ -167,6 +167,9 @@ Table<T,K>::Table(K table_size, hash_func_t _hash)
 template <class T, typename K>
 Table<T,K>::~Table(void)
 {
+    /* Free All Nodes */
+    clear();
+
     /* Free Hash Structure */
     delete [] table;
 }
@@ -304,7 +307,7 @@ T& Table<T,K>::get(K key, match_t match, bool resort)
             best_index = curr_index;
             break;
         }
-        
+
         if(match == MATCH_NEAREST_UNDER)
         {
             if(table[curr_index].key < key)