Change in gapk[master]: tests: make both Valgrind and LeakSanitizer happy

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Vadim Yanitskiy gerrit-no-reply at lists.osmocom.org
Sat Jun 30 14:23:18 UTC 2018


Vadim Yanitskiy has uploaded this change for review. ( https://gerrit.osmocom.org/9793


Change subject: tests: make both Valgrind and LeakSanitizer happy
......................................................................

tests: make both Valgrind and LeakSanitizer happy

The talloc_enable_null_tracking() actually allocates a new talloc
context, which makes both Valgrind and LeakSanitizer angry. This
context should be freed by the talloc_disable_null_tracking().

Change-Id: Ia660d2fdac720f685c0186720d0a476d7e9468be
---
M tests/ecu/ecu_fr_test.c
M tests/io/pq_file_test.c
M tests/io/pq_rtp_test.c
M tests/procqueue/pq_test.c
4 files changed, 12 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/gapk refs/changes/93/9793/1

diff --git a/tests/ecu/ecu_fr_test.c b/tests/ecu/ecu_fr_test.c
index 372df25..609112f 100644
--- a/tests/ecu/ecu_fr_test.c
+++ b/tests/ecu/ecu_fr_test.c
@@ -143,5 +143,8 @@
 	/* Make sure we have no memleaks */
 	talloc_report_depth_cb(NULL, 0, 10, &talloc_ctx_walk_cb, NULL);
 
+	/* Make both Valgrind and LeakSanitizer happy */
+	talloc_disable_null_tracking();
+
 	return 0;
 }
diff --git a/tests/io/pq_file_test.c b/tests/io/pq_file_test.c
index 6691db3..664acf7 100644
--- a/tests/io/pq_file_test.c
+++ b/tests/io/pq_file_test.c
@@ -145,5 +145,8 @@
 	/* Print talloc memory hierarchy */
 	talloc_report_depth_cb(NULL, 0, 10, &talloc_ctx_walk_cb, NULL);
 
+	/* Make both Valgrind and LeakSanitizer happy */
+	talloc_disable_null_tracking();
+
 	return 0;
 }
diff --git a/tests/io/pq_rtp_test.c b/tests/io/pq_rtp_test.c
index f4070b7..2c1bd41 100644
--- a/tests/io/pq_rtp_test.c
+++ b/tests/io/pq_rtp_test.c
@@ -345,5 +345,8 @@
 	/* Memory leak detection test */
 	talloc_report_depth_cb(NULL, 0, 10, &talloc_ctx_walk_cb, NULL);
 
+	/* Make both Valgrind and LeakSanitizer happy */
+	talloc_disable_null_tracking();
+
 	return 0;
 }
diff --git a/tests/procqueue/pq_test.c b/tests/procqueue/pq_test.c
index 198108d..84b3992 100644
--- a/tests/procqueue/pq_test.c
+++ b/tests/procqueue/pq_test.c
@@ -369,5 +369,8 @@
 	printf("Processing queue deallocation test:\n");
 	talloc_report_depth_cb(NULL, 0, 10, &talloc_ctx_walk_cb, NULL);
 
+	/* Make both Valgrind and LeakSanitizer happy */
+	talloc_disable_null_tracking();
+
 	return 0;
 }

-- 
To view, visit https://gerrit.osmocom.org/9793
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: gapk
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia660d2fdac720f685c0186720d0a476d7e9468be
Gerrit-Change-Number: 9793
Gerrit-PatchSet: 1
Gerrit-Owner: Vadim Yanitskiy <axilirator at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20180630/a651ffd0/attachment.htm>


More information about the gerrit-log mailing list