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

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">gscon timeout: use proper cause code for lchan release<br><br>Since we verify the cause code of the RSL channel release in ttcn3-bsc-tests,<br>BSC_Tests.TC_chan_act_ack_est_ind_noreply shows an invalid release cause code<br>of 127. Fix that.<br><br>On gscon timeout, don't send the unrelated RSL code RSL_ERR_INTERWORKING, but a<br>proper RR cause code. GSM48_RR_CAUSE_ABNORMAL_TIMER is defined as the proper<br>cause for an expired timer (3GPP TS 44.018 10.5.2.31).<br><br>Should fix this error in BSC_Tests.TC_chan_act_ack_est_ind_noreply:<br><br>  BSC_Tests.ttcn:1590 Dynamic test case error: Assigning invalid numeric value 127 to a variable of enumerated type @GSM_RR_Types.RR_Cause.<br><br>Change-Id: Ic2e4b692e78f7b134fe57d1077a08adb48398e06<br>---<br>M src/osmo-bsc/bsc_subscr_conn_fsm.c<br>1 file changed, 1 insertion(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><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 e3976eb..6a35c75 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>@@ -866,7 +866,7 @@</span><br><span> </span><br><span>       switch (fi->T) {</span><br><span>  case 993210:</span><br><span style="color: hsl(0, 100%, 40%);">-            gscon_release_lchan(conn, conn->lchan, true, true, RSL_ERR_INTERWORKING);</span><br><span style="color: hsl(120, 100%, 40%);">+          gscon_release_lchan(conn, conn->lchan, true, true, GSM48_RR_CAUSE_ABNORMAL_TIMER);</span><br><span> </span><br><span>            /* MSC has not responded/confirmed connection with CC, this</span><br><span>           * could indicate a bad SCCP connection. We now inform the the</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/19627">change 19627</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/+/19627"/><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: Ic2e4b692e78f7b134fe57d1077a08adb48398e06 </div>
<div style="display:none"> Gerrit-Change-Number: 19627 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>