dexter has uploaded this change for review. ( https://gerrit.osmocom.org/9169


Change subject: vlr_access_req_fsm: use correct cause codes
......................................................................

vlr_access_req_fsm: use correct cause codes

The FSM that controls the VLR ACCESS uses cause code 9
(GSM48_REJECT_MS_IDENTITY_NOT_DERVIVABLE) to signal that the
identity of the MS is currently not known in VLR (MSC-Reboot)
However, this cause code is from the GMM domain and is interpreted
as GSM48_REJECT_SRV_OPT_TMP_OUT_OF_ORDER by the MS, which cauese
the MS not to make a new LOCATION UPDATE on CM SERVICE REQUEST

- use GSM48_REJECT_IMSI_UNKNOWN_IN_VLR and
  GSM48_REJECT_IMSI_UNKNOWN_IN_VLR instead of
  GSM48_REJECT_IMSI_UNKNOWN_IN_VLR

Change-Id: Ic058c93387f9be9af4940f8961839c02b93ee370
Closes: OS#3266
---
M src/libvlr/vlr_access_req_fsm.c
1 file changed, 4 insertions(+), 4 deletions(-)



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

diff --git a/src/libvlr/vlr_access_req_fsm.c b/src/libvlr/vlr_access_req_fsm.c
index dd95821..79eb75f 100644
--- a/src/libvlr/vlr_access_req_fsm.c
+++ b/src/libvlr/vlr_access_req_fsm.c
@@ -381,7 +381,7 @@
        if (!par->by_tmsi) {
                /* We couldn't find a subscriber even by IMSI,
                 * Set User Error: Unidentified Subscriber */
-               proc_arq_fsm_done(fi, GSM48_REJECT_MS_IDENTITY_NOT_DERVIVABLE);
+               proc_arq_fsm_done(fi, GSM48_REJECT_IMSI_UNKNOWN_IN_VLR);
                return;
        } else {
                /* TMSI was included, are we permitted to use it? */
@@ -392,7 +392,7 @@
                        return;
                } else {
                        /* Set User Error: Unidentified Subscriber */
-                       proc_arq_fsm_done(fi, 
GSM48_REJECT_MS_IDENTITY_NOT_DERVIVABLE);
+                       proc_arq_fsm_done(fi, GSM48_REJECT_IMSI_UNKNOWN_IN_VLR);
                        return;
                }
        }
@@ -411,7 +411,7 @@
        vsub = vlr_subscr_find_by_imsi(vlr, par->imsi);
        if (!vsub) {
                /* Set User Error: Unidentified Subscriber */
-               proc_arq_fsm_done(fi, GSM48_REJECT_MS_IDENTITY_NOT_DERVIVABLE);
+               proc_arq_fsm_done(fi, GSM48_REJECT_IMSI_UNKNOWN_IN_VLR);
                return;
        }
        assoc_par_with_subscr(fi, vsub);
@@ -687,7 +687,7 @@
        case GSM_MI_TYPE_IMEI:
                /* TODO: IMEI (emergency call) */
        default:
-               proc_arq_fsm_done(fi, GSM48_REJECT_MS_IDENTITY_NOT_DERVIVABLE);
+               proc_arq_fsm_done(fi, GSM48_REJECT_INVALID_MANDANTORY_INF);
                return;
        }


--
To view, visit https://gerrit.osmocom.org/9169
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic058c93387f9be9af4940f8961839c02b93ee370
Gerrit-Change-Number: 9169
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pma...@sysmocom.de>

Reply via email to