laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-iuh/+/43590?usp=email )
Change subject: hnb-test: use N(SD)=1 on MM Auth Response ......................................................................
hnb-test: use N(SD)=1 on MM Auth Response
The Authentication Response carries a hard-coded N(SD) of 2 in the message type octet: 0x80 | GSM48_MT_MM_AUTH_RESP, with the comment "simulate sequence nr 2". That predates the duplicate detection of 3GPP TS 24.007 section 11.2.3.2.3 in osmo-msc.
The Location Updating Request that opens the connection in the InitialUE-Message is N(SD) = 0, so the Authentication Response, being the second uplink MM message, has to be N(SD) = 1. With 2, osmo-msc logs
Duplicate DTAP: bin=0, expected n_sd == 1, got 2
drops the response, and rejects the Location Update when its timer expires.
Set N(SD) to 1 so the response reaches the VLR. The remaining uplink MM messages are handled later in this series.
Change-Id: I4c71b404733e53d4e47d9ba5eb555b92eb1a771d Signed-off-by: Andrei Gosman andrei.gosman@gmail.com --- M tests/hnb-test.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: Jenkins Builder: Verified neels: Looks good to me, approved laforge: Looks good to me, but someone else must approve
diff --git a/tests/hnb-test.c b/tests/hnb-test.c index bfccf5a..597d04f 100644 --- a/tests/hnb-test.c +++ b/tests/hnb-test.c @@ -220,7 +220,7 @@ { uint8_t id_resp[] = { GSM48_PDISC_MM, - 0x80 | GSM48_MT_MM_AUTH_RESP, /* simulate sequence nr 2 */ + 0x40 | GSM48_MT_MM_AUTH_RESP, /* N(SD) = 1: second uplink MM message after the LU Request */ 0x61, 0xb5, 0x69, 0xf5 /* hardcoded SRES */ };