lynxis lazus has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-msc/+/38492?usp=email )


Change subject: vlr: vlr_lu_fsm: update the HLR only on initial IMSI attach
......................................................................

vlr: vlr_lu_fsm: update the HLR only on initial IMSI attach

As long the VLR isn't supporting multiple MSCs, skip updating the HLR
on PERIODIC and REGULAR Location Updates (same for PS Attach Request/Routing 
Area Update).

Change-Id: I7adc2092557f8cc6bb358234df2d1a16d1036eee
---
M src/libvlr/vlr_lu_fsm.c
1 file changed, 9 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/92/38492/1

diff --git a/src/libvlr/vlr_lu_fsm.c b/src/libvlr/vlr_lu_fsm.c
index f315e6c..ed7c169 100644
--- a/src/libvlr/vlr_lu_fsm.c
+++ b/src/libvlr/vlr_lu_fsm.c
@@ -879,12 +879,20 @@
                /* FIXME: Delete subscriber record */
                /* LU REJ: Roaming not allowed */
                lu_fsm_failure(fi, GSM48_REJECT_ROAMING_NOT_ALLOWED);
-       } else {
+               return;
+       }
+
+       if (lfp->lu_type == VLR_LU_TYPE_IMSI_ATTACH) {
                /* Update_HLR_VLR */
                osmo_fsm_inst_state_chg(fi, VLR_ULA_S_WAIT_HLR_UPD,
                                        LU_TIMEOUT_LONG, 0);
                lfp->upd_hlr_vlr_fsm =
                        upd_hlr_vlr_proc_start(fi, vsub, 
VLR_ULA_E_UPD_HLR_COMPL);
+       } else {
+               /* PERIODIC and REGULAR, as long we don't support multiple MSC 
with different Location Areas */
+               osmo_fsm_inst_state_chg(fi, VLR_ULA_S_WAIT_LU_COMPL,
+                                       LU_TIMEOUT_LONG, 0);
+               vlr_loc_upd_start_lu_compl_fsm(fi);
        }
 }


--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/38492?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: newchange
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I7adc2092557f8cc6bb358234df2d1a16d1036eee
Gerrit-Change-Number: 38492
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <[email protected]>

Reply via email to