lynxis lazus has submitted this change. (
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38920?usp=email )
Change subject: SGSN: Context Req In: use correct TEID-C
......................................................................
SGSN: Context Req In: use correct TEID-C
For a MS/UE moving from 2G to 4G, the Ctx Acks need to transmit
with the correct TEID-C which the SGSN chose in the Context Response.
Change-Id: I84dc7dda8ee1227edd68d1482a4c5bf267c16418
---
M sgsn/SGSN_Tests.ttcn
1 file changed, 6 insertions(+), 2 deletions(-)
Approvals:
pespin: Looks good to me, but someone else must approve
Jenkins Builder: Verified
fixeria: Looks good to me, approved
diff --git a/sgsn/SGSN_Tests.ttcn b/sgsn/SGSN_Tests.ttcn
index 03a0ebb..2ddeaff 100644
--- a/sgsn/SGSN_Tests.ttcn
+++ b/sgsn/SGSN_Tests.ttcn
@@ -3082,12 +3082,16 @@
T.start(2.0);
alt {
[] GTP[GTP_GGSN_IDX].receive(tr_GTPC_SGSNContextResp(?, ?, ctx_rsp)) -> value gtpc_ud
{
+ var OCT4 remote_teidc :=
gtpc_ud.gtpc.gtpc_pdu.sgsn_ContextResponse.teidControlPlane.teidControlPlane;
+
log("Rx SGSN Context Resp from SGSN, sending Ack");
- GTP[GTP_GGSN_IDX].send(ts_GTPC_SGSNContextAck(gtpc_ud.peer, '12345678'O,
seq_nr));
+ GTP[GTP_GGSN_IDX].send(ts_GTPC_SGSNContextAck(gtpc_ud.peer, remote_teidc, seq_nr));
setverdict(pass);
}
[] GTP[GTP_GGSN_IDX].receive(tr_GTPC_SGSNContextResp) -> value gtpc_ud {
- GTP[GTP_GGSN_IDX].send(ts_GTPC_SGSNContextAck(gtpc_ud.peer, '12345678'O,
seq_nr,
+ var OCT4 remote_teidc :=
gtpc_ud.gtpc.gtpc_pdu.sgsn_ContextResponse.teidControlPlane.teidControlPlane;
+
+ GTP[GTP_GGSN_IDX].send(ts_GTPC_SGSNContextAck(gtpc_ud.peer, remote_teidc, seq_nr,
ts_SGSNContextAckPDU(GTP_CAUSE_INVALID_MSG_FORMAT)));
setverdict(fail, "Rx unexpected SGSN Context Resp");
}
--
To view, visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38920?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I84dc7dda8ee1227edd68d1482a4c5bf267c16418
Gerrit-Change-Number: 38920
Gerrit-PatchSet: 6
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>