(mesos) branch master updated: [FIX] Add missing memory deallocation in port_mapping_tests.

[bmahler]

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

bmahler pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git


The following commit(s) were added to refs/heads/master by this push:
     new 2ef09fa64 [FIX] Add missing memory deallocation in port_mapping_tests.
2ef09fa64 is described below

commit 2ef09fa64928b13d02958991526da14332009077
Author: Devin Leamy <dle...@twitter.com>
AuthorDate: Wed Jan 24 19:18:34 2024 +0000

    [FIX] Add missing memory deallocation in port_mapping_tests.
    
    Several port mapping isolator tests were missing deallocations,
    of structures used in the tests. This commit adds these
    deallocations where they are missing.
---
 src/tests/containerizer/port_mapping_tests.cpp | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/src/tests/containerizer/port_mapping_tests.cpp 
b/src/tests/containerizer/port_mapping_tests.cpp
index cfc2fdcaa..ca7588723 100644
--- a/src/tests/containerizer/port_mapping_tests.cpp
+++ b/src/tests/containerizer/port_mapping_tests.cpp
@@ -1844,6 +1844,9 @@ TEST_F(PortMappingIsolatorTest, ROOT_ScaleEgressWithCPU)
   // Kill the container
   AWAIT_READY(launcher.get()->destroy(containerId1));
   AWAIT_READY(isolator.get()->cleanup(containerId1));
+
+  delete launcher.get();
+  delete isolator.get();
 }
 
 
@@ -2083,6 +2086,9 @@ TEST_F(PortMappingIsolatorTest, ROOT_ScaleIngressWithCPU)
   // Kill the container
   AWAIT_READY(launcher.get()->destroy(containerId1));
   AWAIT_READY(isolator.get()->cleanup(containerId1));
+
+  delete launcher.get();
+  delete isolator.get();
 }
 
 
@@ -2213,6 +2219,9 @@ TEST_F(PortMappingIsolatorTest, 
ROOT_ScaleIngressWithCPUAutoConfig)
   // Kill the container
   AWAIT_READY(launcher.get()->destroy(containerId1));
   AWAIT_READY(isolator.get()->cleanup(containerId1));
+
+  delete launcher.get();
+  delete isolator.get();
 }