<p>osmith has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/14899">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">vlr_lu_fsm.c: don't send LU reject twice<br><br>Don't call tx_lu_rej() in the "vlr_lu_compl" FSM. It is always getting<br>called in the parent "lu" FSM and is therefore redundant:<br><br>vlr_lu_compl_fsm_failure()<br>-> _vlr_lu_compl_fsm_done(fi, VLR_FSM_RESULT_FAILURE, cause)<br>-> osmo_fsm_inst_state_chg(fi, LU_COMPL_VLR_S_DONE, 0, 0)<br>-> vlr_lu_compl_fsm_dispatch_result()<br>-> lu_fsm_wait_lu_compl()/lu_fsm_wait_lu_compl_standalone()<br>-> lu_fsm_failure()<br>-> lfp->vlr->ops.tx_lu_rej()<br><br>I have noticed the bug with the TTCN3 tests. This patch fixes<br>TC_lu_imsi_auth_tmsi_check_imei_{nack,err} after stricter checking<br>in [1] and also TC_iu_mo_crcx_ran_reject.<br><br>[1] I836f76242463789c4c003feec757714827f2a31b (osmo-ttcn3-hacks)<br>Change-Id: I127b27937613ea0ff29d67991c0414fca6d441d9<br>---<br>M src/libvlr/vlr_lu_fsm.c<br>M tests/msc_vlr/msc_vlr_test_gsm_authen.err<br>2 files changed, 0 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/99/14899/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/libvlr/vlr_lu_fsm.c b/src/libvlr/vlr_lu_fsm.c</span><br><span>index 9dff4aa..caa4b28 100644</span><br><span>--- a/src/libvlr/vlr_lu_fsm.c</span><br><span>+++ b/src/libvlr/vlr_lu_fsm.c</span><br><span>@@ -368,8 +368,6 @@</span><br><span> </span><br><span> static void vlr_lu_compl_fsm_failure(struct osmo_fsm_inst *fi, uint8_t cause)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-     struct lu_compl_vlr_priv *lcvp = lu_compl_vlr_fi_priv(fi);</span><br><span style="color: hsl(0, 100%, 40%);">-      lcvp->vsub->vlr->ops.tx_lu_rej(lcvp->msc_conn_ref, cause);</span><br><span>       _vlr_lu_compl_fsm_done(fi, VLR_FSM_RESULT_FAILURE, cause);</span><br><span> }</span><br><span> </span><br><span>diff --git a/tests/msc_vlr/msc_vlr_test_gsm_authen.err b/tests/msc_vlr/msc_vlr_test_gsm_authen.err</span><br><span>index 0a18302..c9be4ef 100644</span><br><span>--- a/tests/msc_vlr/msc_vlr_test_gsm_authen.err</span><br><span>+++ b/tests/msc_vlr/msc_vlr_test_gsm_authen.err</span><br><span>@@ -1907,7 +1907,6 @@</span><br><span> DREF VLR subscr IMSI-901700000004620:MSISDN-46071 + vlr_gsup_rx: now used by 2 (active-conn,vlr_gsup_rx)</span><br><span> DVLR vlr_lu_fsm(IMSI-901700000004620:MSISDN-46071:GERAN-A:LU){VLR_ULA_S_WAIT_LU_COMPL}: Received Event VLR_ULA_E_HLR_IMEI_NACK</span><br><span> DVLR lu_compl_vlr_fsm(IMSI-901700000004620:MSISDN-46071:GERAN-A:LU){LU_COMPL_VLR_S_WAIT_IMEI}: Received Event LU_COMPL_VLR_E_IMEI_CHECK_NACK</span><br><span style="color: hsl(0, 100%, 40%);">-- sending LU Reject for IMSI-901700000004620:MSISDN-46071:GERAN-A:LU, cause 6</span><br><span> DVLR lu_compl_vlr_fsm(IMSI-901700000004620:MSISDN-46071:GERAN-A:LU){LU_COMPL_VLR_S_WAIT_IMEI}: state_chg to LU_COMPL_VLR_S_DONE</span><br><span> DVLR vlr_lu_fsm(IMSI-901700000004620:MSISDN-46071:GERAN-A:LU){VLR_ULA_S_WAIT_LU_COMPL}: Received Event VLR_ULA_E_LU_COMPL_FAILURE</span><br><span> DVLR lu_compl_vlr_fsm(IMSI-901700000004620:MSISDN-46071:GERAN-A:LU){LU_COMPL_VLR_S_DONE}: Terminating (cause = OSMO_FSM_TERM_PARENT)</span><br><span>@@ -2174,7 +2173,6 @@</span><br><span> DVLR SUBSCR(IMSI-901700000004620:MSISDN-46071) Check_IMEI_VLR failed; gmm_cause: Invalid mandatory information</span><br><span> DVLR vlr_lu_fsm(IMSI-901700000004620:MSISDN-46071:GERAN-A:LU){VLR_ULA_S_WAIT_LU_COMPL}: Received Event VLR_ULA_E_HLR_IMEI_NACK</span><br><span> DVLR lu_compl_vlr_fsm(IMSI-901700000004620:MSISDN-46071:GERAN-A:LU){LU_COMPL_VLR_S_WAIT_IMEI}: Received Event LU_COMPL_VLR_E_IMEI_CHECK_NACK</span><br><span style="color: hsl(0, 100%, 40%);">-- sending LU Reject for IMSI-901700000004620:MSISDN-46071:GERAN-A:LU, cause 6</span><br><span> DVLR lu_compl_vlr_fsm(IMSI-901700000004620:MSISDN-46071:GERAN-A:LU){LU_COMPL_VLR_S_WAIT_IMEI}: state_chg to LU_COMPL_VLR_S_DONE</span><br><span> DVLR vlr_lu_fsm(IMSI-901700000004620:MSISDN-46071:GERAN-A:LU){VLR_ULA_S_WAIT_LU_COMPL}: Received Event VLR_ULA_E_LU_COMPL_FAILURE</span><br><span> DVLR lu_compl_vlr_fsm(IMSI-901700000004620:MSISDN-46071:GERAN-A:LU){LU_COMPL_VLR_S_DONE}: Terminating (cause = OSMO_FSM_TERM_PARENT)</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-msc/+/14899">change 14899</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-msc/+/14899"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I127b27937613ea0ff29d67991c0414fca6d441d9 </div>
<div style="display:none"> Gerrit-Change-Number: 14899 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>