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

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Introduce msc_is_{sccplite,aoip}() helpers<br><br>Having the helper makes it easier to read/find for transport type checks. It<br>will be ifurther re-used in forthcoming commits.<br><br>Change-Id: Ic0ee4c472e29ec3092049e5e23b744395613616d<br>---<br>M include/osmocom/bsc/bsc_msc_data.h<br>M src/osmo-bsc/bsc_subscr_conn_fsm.c<br>M src/osmo-bsc/osmo_bsc_sigtran.c<br>3 files changed, 23 insertions(+), 17 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/include/osmocom/bsc/bsc_msc_data.h b/include/osmocom/bsc/bsc_msc_data.h</span><br><span>index 4720845..1f9872a 100644</span><br><span>--- a/include/osmocom/bsc/bsc_msc_data.h</span><br><span>+++ b/include/osmocom/bsc/bsc_msc_data.h</span><br><span>@@ -191,5 +191,25 @@</span><br><span>    return (port - base) / 2;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+static inline bool msc_is_aoip(const struct bsc_msc_data *msc)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+ switch (msc->a.asp_proto) {</span><br><span style="color: hsl(120, 100%, 40%);">+        case OSMO_SS7_ASP_PROT_SUA:</span><br><span style="color: hsl(120, 100%, 40%);">+   case OSMO_SS7_ASP_PROT_M3UA:</span><br><span style="color: hsl(120, 100%, 40%);">+          return true;</span><br><span style="color: hsl(120, 100%, 40%);">+  default:</span><br><span style="color: hsl(120, 100%, 40%);">+              return false;</span><br><span style="color: hsl(120, 100%, 40%);">+ }</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+static inline bool msc_is_sccplite(const struct bsc_msc_data *msc)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+ switch (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%);">+           return true;</span><br><span style="color: hsl(120, 100%, 40%);">+  default:</span><br><span style="color: hsl(120, 100%, 40%);">+              return false;</span><br><span style="color: hsl(120, 100%, 40%);">+ }</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span> </span><br><span> #endif</span><br><span>diff --git a/src/osmo-bsc/bsc_subscr_conn_fsm.c b/src/osmo-bsc/bsc_subscr_conn_fsm.c</span><br><span>index 87ad567..7b9f912 100644</span><br><span>--- a/src/osmo-bsc/bsc_subscr_conn_fsm.c</span><br><span>+++ b/src/osmo-bsc/bsc_subscr_conn_fsm.c</span><br><span>@@ -1058,14 +1058,7 @@</span><br><span>     if (!conn || !conn->sccp.msc)</span><br><span>             return false;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-       switch (conn->sccp.msc->a.asp_proto) {</span><br><span style="color: hsl(0, 100%, 40%);">-    case OSMO_SS7_ASP_PROT_SUA:</span><br><span style="color: hsl(0, 100%, 40%);">-     case OSMO_SS7_ASP_PROT_M3UA:</span><br><span style="color: hsl(0, 100%, 40%);">-            return true;</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-    default:</span><br><span style="color: hsl(0, 100%, 40%);">-                return false;</span><br><span style="color: hsl(0, 100%, 40%);">-   }</span><br><span style="color: hsl(120, 100%, 40%);">+     return msc_is_aoip(conn->sccp.msc);</span><br><span> }</span><br><span> </span><br><span> bool gscon_is_sccplite(struct gsm_subscriber_connection *conn)</span><br><span>@@ -1073,11 +1066,5 @@</span><br><span>   if (!conn || !conn->sccp.msc)</span><br><span>             return false;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-       switch (conn->sccp.msc->a.asp_proto) {</span><br><span style="color: hsl(0, 100%, 40%);">-    case OSMO_SS7_ASP_PROT_IPA:</span><br><span style="color: hsl(0, 100%, 40%);">-             return true;</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-    default:</span><br><span style="color: hsl(0, 100%, 40%);">-                return false;</span><br><span style="color: hsl(0, 100%, 40%);">-   }</span><br><span style="color: hsl(120, 100%, 40%);">+     return msc_is_sccplite(conn->sccp.msc);</span><br><span> }</span><br><span>diff --git a/src/osmo-bsc/osmo_bsc_sigtran.c b/src/osmo-bsc/osmo_bsc_sigtran.c</span><br><span>index 5d61c32..8303c4f 100644</span><br><span>--- a/src/osmo-bsc/osmo_bsc_sigtran.c</span><br><span>+++ b/src/osmo-bsc/osmo_bsc_sigtran.c</span><br><span>@@ -523,8 +523,7 @@</span><br><span>                * an X-Osmo-IGN flag telling osmo-mgw to ignore CallID mismatches for this endpoint.</span><br><span>                 * If an explicit VTY command has already indicated whether or not to send X-Osmo-IGN, do</span><br><span>             * not overwrite that setting. */</span><br><span style="color: hsl(0, 100%, 40%);">-               if (msc->a.asp_proto == OSMO_SS7_ASP_PROT_IPA</span><br><span style="color: hsl(0, 100%, 40%);">-                    && !msc->x_osmo_ign_configured)</span><br><span style="color: hsl(120, 100%, 40%);">+                if (msc_is_sccplite(msc) && !msc->x_osmo_ign_configured)</span><br><span>                  msc->x_osmo_ign |= MGCP_X_OSMO_IGN_CALLID;</span><br><span> </span><br><span>            /* If unset, use default local SCCP address */</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/14383">change 14383</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/c/osmo-bsc/+/14383"/><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-Change-Id: Ic0ee4c472e29ec3092049e5e23b744395613616d </div>
<div style="display:none"> Gerrit-Change-Number: 14383 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>