<p>Pau Espin Pedrol has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/14028">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mgcp-cli: Allow submitting X-Osmux on MDCX request<br><br>Change-Id: I41243f3ed212ace6087d5b0341e3a52f4069e37d<br>---<br>M src/libosmo-mgcp-client/mgcp_client_fsm.c<br>M tests/mgcp_client/mgcp_client_test.c<br>M tests/mgcp_client/mgcp_client_test.ok<br>3 files changed, 24 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/28/14028/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/libosmo-mgcp-client/mgcp_client_fsm.c b/src/libosmo-mgcp-client/mgcp_client_fsm.c</span><br><span>index efbb365..0bbe24b 100644</span><br><span>--- a/src/libosmo-mgcp-client/mgcp_client_fsm.c</span><br><span>+++ b/src/libosmo-mgcp-client/mgcp_client_fsm.c</span><br><span>@@ -173,6 +173,11 @@</span><br><span> </span><br><span>  set_conn_mode(&mgcp_msg, &mgcp_ctx->conn_peer_local);</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+  if (mgcp_ctx->conn_peer_local.x_osmo_osmux_use) {</span><br><span style="color: hsl(120, 100%, 40%);">+          mgcp_msg.x_osmo_osmux_cid = mgcp_ctx->conn_peer_local.x_osmo_osmux_cid;</span><br><span style="color: hsl(120, 100%, 40%);">+            mgcp_msg.presence |= MGCP_MSG_PRESENCE_X_OSMO_OSMUX_CID;</span><br><span style="color: hsl(120, 100%, 40%);">+      }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>  /* Note: We take the endpoint and the call_id from the remote</span><br><span>         * connection info, because we can be confident that the</span><br><span>      * information there is valid. For the local info, we explicitly</span><br><span>diff --git a/tests/mgcp_client/mgcp_client_test.c b/tests/mgcp_client/mgcp_client_test.c</span><br><span>index 432a09c..9c4392f 100644</span><br><span>--- a/tests/mgcp_client/mgcp_client_test.c</span><br><span>+++ b/tests/mgcp_client/mgcp_client_test.c</span><br><span>@@ -276,6 +276,17 @@</span><br><span>         msg = mgcp_msg_gen(mgcp, &mgcp_msg);</span><br><span>     printf("%s\n", (char *)msg->data);</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+   printf("Generate X-Osmo-Osmux message (MDCX):\n");</span><br><span style="color: hsl(120, 100%, 40%);">+  msg = mgcp_msg_gen(mgcp, &mgcp_msg);</span><br><span style="color: hsl(120, 100%, 40%);">+      mgcp_msg.verb = MGCP_VERB_MDCX;</span><br><span style="color: hsl(120, 100%, 40%);">+       mgcp_msg.x_osmo_osmux_cid = 2;</span><br><span style="color: hsl(120, 100%, 40%);">+        mgcp_msg.presence =</span><br><span style="color: hsl(120, 100%, 40%);">+       (MGCP_MSG_PRESENCE_ENDPOINT | MGCP_MSG_PRESENCE_CALL_ID |</span><br><span style="color: hsl(120, 100%, 40%);">+      MGCP_MSG_PRESENCE_CONN_ID | MGCP_MSG_PRESENCE_CONN_MODE</span><br><span style="color: hsl(120, 100%, 40%);">+       | MGCP_MSG_PRESENCE_X_OSMO_OSMUX_CID);</span><br><span style="color: hsl(120, 100%, 40%);">+   msg = mgcp_msg_gen(mgcp, &mgcp_msg);</span><br><span style="color: hsl(120, 100%, 40%);">+      printf("%s\n", (char *)msg->data);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>    printf("Overfolow test:\n");</span><br><span>       mgcp_msg.verb = MGCP_VERB_MDCX;</span><br><span>      mgcp_msg.presence =</span><br><span>diff --git a/tests/mgcp_client/mgcp_client_test.ok b/tests/mgcp_client/mgcp_client_test.ok</span><br><span>index 8fbe3ae..8b3e15b 100644</span><br><span>--- a/tests/mgcp_client/mgcp_client_test.ok</span><br><span>+++ b/tests/mgcp_client/mgcp_client_test.ok</span><br><span>@@ -100,6 +100,14 @@</span><br><span> M: sendrecv </span><br><span> X-Osmux: 2 </span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+Generate X-Osmo-Osmux message (MDCX):</span><br><span style="color: hsl(120, 100%, 40%);">+MDCX 17 23@mgw MGCP 1.0 </span><br><span style="color: hsl(120, 100%, 40%);">+C: 2f </span><br><span style="color: hsl(120, 100%, 40%);">+I: 11 </span><br><span style="color: hsl(120, 100%, 40%);">+L: p:20, a:GSM, nt:IN </span><br><span style="color: hsl(120, 100%, 40%);">+M: sendrecv </span><br><span style="color: hsl(120, 100%, 40%);">+X-Osmux: 2 </span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> Overfolow test:</span><br><span> </span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/14028">change 14028</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/14028"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I41243f3ed212ace6087d5b0341e3a52f4069e37d </div>
<div style="display:none"> Gerrit-Change-Number: 14028 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>