<p><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/15862">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/15862/1/src/xua_asp_fsm.c">File src/xua_asp_fsm.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/15862/1/src/xua_asp_fsm.c@241">Patch Set #1, Line 241:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">                if (rctx_cnt) {<br>                       as = osmo_ss7_as_find_by_rctx(asp->inst, rctxs[0]);<br>                        tmode = osmo_ss7_tmode_to_xua(as->cfg.mode);<br>                       if (as && tmode != -1)<br>                                xua_msg_add_u32(xua, SUA_IEI_TRAF_MODE_TYP, tmode);<br>           }<br>             /* Optional: Routing Context */<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think the condition whether or not to add these here is whether or not they were present in the ASPAC.  This may not be the same as us having routing contexts configured in the AS object.  I don't have a clear plan what to do, I'm just pointing out what I see as a possible inconsistency.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/15862">change 15862</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/libosmo-sccp/+/15862"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-sccp </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ic99131489efaef87f882312a042f5feb6a60bb86 </div>
<div style="display:none"> Gerrit-Change-Number: 15862 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 23 Oct 2019 21:36:18 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>