Change in ...osmo-pcu[master]: gprs_bssgp_pcu.cpp: check return code of gsm48_mi_to_string()

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

fixeria gerrit-no-reply at lists.osmocom.org
Wed Jul 24 09:44:30 UTC 2019


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


Change subject: gprs_bssgp_pcu.cpp: check return code of gsm48_mi_to_string()
......................................................................

gprs_bssgp_pcu.cpp: check return code of gsm48_mi_to_string()

Change-Id: Id1ad279ce9bef38eb3d11ac62337276207e8d8bd
---
M src/gprs_bssgp_pcu.cpp
1 file changed, 14 insertions(+), 3 deletions(-)



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

diff --git a/src/gprs_bssgp_pcu.cpp b/src/gprs_bssgp_pcu.cpp
index 3ea3e59..730176a 100644
--- a/src/gprs_bssgp_pcu.cpp
+++ b/src/gprs_bssgp_pcu.cpp
@@ -93,6 +93,7 @@
 	char imsi[16] = "000";
 	uint8_t ms_class = 0;
 	uint8_t egprs_ms_class = 0;
+	int rc;
 #if 0
 	MS_Radio_Access_capability_t rac;
 #endif
@@ -120,8 +121,12 @@
 	 * will listen to all paging blocks. */
 	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));
+		rc = gsm48_mi_to_string(imsi, sizeof(imsi), TLVP_VAL(tp, BSSGP_IE_IMSI),
+							    TLVP_LEN(tp, BSSGP_IE_IMSI));
+		if (rc) {
+			LOGP(DBSSGP, LOGL_NOTICE, "BSSGP TLLI=0x%08x Failed to parse IMSI IE\n", tlli);
+			return bssgp_tx_status(BSSGP_CAUSE_COND_IE_ERR, NULL, msg);
+		}
 	}
 
 #if 0 /* Do not rely on this IE. TODO: make this configurable */
@@ -172,6 +177,7 @@
 	char imsi[16];
 	uint8_t *ptmsi = (uint8_t *) TLVP_VAL(tp, BSSGP_IE_TMSI);
 	uint16_t ptmsi_len = TLVP_LEN(tp, BSSGP_IE_TMSI);
+	int rc;
 
 	LOGP(DBSSGP, LOGL_NOTICE, " P-TMSI = ");
 	for (int i = 0; i < ptmsi_len; i++)
@@ -185,7 +191,12 @@
 		return -EINVAL;
 	}
 
-	gsm48_mi_to_string(imsi, sizeof(imsi), TLVP_VAL(tp, BSSGP_IE_IMSI), TLVP_LEN(tp, BSSGP_IE_IMSI));
+	rc = gsm48_mi_to_string(imsi, sizeof(imsi), TLVP_VAL(tp, BSSGP_IE_IMSI),
+						    TLVP_LEN(tp, BSSGP_IE_IMSI));
+	if (rc) {
+		LOGP(DBSSGP, LOGL_NOTICE, "Failed to parse IMSI IE\n");
+		return bssgp_tx_status(BSSGP_CAUSE_COND_IE_ERR, NULL, msg);
+	}
 
 	return gprs_rlcmac_paging_request(ptmsi, ptmsi_len, imsi);
 }

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

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Id1ad279ce9bef38eb3d11ac62337276207e8d8bd
Gerrit-Change-Number: 14923
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <axilirator at gmail.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190724/ed0e8c6b/attachment.htm>


More information about the gerrit-log mailing list