fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-pcu/+/14922


Change subject: gprs_bssgp_pcu_rx_dl_ud(): fix: BSSGP_IE_IMSI is optional
......................................................................

gprs_bssgp_pcu_rx_dl_ud(): fix: BSSGP_IE_IMSI is optional

Change-Id: I940d220a399166122f33e67a222dd572085e1401
---
M src/gprs_bssgp_pcu.cpp
1 file changed, 5 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/22/14922/1

diff --git a/src/gprs_bssgp_pcu.cpp b/src/gprs_bssgp_pcu.cpp
index b34659b..3ea3e59 100644
--- a/src/gprs_bssgp_pcu.cpp
+++ b/src/gprs_bssgp_pcu.cpp
@@ -118,7 +118,11 @@
        /* read IMSI. if no IMSI exists, use first paging block (any paging),
         * because during attachment the IMSI might not be known, so the MS
         * will listen to all paging blocks. */
-       gsm48_mi_to_string(imsi, sizeof(imsi), TLVP_VAL(tp, BSSGP_IE_IMSI), 
TLVP_LEN(tp, BSSGP_IE_IMSI));
+       if (TLVP_PRESENT(tp, BSSGP_IE_IMSI))
+       {
+               gsm48_mi_to_string(imsi, sizeof(imsi), TLVP_VAL(tp, 
BSSGP_IE_IMSI),
+                                                      TLVP_LEN(tp, 
BSSGP_IE_IMSI));
+       }

 #if 0 /* Do not rely on this IE. TODO: make this configurable */
        /* parse ms radio access capability */

--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/14922
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I940d220a399166122f33e67a222dd572085e1401
Gerrit-Change-Number: 14922
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-MessageType: newchange

Reply via email to