>From 54dc1a7132aa71438e0e31ed78a23d0925621a89 Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Wed, 20 Aug 2014 19:21:40 +0200 Subject: [PATCH 2/7] responder-get-domains-tests: fix checking of leaks global_talloc_context should not be NULL in tests. It should be initialised with function leak_check_setup otherwise memory leak will not be detected. check_leaks_pop should not be directly called for global_talloc_context. It is a purpose of function leak_check_teardown --- src/tests/cmocka/test_responder_common.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/tests/cmocka/test_responder_common.c b/src/tests/cmocka/test_responder_common.c index 24bb54ff5120b281d43ddc7d5d4443c87a22af1c..77437602b700d6224ed0ca844fb635c59897db30 100644 --- a/src/tests/cmocka/test_responder_common.c +++ b/src/tests/cmocka/test_responder_common.c @@ -88,8 +88,7 @@ void parse_inp_test_setup(void **state) { struct parse_inp_test_ctx *parse_inp_ctx; - check_leaks_push(global_talloc_context); - + assert_true(leak_check_setup()); parse_inp_ctx = talloc_zero(global_talloc_context, struct parse_inp_test_ctx); assert_non_null(parse_inp_ctx); @@ -119,7 +118,7 @@ void parse_inp_test_teardown(void **state) assert_true(check_leaks_pop(parse_inp_ctx) == true); talloc_free(parse_inp_ctx); - assert_true(check_leaks_pop(global_talloc_context) == true); + assert_true(leak_check_teardown()); } int __real_sss_parse_name_for_domains(TALLOC_CTX *memctx, -- 2.1.0