pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/37875?usp=email ) Change subject: tests/mgcp/mgcp_test: Add some extra asserts in code ...................................................................... tests/mgcp/mgcp_test: Add some extra asserts in code This makes it easier to catch possible failures while running the test, plus makes gcc on OpenSUSE_Factory happy and not warn. Related: OS#5044 Change-Id: I01c3c84e58a11055f0e0a6955016dc2489c73379 --- M tests/mgcp/mgcp_test.c 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: osmith: Looks good to me, but someone else must approve Jenkins Builder: Verified fixeria: Looks good to me, approved diff --git a/tests/mgcp/mgcp_test.c b/tests/mgcp/mgcp_test.c index 21be037..de359d5 100644 --- a/tests/mgcp/mgcp_test.c +++ b/tests/mgcp/mgcp_test.c @@ -1208,7 +1208,9 @@ _conn = mgcp_conn_alloc(NULL, &endp, MGCP_CONN_TYPE_RTP, "test-connection"); + OSMO_ASSERT(_conn); conn = mgcp_conn_get_rtp(&endp, _conn->id); + OSMO_ASSERT(conn); state = &conn->state; packets_rx = rate_ctr_group_get_ctr(conn->ctrg, RTP_PACKETS_RX_CTR); -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/37875?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I01c3c84e58a11055f0e0a6955016dc2489c73379 Gerrit-Change-Number: 37875 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>