pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39207?usp=email ) Change subject: mgw: mgcp_protocol: assert freeing last conn allows creating new conn ...................................................................... mgw: mgcp_protocol: assert freeing last conn allows creating new conn Change-Id: Ib8c9089a7f04bc67e5bdfb1148cfc603f0e17bb7 --- M src/libosmo-mgcp/mgcp_protocol.c 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: pespin: Looks good to me, approved laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified daniel: Looks good to me, but someone else must approve diff --git a/src/libosmo-mgcp/mgcp_protocol.c b/src/libosmo-mgcp/mgcp_protocol.c index a425c99..7df2544 100644 --- a/src/libosmo-mgcp/mgcp_protocol.c +++ b/src/libosmo-mgcp/mgcp_protocol.c @@ -976,6 +976,7 @@ * room by blindly tossing the oldest of the two two * connections */ mgcp_endp_free_conn_oldest(endp); + OSMO_ASSERT(!mgcp_endp_is_full(endp)); } else { /* There is no more room for a connection, leave * everything as it is and return with an error */ -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39207?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: Ib8c9089a7f04bc67e5bdfb1148cfc603f0e17bb7 Gerrit-Change-Number: 39207 Gerrit-PatchSet: 3 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>