diff options
| author | Lukas Slebodnik <lslebodn@redhat.com> | 2014-08-20 19:21:40 +0200 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-08-22 10:55:02 +0200 |
| commit | 5b8ed5702b38c2bd39e9c9a44d94ad7dac3040a1 (patch) | |
| tree | 4b6687b8f1254bfc104ec79598357451400402f0 | |
| parent | 0789077faa81113e3e6ef46f71bde878d8c58023 (diff) | |
| download | sssd-5b8ed5702b38c2bd39e9c9a44d94ad7dac3040a1.tar.gz sssd-5b8ed5702b38c2bd39e9c9a44d94ad7dac3040a1.tar.xz sssd-5b8ed5702b38c2bd39e9c9a44d94ad7dac3040a1.zip | |
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
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
(cherry picked from commit bf65fbdd8c3fecd38a66363c3517e7a2679b8186)
| -rw-r--r-- | src/tests/cmocka/test_responder_common.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/tests/cmocka/test_responder_common.c b/src/tests/cmocka/test_responder_common.c index 58aa32b5d..fb1217700 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, |