<p>Pau Espin Pedrol has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/14067">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">osmux: Use DUMMY ft msg as per Osmux spec<br><br>That MGCP_DUMMY_LOAD is an old hack prior to Osmux spec update, and it's<br>not nice since it cannot be 100% distinguished from a usual AMR ft<br>frame.<br><br>Let's use the correct DUMMY ft type and build it according spec. Allow<br>handling differently the old format for a while until we are sure no old<br>implementations (like bsc-nat) exist sending that kind of message.<br><br>Change-Id: Ib17d20b87b28aade49ba60519b56a96e694819af<br>---<br>M src/libosmo-mgcp/mgcp_osmux.c<br>1 file changed, 11 insertions(+), 5 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/67/14067/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/libosmo-mgcp/mgcp_osmux.c b/src/libosmo-mgcp/mgcp_osmux.c</span><br><span>index 7a817f8..3b788b6 100644</span><br><span>--- a/src/libosmo-mgcp/mgcp_osmux.c</span><br><span>+++ b/src/libosmo-mgcp/mgcp_osmux.c</span><br><span>@@ -20,6 +20,7 @@</span><br><span> </span><br><span> #include <osmocom/netif/osmux.h></span><br><span> #include <osmocom/netif/rtp.h></span><br><span style="color: hsl(120, 100%, 40%);">+#include <osmocom/netif/amr.h></span><br><span> </span><br><span> #include <osmocom/mgcp/mgcp.h></span><br><span> #include <osmocom/mgcp/mgcp_internal.h></span><br><span>@@ -554,7 +555,8 @@</span><br><span>  *  \returns bytes sent, -1 on error */</span><br><span> int osmux_send_dummy(struct mgcp_endpoint *endp, struct mgcp_conn_rtp *conn)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-    char buf[1 + sizeof(uint8_t)];</span><br><span style="color: hsl(120, 100%, 40%);">+        struct osmux_hdr *osmuxh;</span><br><span style="color: hsl(120, 100%, 40%);">+     int buf_len;</span><br><span>         struct in_addr addr_unset = {};</span><br><span> </span><br><span>  /*! The dummy packet will not be sent via the actual OSMUX connection,</span><br><span>@@ -566,9 +568,6 @@</span><br><span>          *  endpoint may have already punched the hole in the firewall. This</span><br><span>          *  approach is simple though. */</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   buf[0] = MGCP_DUMMY_LOAD;</span><br><span style="color: hsl(0, 100%, 40%);">-       memcpy(&buf[1], &conn->osmux.cid, sizeof(conn->osmux.cid));</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span>    /* Wait until we have the connection information from MDCX */</span><br><span>        if (memcmp(&conn->end.addr, &addr_unset, sizeof(addr_unset)) == 0)</span><br><span>                return 0;</span><br><span>@@ -576,12 +575,19 @@</span><br><span>    if (endp_osmux_state_check(endp, conn, true) < 0)</span><br><span>                 return 0;</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+ buf_len = sizeof(struct osmux_hdr) + osmo_amr_bytes(AMR_FT_0);</span><br><span style="color: hsl(120, 100%, 40%);">+        osmuxh = (struct osmux_hdr *) alloca(buf_len);</span><br><span style="color: hsl(120, 100%, 40%);">+        memset(osmuxh, 0, buf_len);</span><br><span style="color: hsl(120, 100%, 40%);">+   osmuxh->ft = OSMUX_FT_DUMMY;</span><br><span style="color: hsl(120, 100%, 40%);">+       osmuxh->amr_ft = AMR_FT_0;</span><br><span style="color: hsl(120, 100%, 40%);">+ osmuxh->circuit_id = conn->osmux.cid;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>        LOGPCONN(conn->conn, DLMGCP, LOGL_DEBUG,</span><br><span>           "sending OSMUX dummy load to %s:%u CID %u\n",</span><br><span>              inet_ntoa(conn->end.addr), ntohs(conn->end.rtp_port), conn->osmux.cid);</span><br><span> </span><br><span>        return mgcp_udp_send(osmux_fd.fd, &conn->end.addr,</span><br><span style="color: hsl(0, 100%, 40%);">-                            conn->end.rtp_port, buf, sizeof(buf));</span><br><span style="color: hsl(120, 100%, 40%);">+                             conn->end.rtp_port, (char*)osmuxh, buf_len);</span><br><span> }</span><br><span> </span><br><span> /* bsc-nat allocates/releases the Osmux circuit ID. +7 to round up to 8 bit boundary. */</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/14067">change 14067</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/14067"/><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: Ib17d20b87b28aade49ba60519b56a96e694819af </div>
<div style="display:none"> Gerrit-Change-Number: 14067 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>