pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/29535 ) Change subject: osmux: Keep decoding osmux pkt if a batch contains an unknown CID ...................................................................... osmux: Keep decoding osmux pkt if a batch contains an unknown CID Receiving an unknown CID (for which we don't have an active conn) is totally expected at the end of the call, were some messages from the peer may be send for a while, specially if latency is high. If this occurs, we simply drop that batch and keep decoding the osmux message in order to keep delivering content to other circuits. Related: SYS#5987 Change-Id: I315949853bdcc07bef15d2e8579029cc72c427cf --- M src/libosmo-mgcp/mgcp_osmux.c 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve fixeria: Looks good to me, approved diff --git a/src/libosmo-mgcp/mgcp_osmux.c b/src/libosmo-mgcp/mgcp_osmux.c index c3c8427..a4ce024 100644 --- a/src/libosmo-mgcp/mgcp_osmux.c +++ b/src/libosmo-mgcp/mgcp_osmux.c @@ -443,7 +443,8 @@ LOGP(DOSMUX, LOGL_DEBUG, "Cannot find a src conn for circuit_id=%d\n", osmuxh->circuit_id); - goto out; + rem = msg->len; + continue; } endp = conn_src->conn->endp; mgcp_conn_watchdog_kick(conn_src->conn); -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/29535 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I315949853bdcc07bef15d2e8579029cc72c427cf Gerrit-Change-Number: 29535 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged