<p>Harald Welte <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/9319">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bsc: Use correct MGCP endpoint name for IPA/SCCPlite<br><br>In IPA/SCCPlite, we have to use the CIC to construct the MGCP<br>endpoint name instead of the usual dynamic endpoint allocation.<br><br>Change-Id: I03e2cdbc8e40169e52df3720c40b66734e880525<br>---<br>M src/libbsc/bsc_subscr_conn_fsm.c<br>1 file changed, 20 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/libbsc/bsc_subscr_conn_fsm.c b/src/libbsc/bsc_subscr_conn_fsm.c</span><br><span>index 5feca69..63d99a9 100644</span><br><span>--- a/src/libbsc/bsc_subscr_conn_fsm.c</span><br><span>+++ b/src/libbsc/bsc_subscr_conn_fsm.c</span><br><span>@@ -383,6 +383,25 @@</span><br><span>        }</span><br><span> }</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+static const char *get_mgw_ep_name(struct gsm_subscriber_connection *conn)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+     static char ep_name[256];</span><br><span style="color: hsl(120, 100%, 40%);">+     struct bsc_msc_data *msc = conn->sccp.msc;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+       switch (conn->sccp.msc->a.asp_proto) {</span><br><span style="color: hsl(120, 100%, 40%);">+  case OSMO_SS7_ASP_PROT_IPA:</span><br><span style="color: hsl(120, 100%, 40%);">+           /* derive endpoint name from CIC on A interface side */</span><br><span style="color: hsl(120, 100%, 40%);">+               snprintf(ep_name, sizeof(ep_name), "%x@mgw",</span><br><span style="color: hsl(120, 100%, 40%);">+                         mgcp_port_to_cic(conn->user_plane.rtp_port, msc->rtp_base));</span><br><span style="color: hsl(120, 100%, 40%);">+           break;</span><br><span style="color: hsl(120, 100%, 40%);">+        default:</span><br><span style="color: hsl(120, 100%, 40%);">+              /* use dynamic RTPBRIDGE endpoint allocation in MGW */</span><br><span style="color: hsl(120, 100%, 40%);">+                osmo_strlcpy(ep_name, ENDPOINT_ID, sizeof(ep_name));</span><br><span style="color: hsl(120, 100%, 40%);">+          break;</span><br><span style="color: hsl(120, 100%, 40%);">+        }</span><br><span style="color: hsl(120, 100%, 40%);">+     return ep_name;</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> /* We're on an active subscriber connection, passing DTAP back and forth */</span><br><span> static void gscon_fsm_active(struct osmo_fsm_inst *fi, uint32_t event, void *data)</span><br><span> {</span><br><span>@@ -413,7 +432,7 @@</span><br><span>                      * mgcp-ass-mgcp state-chain (see FIXME above) */</span><br><span>                    memset(&conn_peer, 0, sizeof(conn_peer));</span><br><span>                        conn_peer.call_id = conn->sccp.conn_id;</span><br><span style="color: hsl(0, 100%, 40%);">-                      osmo_strlcpy(conn_peer.endpoint, ENDPOINT_ID, sizeof(conn_peer.endpoint));</span><br><span style="color: hsl(120, 100%, 40%);">+                    osmo_strlcpy(conn_peer.endpoint, get_mgw_ep_name(conn), sizeof(conn_peer.endpoint));</span><br><span> </span><br><span>                     /* (Pre)Change state and create the connection */</span><br><span>                    osmo_fsm_inst_state_chg(fi, ST_WAIT_CRCX_BTS, MGCP_MGW_TIMEOUT, MGCP_MGW_TIMEOUT_TIMER_NR);</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/9319">change 9319</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/9319"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I03e2cdbc8e40169e52df3720c40b66734e880525 </div>
<div style="display:none"> Gerrit-Change-Number: 9319 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>