This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
Neels Hofmeyr gerrit-no-reply at lists.osmocom.orgNeels Hofmeyr has submitted this change and it was merged. Change subject: mgcp_test: sanitize: free msgb_ctx ...................................................................... mgcp_test: sanitize: free msgb_ctx Ensure that all msgb were cleaned up, then free the overall msgb_ctx, in order to not leave any memory leaks the sanitizer build complains about. Change-Id: I53373023a6c3f490d6d6cb1c283db5dfb915882c --- M tests/mgcp/mgcp_test.c 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified Holger Freyther: Looks good to me, approved diff --git a/tests/mgcp/mgcp_test.c b/tests/mgcp/mgcp_test.c index da71b46..99491e6 100644 --- a/tests/mgcp/mgcp_test.c +++ b/tests/mgcp/mgcp_test.c @@ -1331,7 +1331,7 @@ int main(int argc, char **argv) { - msgb_talloc_ctx_init(NULL, 0); + void *msgb_ctx = msgb_talloc_ctx_init(NULL, 0); osmo_init_logging(&log_info); test_strline(); @@ -1350,6 +1350,9 @@ test_no_name(); test_osmux_cid(); + OSMO_ASSERT(talloc_total_size(msgb_ctx) == 0); + OSMO_ASSERT(talloc_total_blocks(msgb_ctx) == 1); + talloc_free(msgb_ctx); printf("Done\n"); return EXIT_SUCCESS; } -- To view, visit https://gerrit.osmocom.org/4920 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53373023a6c3f490d6d6cb1c283db5dfb915882c Gerrit-PatchSet: 2 Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de> Gerrit-Reviewer: Holger Freyther <holger at freyther.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>