When running "make test" with the Address Sanitizer (by adding -fsanitize=address to compiler flags), a lot of memory leaks are reported from checkpolicy. Anyway some leaks come from the tests and it seems cleaner to start fixing these ones.
Signed-off-by: Nicolas Iooss <[email protected]> --- libsepol/tests/test-deps.c | 2 ++ libsepol/tests/test-expander.c | 17 +++++++++++++++++ 2 files changed, 19 insertions(+) diff --git a/libsepol/tests/test-deps.c b/libsepol/tests/test-deps.c index e7d2bebe0d93..ed23e0992ced 100644 --- a/libsepol/tests/test-deps.c +++ b/libsepol/tests/test-deps.c @@ -166,6 +166,7 @@ static void do_deps_modreq_global(int req_met, int b, char *policy, char *decl_t ret = link_modules(h, base, mods, 1, 0); CU_ASSERT_FATAL(ret == link_ret); policydb_destroy(&mod); + sepol_handle_destroy(h); if (!req_met) return; @@ -246,6 +247,7 @@ static void do_deps_modreq_opt(int req_met, int ret_val, int b, char *policy, ch ret = link_modules(h, base, mods, 1, 0); CU_ASSERT_FATAL(ret == ret_val); policydb_destroy(&mod); + sepol_handle_destroy(h); if (ret_val < 0) return; diff --git a/libsepol/tests/test-expander.c b/libsepol/tests/test-expander.c index ded1d9de2003..bc946c778b1d 100644 --- a/libsepol/tests/test-expander.c +++ b/libsepol/tests/test-expander.c @@ -122,6 +122,9 @@ int expander_policy_init(policydb_t * mybase, int num_modules, policydb_t ** mym return -1; } + for (i = 0; i < num_modules + 1; i++) { + free(filename[i]); + } return 0; } @@ -173,6 +176,20 @@ int expander_test_cleanup(void) { policydb_destroy(&basemod); policydb_destroy(&base_expanded); + policydb_destroy(&basemod2); + policydb_destroy(&base_expanded2); + policydb_destroy(&mod2); + policydb_destroy(&base_only_mod); + policydb_destroy(&base_only_expanded); + policydb_destroy(&role_basemod); + policydb_destroy(&role_expanded); + policydb_destroy(&role_mod); + policydb_destroy(&user_basemod); + policydb_destroy(&user_expanded); + policydb_destroy(&user_mod); + policydb_destroy(&alias_basemod); + policydb_destroy(&alias_expanded); + policydb_destroy(&alias_mod); free(typemap); return 0; -- 2.11.0 _______________________________________________ Selinux mailing list [email protected] To unsubscribe, send email to [email protected]. To get help, send an email containing "help" to [email protected].
