Harald Welte has submitted this change and it was merged.
Change subject: protocol: allow wildcarded DLCX
......................................................................
protocol: allow wildcarded DLCX
In many cases it is simpler to instruct the mgcp-gw to drop all
connections at once instead of removing each connection
individually.
drop all connections and release the endpoint in when no connection
id is supplied with the DLCX command.
Change-Id: Ib5fcc72775bf72b489ff79ade36fb345d8d20736
---
M src/libosmo-mgcp/mgcp_protocol.c
1 file changed, 19 insertions(+), 2 deletions(-)
Approvals:
Neels Hofmeyr: Looks good to me, but someone else must approve
Harald Welte: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/libosmo-mgcp/mgcp_protocol.c b/src/libosmo-mgcp/mgcp_protocol.c
index 6c611f7..9c92c65 100644
--- a/src/libosmo-mgcp/mgcp_protocol.c
+++ b/src/libosmo-mgcp/mgcp_protocol.c
@@ -910,14 +910,31 @@
}
}
- /* find the connection */
+ /* When no connection id is supplied, we will interpret this as a
+ * wildcarded DLCX and drop all connections at once. (See also
+ * RFC3435 Section F.7) */
+ if (!ci) {
+ LOGP(DLMGCP, LOGL_NOTICE,
+ "DLCX: endpoint:%x missing ci (connectionIdentifier), will
remove all connections at once\n",
+ ENDPOINT_NUMBER(endp));
+
+ mgcp_release_endp(endp);
+
+ /* Note: In this case we do not return any statistics,
+ * as we assume that the client is not interested in
+ * this case. */
+ return create_ok_response(endp, 200, "DLCX", p->trans);
+ }
+
+ /* Parse the connection id */
if (mgcp_parse_ci(&conn_id, ci)) {
LOGP(DLMGCP, LOGL_ERROR,
- "DLCX: endpoint:%x insufficient parameters, missing ci
(connectionIdentifier)\n",
+ "DLCX: endpoint:%x insufficient parameters, invalid ci
(connectionIdentifier)\n",
ENDPOINT_NUMBER(endp));
return create_err_response(endp, 400, "DLCX", p->trans);
}
+ /* Find the connection */
conn = mgcp_conn_get_rtp(endp, conn_id);
if (!conn)
goto error3;
--
To view, visit https://gerrit.osmocom.org/4227
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib5fcc72775bf72b489ff79ade36fb345d8d20736
Gerrit-PatchSet: 11
Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Owner: dexter <pma...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <nhofm...@sysmocom.de>
