[Libunwind-devel] [PATCH 26/57] Cleanup dynamically allocated memory before exit in tests

[Tommi Rantala]

Cleanup dynamically allocated memory before exit in tests in a few
places where missing. While such cleanups right before exit do not
usually make much sense (as the operating system would cleanup anyway,
so manual cleanups only burn CPU cycles), we will want to catch any
potential problems in libunwind related to the cleanups. This also stops
valgrind complaining about unreleased memory.
---
 tests/Gtest-bt.c             |    6 ++++++
 tests/Gtest-trace.c          |    6 ++++++
 tests/test-coredump-unwind.c |    1 +
 tests/test-proc-info.c       |    2 ++
 tests/test-ptrace.c          |    1 +
 5 files changed, 16 insertions(+)

diff --git a/tests/Gtest-bt.c b/tests/Gtest-bt.c
index 1bc3f4d..4a07c6f 100644
--- a/tests/Gtest-bt.c
+++ b/tests/Gtest-bt.c
@@ -252,5 +252,11 @@ main (int argc, char **argv __attribute__((unused)))
     }
   if (verbose)
     printf ("SUCCESS.\n");
+
+  signal (SIGTERM, SIG_DFL);
+  stk.ss_flags = SS_DISABLE;
+  sigaltstack (&stk, NULL);
+  free (stk.ss_sp);
+
   return 0;
 }
diff --git a/tests/Gtest-trace.c b/tests/Gtest-trace.c
index 9d43ce2..e8dd989 100644
--- a/tests/Gtest-trace.c
+++ b/tests/Gtest-trace.c
@@ -267,5 +267,11 @@ main (int argc, char **argv __attribute__((unused)))
 
   if (verbose)
     printf ("SUCCESS.\n");
+
+  signal (SIGTERM, SIG_DFL);
+  stk.ss_flags = SS_DISABLE;
+  sigaltstack (&stk, NULL);
+  free (stk.ss_sp);
+
   return 0;
 }
diff --git a/tests/test-coredump-unwind.c b/tests/test-coredump-unwind.c
index fc784d4..d7c7954 100644
--- a/tests/test-coredump-unwind.c
+++ b/tests/test-coredump-unwind.c
@@ -389,6 +389,7 @@ main(int argc __attribute__((unused)), char **argv)
     }
 
   _UCD_destroy(ui);
+  unw_destroy_addr_space(as);
 
   return 0;
 }
diff --git a/tests/test-proc-info.c b/tests/test-proc-info.c
index 3c8fdea..e97ef2b 100644
--- a/tests/test-proc-info.c
+++ b/tests/test-proc-info.c
@@ -162,6 +162,8 @@ main (int argc, char **argv)
     panic ("Second call to unw_step() returned %d instead of %d\n",
           ret, -UNW_ESTOPUNWIND);
 
+  unw_destroy_addr_space (as);
+
   if (verbose)
     printf ("SUCCESS\n");
   return 0;
diff --git a/tests/test-ptrace.c b/tests/test-ptrace.c
index 17a1b61..aecb06e 100644
--- a/tests/test-ptrace.c
+++ b/tests/test-ptrace.c
@@ -341,6 +341,7 @@ main (int argc, char **argv)
     }
 
   _UPT_destroy (ui);
+  unw_destroy_addr_space (as);
 
   if (nerrors)
     {
-- 
1.7.9.5