msuraev submitted this change.

View Change



3 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified
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(-)

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 change 31868. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I5739bceb90d36164fd4cbf21242bbe26bd1e7075
Gerrit-Change-Number: 31868
Gerrit-PatchSet: 4
Gerrit-Owner: msuraev <msuraev@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: msuraev <msuraev@sysmocom.de>
Gerrit-CC: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged