pespin submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve fixeria: Looks good to me, approved
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(-)

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 change 29535. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: I315949853bdcc07bef15d2e8579029cc72c427cf
Gerrit-Change-Number: 29535
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged