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