msuraev has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/31868 ) ( 3 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: logging: print talloc report on exit from vty test ...................................................................... logging: print talloc report on exit from vty test There's not much point in deallocating memory in a test fixture which is about to terminate anyway. Having talloc report though is handy to make sure we're not leaking smth. Change-Id: I5739bceb90d36164fd4cbf21242bbe26bd1e7075 --- M tests/logging/logging_vty_test.c 1 file changed, 16 insertions(+), 3 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/tests/logging/logging_vty_test.c b/tests/logging/logging_vty_test.c index 15f7fc2..c1a2853 100644 --- a/tests/logging/logging_vty_test.c +++ b/tests/logging/logging_vty_test.c @@ -285,10 +285,10 @@ osmo_select_main(0); } - log_fini(); + talloc_report(tall_vty_ctx, stderr); + talloc_report_full(root_ctx, stderr); - talloc_free(root_ctx); - talloc_free(tall_vty_ctx); + log_fini(); return 0; } -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/31868 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I5739bceb90d36164fd4cbf21242bbe26bd1e7075 Gerrit-Change-Number: 31868 Gerrit-PatchSet: 4 Gerrit-Owner: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged