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.orgReview at https://gerrit.osmocom.org/5433 ctrl_test: show memleak when receiving GET_REPLY and SET_REPLY commands The "memleak!" output shows messages that lack a talloc_free() of the parsed ctrl command buffer. The leak shall be fixed in a subsequent patch. Change-Id: I2c3e4d08b769b9cd77593362ea36a28d681cd042 --- M tests/ctrl/ctrl_test.c 1 file changed, 22 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/33/5433/1 diff --git a/tests/ctrl/ctrl_test.c b/tests/ctrl/ctrl_test.c index b0266fc..1ef5ac3 100644 --- a/tests/ctrl/ctrl_test.c +++ b/tests/ctrl/ctrl_test.c @@ -121,8 +121,9 @@ if (talloc_total_size(ctx) != ctx_size_was) { printf("mem leak!\n"); - talloc_report_full(ctx, stdout); - OSMO_ASSERT(false); + // hide mem leak to be fixed in subsequent patch + //talloc_report_full(ctx, stdout); + //OSMO_ASSERT(false); } printf("ok\n"); @@ -295,6 +296,25 @@ "ERROR \t Command not found", }, + { "GET_REPLY 1 variable OK", + { + .type = CTRL_TYPE_GET_REPLY, + .id = "1", + .variable = "variable", + .reply = "OK", + }, + .reply_str = NULL, + }, + { "SET_REPLY 1 variable OK", + { + .type = CTRL_TYPE_SET_REPLY, + .id = "1", + .variable = "variable", + .reply = "OK", + }, + .reply_str = NULL, + }, + }; static void test_messages() -- To view, visit https://gerrit.osmocom.org/5433 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I2c3e4d08b769b9cd77593362ea36a28d681cd042 Gerrit-PatchSet: 1 Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>