From: Tom Peters (thopeter) <thopeter@cisco.com>
Date: Wed, 19 May 2021 20:08:05 +0000 (+0000)
Subject: Merge pull request #2890 in SNORT/snort3 from ~MDAGON/snort3:memory to master
X-Git-Tag: 3.1.5.0~1
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b3e17118a58ab9c27495086c12f900cf91ed721a;p=thirdparty%2Fsnort3.git

Merge pull request #2890 in SNORT/snort3 from ~MDAGON/snort3:memory to master

Squashed commit of the following:

commit 506959d13ea92852540f493e0d3818ed3977e6e4
Author: Maya Dagon <mdagon@cisco.com>
Date:   Mon May 17 13:15:59 2021 -0400

    memory: free_space cleanup
---

diff --git a/src/flow/test/flow_test.cc b/src/flow/test/flow_test.cc
index 17e87e95d..f33049567 100644
--- a/src/flow/test/flow_test.cc
+++ b/src/flow/test/flow_test.cc
@@ -51,7 +51,7 @@ void memory::MemoryCap::update_allocations(size_t) {}
 
 void memory::MemoryCap::update_deallocations(size_t) {}
 
-bool memory::MemoryCap::free_space(size_t) { return false; }
+void memory::MemoryCap::free_space(size_t) { }
 
 bool HighAvailabilityManager::active() { return false; }
 
diff --git a/src/memory/memory_cap.cc b/src/memory/memory_cap.cc
index f2f91cb71..49d71751f 100644
--- a/src/memory/memory_cap.cc
+++ b/src/memory/memory_cap.cc
@@ -124,25 +124,22 @@ size_t MemoryCap::preemptive_threshold = 0;
 // public interface
 // -----------------------------------------------------------------------------
 
-bool MemoryCap::free_space(size_t n)
+void MemoryCap::free_space(size_t n)
 {
     if ( !is_packet_thread() )
-        return true;
+        return;
 
     if ( !thread_cap )
-        return true;
+        return;
 
     static THREAD_LOCAL bool entered = false;
-    assert(!entered);
-
+    
     if ( entered )
-        return false;
+        return;
 
     entered = true;
-    bool avail = memory::free_space(n, thread_cap, s_tracker, prune_handler);
+    memory::free_space(n, thread_cap, s_tracker, prune_handler);
     entered = false;
-
-    return avail;
 }
 
 static size_t fudge_it(size_t n)
diff --git a/src/memory/memory_cap.h b/src/memory/memory_cap.h
index 8a170e376..b28bb47e7 100644
--- a/src/memory/memory_cap.h
+++ b/src/memory/memory_cap.h
@@ -31,7 +31,7 @@ namespace memory
 class SO_PUBLIC MemoryCap
 {
 public:
-    static bool free_space(size_t);
+    static void free_space(size_t);
     // The following functions perform internal rounding. Allocations and deallocations must be
     // performed in identical increments or leakage may occur.
     static void update_allocations(size_t);