On 12/26/2016 04:10 PM, Nicolas Iooss wrote:
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]>
Applied.
Thanks,
---
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;
--
James Carter <[email protected]>
National Security Agency
_______________________________________________
Selinux mailing list
[email protected]
To unsubscribe, send email to [email protected].
To get help, send an email containing "help" to [email protected].
