pespin has submitted this change. (
https://gerrit.osmocom.org/c/libosmo-gprs/+/33809 )
Change subject: gmm: Fix typos in test
......................................................................
gmm: Fix typos in test
The TLLI was wrong (I tooked the one from the Attach Accept msg instead
of RAU accept message defined above).
Change-Id: I2f7d61c72febad24fde285578e20485a23b8e175
---
M tests/gmm/gmm_prim_test.c
1 file changed, 15 insertions(+), 3 deletions(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, approved
diff --git a/tests/gmm/gmm_prim_test.c b/tests/gmm/gmm_prim_test.c
index 27b9336..2297518 100644
--- a/tests/gmm/gmm_prim_test.c
+++ b/tests/gmm/gmm_prim_test.c
@@ -438,11 +438,11 @@
/* As a result, MS submits GMMREG ATTACH.cnf */
/* Wait for READY timer to expire: */
- clock_override_add(44, 0); /* 44: See GMM Attach Accept (pdu_gmm_att_acc) feed above */
+ clock_override_add(44, 0); /* 44: See GMM Attach Accept (pdu_gmm_att_acc) fed above */
clock_debug("Expect T3314 (READY) timeout");
osmo_select_main(0);
- clock_override_add(10*60, 0); /* 44: See GMM Attach Accept (pdu_gmm_att_acc) feed above
*/
+ clock_override_add(10*60, 0); /* 10*60: See GMM Attach Accept (pdu_gmm_att_acc) fed
above */
clock_debug("Expect T3312 (periodic RAU) timeout");
osmo_select_main(0);
@@ -452,7 +452,7 @@
rc = osmo_gprs_gmm_prim_llc_lower_up(llc_prim);
OSMO_ASSERT(rc == 0);
/* update the used ptmsi to align with what was assigned from the network: */
- ptmsi = 0xea711b41;
+ ptmsi = 0xec999002;
tlli = gprs_tmsi2tlli(ptmsi, TLLI_LOCAL);
/* As a result, MS answers GMM RAU Complete */
--
To view, visit
https://gerrit.osmocom.org/c/libosmo-gprs/+/33809
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I2f7d61c72febad24fde285578e20485a23b8e175
Gerrit-Change-Number: 33809
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged