atomics: remove useless SC_ATOMIC_DESTROY

diff --git a/src/app-layer-htp-mem.c b/src/app-layer-htp-mem.c
index ff36ca937cf6d066659c92948c3e366488d238c9..e1893f1e900807f7dfcce1eb0ccf1deadd985084 100644 (file)
--- a/src/app-layer-htp-mem.c
+++ b/src/app-layer-htp-mem.c
@@ -195,13 +195,6 @@ void HTPFree(void *ptr, size_t size)
     HTPDecrMemuse((uint64_t)size);
 }
 
-void HTPDestroyMemcap(void)
-{
-    SC_ATOMIC_DESTROY(htp_config_memcap);
-    SC_ATOMIC_DESTROY(htp_memcap);
-    SC_ATOMIC_DESTROY(htp_memuse);
-}
-
 /**
  * @}
  */

diff --git a/src/app-layer-htp-mem.h b/src/app-layer-htp-mem.h
index 8c5112e1e86f24d6d7e0e08d02c46a440bc86d8c..414b48821a74719e7b48963209fac88f300d80ce 100644 (file)
--- a/src/app-layer-htp-mem.h
+++ b/src/app-layer-htp-mem.h
@@ -22,7 +22,6 @@ void *HTPMalloc(size_t size);
 void *HTPCalloc(size_t n, size_t size);
 void *HTPRealloc(void *ptr, size_t orig_size, size_t size);
 void HTPFree(void *ptr, size_t size);
-void HTPDestroyMemcap(void);
 
 int HTPSetMemcap(uint64_t size);
 uint64_t HTPGetMemcap(void);

diff --git a/src/app-layer-htp.c b/src/app-layer-htp.c
index cb10867fd48b7630230770a37b378179ca0b2ce1..1bf5b2fa173e6f3045d24e8f73d69a4e9a134d49 100644 (file)
--- a/src/app-layer-htp.c
+++ b/src/app-layer-htp.c
@@ -2083,7 +2083,6 @@ void HTPFreeConfig(void)
         htp_config_destroy(htprec->cfg);
         SCFree(htprec);
     }
-    HTPDestroyMemcap();
     SCReturn;
 }
 

diff --git a/src/defrag-hash.c b/src/defrag-hash.c
index 227977c57b5fb018929b2acb1dccc5b066a36513..88a8228560bd13020f9fde6f509607d24bafe3b9 100644 (file)
--- a/src/defrag-hash.c
+++ b/src/defrag-hash.c
@@ -156,7 +156,6 @@ void DefragTrackerRelease(DefragTracker *t)
 void DefragTrackerClearMemory(DefragTracker *dt)
 {
     DefragTrackerFreeFrags(dt);
-    SC_ATOMIC_DESTROY(dt->use_cnt);
 }
 
 #define DEFRAG_DEFAULT_HASHSIZE 4096
@@ -329,12 +328,6 @@ void DefragHashShutdown(void)
     }
     (void) SC_ATOMIC_SUB(defrag_memuse, defrag_config.hash_size * sizeof(DefragTrackerHashRow));
     DefragTrackerQueueDestroy(&defragtracker_spare_q);
-
-    SC_ATOMIC_DESTROY(defragtracker_prune_idx);
-    SC_ATOMIC_DESTROY(defrag_memuse);
-    SC_ATOMIC_DESTROY(defragtracker_counter);
-    SC_ATOMIC_DESTROY(defrag_config.memcap);
-    //SC_ATOMIC_DESTROY(flow_flags);
     return;
 }
 

diff --git a/src/detect-engine-tag.c b/src/detect-engine-tag.c
index df9647357345769a284a1accab2e6fb1268963d9..f2ede4cee0f51e4534b42fec41b0770b472cac73 100644 (file)
--- a/src/detect-engine-tag.c
+++ b/src/detect-engine-tag.c
@@ -75,7 +75,6 @@ void TagDestroyCtx(void)
 #ifdef DEBUG
     BUG_ON(SC_ATOMIC_GET(num_tags) != 0);
 #endif
-    SC_ATOMIC_DESTROY(num_tags);
 }
 
 /** \brief Reset the tagging engine context

diff --git a/src/flow-util.h b/src/flow-util.h
index 33c465d005b86e8a29ecf8d7d8463e4e519ea75d..96ed814c7436e7e92686616ccbaf02d7ba81e42f 100644 (file)
--- a/src/flow-util.h
+++ b/src/flow-util.h
@@ -120,8 +120,6 @@
 
 #define FLOW_DESTROY(f) do { \
         FlowCleanupAppLayer((f)); \
-        SC_ATOMIC_DESTROY((f)->flow_state); \
-        SC_ATOMIC_DESTROY((f)->use_cnt); \
         \
         FLOWLOCK_DESTROY((f)); \
         GenericVarFree((f)->flowvar); \

diff --git a/src/flow-worker.c b/src/flow-worker.c
index 35bfe367006741fdd7aacd308eada03ca0cfd53a..c65483d71566147fcad5b87c33efb4cbf5cba5bd 100644 (file)
--- a/src/flow-worker.c
+++ b/src/flow-worker.c
@@ -170,7 +170,6 @@ static TmEcode FlowWorkerThreadDeinit(ThreadVars *tv, void *data)
     /* free pq */
     BUG_ON(fw->pq.len);
 
-    SC_ATOMIC_DESTROY(fw->detect_thread);
     SCFree(fw);
     return TM_ECODE_OK;
 }

diff --git a/src/flow.c b/src/flow.c
index 7f7ea6f8c072558e23d8c988c33a9c7ff4baa4b1..899e4fad86dfa778adcc7036a96e2d15e9ce64d6 100644 (file)
--- a/src/flow.c
+++ b/src/flow.c
@@ -714,7 +714,6 @@ void FlowShutdown(void)
             }
 
             FBLOCK_DESTROY(&flow_hash[u]);
-            SC_ATOMIC_DESTROY(flow_hash[u].next_ts);
         }
         SCFreeAligned(flow_hash);
         flow_hash = NULL;
@@ -722,11 +721,6 @@ void FlowShutdown(void)
     (void) SC_ATOMIC_SUB(flow_memuse, flow_config.hash_size * sizeof(FlowBucket));
     FlowQueueDestroy(&flow_spare_q);
     FlowQueueDestroy(&flow_recycle_q);
-
-    SC_ATOMIC_DESTROY(flow_config.memcap);
-    SC_ATOMIC_DESTROY(flow_prune_idx);
-    SC_ATOMIC_DESTROY(flow_memuse);
-    SC_ATOMIC_DESTROY(flow_flags);
     return;
 }
 

diff --git a/src/host.c b/src/host.c
index 6b294a8e67d731c7a1af2a750fc25ef14fd83104..efbc1528aa9d93cda3edb95e62e339ae3ccc216e 100644 (file)
--- a/src/host.c
+++ b/src/host.c
@@ -132,8 +132,6 @@ void HostFree(Host *h)
 {
     if (h != NULL) {
         HostClearMemory(h);
-
-        SC_ATOMIC_DESTROY(h->use_cnt);
         SCMutexDestroy(&h->m);
         SCFree(h);
         (void) SC_ATOMIC_SUB(host_memuse, g_host_size);
@@ -334,12 +332,6 @@ void HostShutdown(void)
     }
     (void) SC_ATOMIC_SUB(host_memuse, host_config.hash_size * sizeof(HostHashRow));
     HostQueueDestroy(&host_spare_q);
-
-    SC_ATOMIC_DESTROY(host_prune_idx);
-    SC_ATOMIC_DESTROY(host_memuse);
-    SC_ATOMIC_DESTROY(host_counter);
-    SC_ATOMIC_DESTROY(host_config.memcap);
-    //SC_ATOMIC_DESTROY(flow_flags);
     return;
 }
 

diff --git a/src/ippair.c b/src/ippair.c
index 0af92d2692417d47e99588e8d22b2b8641f9019b..900cb2fb85906dbb687aa2bb13632377df5ce609 100644 (file)
--- a/src/ippair.c
+++ b/src/ippair.c
@@ -131,8 +131,6 @@ void IPPairFree(IPPair *h)
 {
     if (h != NULL) {
         IPPairClearMemory(h);
-
-        SC_ATOMIC_DESTROY(h->use_cnt);
         SCMutexDestroy(&h->m);
         SCFree(h);
         (void) SC_ATOMIC_SUB(ippair_memuse, g_ippair_size);
@@ -329,12 +327,6 @@ void IPPairShutdown(void)
     }
     (void) SC_ATOMIC_SUB(ippair_memuse, ippair_config.hash_size * sizeof(IPPairHashRow));
     IPPairQueueDestroy(&ippair_spare_q);
-
-    SC_ATOMIC_DESTROY(ippair_prune_idx);
-    SC_ATOMIC_DESTROY(ippair_memuse);
-    SC_ATOMIC_DESTROY(ippair_counter);
-    SC_ATOMIC_DESTROY(ippair_config.memcap);
-    //SC_ATOMIC_DESTROY(flow_flags);
     return;
 }
 

diff --git a/src/source-af-packet.c b/src/source-af-packet.c
index e64af60b816409c50715f9da2e631135206fd797..c38066155877cdfa3f186f476c69c126a2bd4f57 100644 (file)
--- a/src/source-af-packet.c
+++ b/src/source-af-packet.c
@@ -384,9 +384,6 @@ static void AFPPeerClean(AFPPeer *peer)
 {
     if (peer->flags & AFP_SOCK_PROTECT)
         SCMutexDestroy(&peer->sock_protect);
-    SC_ATOMIC_DESTROY(peer->socket);
-    SC_ATOMIC_DESTROY(peer->if_idx);
-    SC_ATOMIC_DESTROY(peer->state);
     SCFree(peer);
 }
 

diff --git a/src/stream-tcp.c b/src/stream-tcp.c
index 51fe213c7b0486504b12f60d07bd9c9d246df668..ec3b7fa896412aa5251c23c04520c62b6b3a9a12 100644 (file)
--- a/src/stream-tcp.c
+++ b/src/stream-tcp.c
@@ -668,9 +668,6 @@ void StreamTcpInitConfig(char quiet)
 
 void StreamTcpFreeConfig(char quiet)
 {
-    SC_ATOMIC_DESTROY(stream_config.memcap);
-    SC_ATOMIC_DESTROY(stream_config.reassembly_memcap);
-
     StreamTcpReassembleFree(quiet);
 
     SCMutexLock(&ssn_pool_mutex);

diff --git a/src/suricata.c b/src/suricata.c
index 84e3b8340601206d64e2191fa32a23ae2c2ee91e..bb92e2cf6c54e5a52119725eda4ed3d051314f3c 100644 (file)
--- a/src/suricata.c
+++ b/src/suricata.c
@@ -371,8 +371,6 @@ static void GlobalsDestroy(SCInstance *suri)
     NFQContextsClean();
 #endif
 
-    SC_ATOMIC_DESTROY(engine_stage);
-
 #ifdef BUILD_HYPERSCAN
     MpmHSGlobalCleanup();
 #endif

diff --git a/src/tmqh-packetpool.c b/src/tmqh-packetpool.c
index cea1212197b7ae782606c2c79994b300a21888e8..b8b963022347319c93b638813833bec19c545913 100644 (file)
--- a/src/tmqh-packetpool.c
+++ b/src/tmqh-packetpool.c
@@ -433,8 +433,6 @@ void PacketPoolDestroy(void)
         PacketFree(p);
     }
 
-    SC_ATOMIC_DESTROY(my_pool->return_stack.sync_now);
-
 #ifdef DEBUG_VALIDATION
     my_pool->initialized = 0;
     my_pool->destroyed = 1;

diff --git a/src/util-atomic.h b/src/util-atomic.h
index 3424d04f01cbea6f6b97afa30b7db9ff22a0cd2f..ba38ffc1b137e7d79f7037a1bb18f5afc53e3d24 100644 (file)
--- a/src/util-atomic.h
+++ b/src/util-atomic.h
@@ -178,11 +178,6 @@
 #define SC_ATOMIC_RESET(name) \
     (name ## _sc_atomic__) = 0
 
-/**
- *  \brief No-op.
- */
-#define SC_ATOMIC_DESTROY(name)
-
 /**
  *  \brief add a value to our atomic variable
  *

diff --git a/src/util-device.c b/src/util-device.c
index 4c0779ea6307ed079928c903c6e40103fc0e50ef..67f47a7934ed2ebc00f5ece1b76590fb5a6a28a1 100644 (file)
--- a/src/util-device.c
+++ b/src/util-device.c
@@ -364,9 +364,6 @@ int LiveDeviceListClean()
 
         if (pd->dev)
             SCFree(pd->dev);
-        SC_ATOMIC_DESTROY(pd->pkts);
-        SC_ATOMIC_DESTROY(pd->drop);
-        SC_ATOMIC_DESTROY(pd->invalid_checksums);
         LiveDevFreeStorage(pd);
         SCFree(pd);
     }

diff --git a/src/util-thash.c b/src/util-thash.c
index cdce4edbd4e10e3afb960ee16bad29330cb070f3..abe6131c7e004816afc655162eee27f3c6310529 100644 (file)
--- a/src/util-thash.c
+++ b/src/util-thash.c
@@ -189,8 +189,6 @@ static void THashDataFree(THashTableContext *ctx, THashData *h)
         if (h->data != NULL) {
             ctx->config.DataFree(h->data);
         }
-
-        SC_ATOMIC_DESTROY(h->use_cnt);
         SCMutexDestroy(&h->m);
         SCFree(h);
         (void) SC_ATOMIC_SUB(ctx->memuse, THASH_DATA_SIZE(ctx));
@@ -352,11 +350,6 @@ void THashShutdown(THashTableContext *ctx)
     }
     (void) SC_ATOMIC_SUB(ctx->memuse, ctx->config.hash_size * sizeof(THashHashRow));
     THashDataQueueDestroy(&ctx->spare_q);
-
-    SC_ATOMIC_DESTROY(ctx->prune_idx);
-    SC_ATOMIC_DESTROY(ctx->memuse);
-    SC_ATOMIC_DESTROY(ctx->counter);
-
     SCFree(ctx);
     return;
 }