<p><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/15817">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/15817/4/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/+/15817/4/src/xua_asp_fsm.c@220">Patch Set #4, Line 220:</a> <code style="font-family:monospace,monospace">as = osmo_ss7_as_find_by_rctx(asp->inst, rctxs[0]);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I guess the proper thing to do here is to check if all of them have the same traffic mode, not just blindly use the traffic mode set by the first routing context/AS and assume all other contexts/AS have the same.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The full-blown spec-complete implementation would actually perform different ASP-ACTIVATE procedures for different (groups of) routing contexts, where all contexts within one group have the same traffic mode.  But this is highly theoretical and nothing we want to do here.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So either only support one routing context at all (and use that trfafic mode), or if you want to do multiple routing contexts in the ROUTE_CTX IE, then please check that they all agree in terms of the traffic mode.  This way we don't support all features, but at least behave only correctly.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/15817/4/src/xua_asp_fsm.c@226">Patch Set #4, Line 226:</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>                       for (i = 0; i < rctx_cnt; i++)<br>                             rctxs[i] = htonl(rctxs[i]);<br>                   xua_msg_add_data(xua, SUA_IEI_ROUTE_CTX, rctx_cnt * sizeof(uint32_t), (uint8_t *) &rctxs[0]);<br>             }<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">AFAIR I intentionally didn't add the routing contexts IE, as the ASP-ACTIVATE is automatically for all routing contexts if the optional IE is missing.  Yes, we might add all of them, but I'm not sure what we gain by it - or whether we should mix it with the traffic-mode in one patch.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/15817">change 15817</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/+/15817"/><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: Icce0e672f6180ebc92ca34f538d41161d02fecb9 </div>
<div style="display:none"> Gerrit-Change-Number: 15817 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </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:33:00 +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>