Change in osmo-pcu[master]: BSSGP: use tlvp_val8() in gprs_bssgp_pcu_rx_paging_cs()

[laforge]

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/20986 )

Change subject: BSSGP: use tlvp_val8() in gprs_bssgp_pcu_rx_paging_cs()
......................................................................

BSSGP: use tlvp_val8() in gprs_bssgp_pcu_rx_paging_cs()

Change-Id: Ic1e37cb9938323c9b9f0466be5cf7251a6db1008
---
M src/gprs_bssgp_pcu.cpp
1 file changed, 1 insertion(+), 2 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  pespin: Looks good to me, but someone else must approve
  Jenkins Builder: Verified



diff --git a/src/gprs_bssgp_pcu.cpp b/src/gprs_bssgp_pcu.cpp
index 9b6cbaa..679f281 100644
--- a/src/gprs_bssgp_pcu.cpp
+++ b/src/gprs_bssgp_pcu.cpp
@@ -205,12 +205,11 @@
 {
 	struct osmo_mobile_identity mi;
 	int rc;
-	uint8_t *chan_needed = (uint8_t *)TLVP_VAL(tp, BSSGP_IE_CHAN_NEEDED);
 
 	if ((rc = get_paging_mi(&mi, tp)) > 0)
 		return bssgp_tx_status((enum gprs_bssgp_cause) rc, NULL, msg);
 
-	return BTS::main_bts()->add_paging(chan_needed ? *chan_needed : 0, &mi);
+	return BTS::main_bts()->add_paging(tlvp_val8(tp, BSSGP_IE_CHAN_NEEDED, 0), &mi);
 }
 
 static int gprs_bssgp_pcu_rx_paging_ps(struct msgb *msg, struct tlv_parsed *tp)

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

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Ic1e37cb9938323c9b9f0466be5cf7251a6db1008
Gerrit-Change-Number: 20986
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20201102/7e33bdcd/attachment.htm>