October 2024 - gerrit-log - lists.osmocom.org

[L] Change in osmo-sgsn[master]: Add Routing Areas
by laforge 12 Oct '24

12 Oct '24

Attention is currently required from: daniel, fixeria, lynxis lazus. laforge has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/37864?usp=email ) Change subject: Add Routing Areas ...................................................................... Patch Set 17: Code-Review+1 (1 comment) File tests/testsuite.at: https://gerrit.osmocom.org/c/osmo-sgsn/+/37864/comment/ab6e7ebf_0a2254d5?us… : PS15, Line 21: ignore > No, there aren't any changes. I still think it is fine, not checking the stderr. […] Done -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/37864?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-sgsn Gerrit-Branch: master Gerrit-Change-Id: I2474b19a7471a1dea3c863ddf8372b16180211aa Gerrit-Change-Number: 37864 Gerrit-PatchSet: 17 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Sat, 12 Oct 2024 17:01:08 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de> Comment-In-Reply-To: lynxis lazus <lynxis(a)fe80.eu>

1 0

[XS] Change in osmo-msc[master]: vlr: fix typo in comment
by laforge 12 Oct '24

12 Oct '24

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/38355?usp=email ) Change subject: vlr: fix typo in comment ...................................................................... vlr: fix typo in comment Change-Id: I06baf0494fde1d0c8eb6aaf72f5ce6e9134f3905 --- M src/libvlr/vlr.c 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/libvlr/vlr.c b/src/libvlr/vlr.c index e7569a1..f160578 100644 --- a/src/libvlr/vlr.c +++ b/src/libvlr/vlr.c @@ -1581,7 +1581,7 @@ return 0; } -/* MSC->VLR: Subscribre has disconnected */ +/* MSC->VLR: Subscriber has disconnected */ int vlr_subscr_disconnected(struct vlr_subscr *vsub) { /* This corresponds to a MAP-ABORT from MSC->VLR on a classic B -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/38355?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: I06baf0494fde1d0c8eb6aaf72f5ce6e9134f3905 Gerrit-Change-Number: 38355 Gerrit-PatchSet: 2 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[M] Change in osmo-msc[master]: vlr: refactor the reject cause codes
by laforge 12 Oct '24

12 Oct '24

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/38128?usp=email ) Change subject: vlr: refactor the reject cause codes ...................................................................... vlr: refactor the reject cause codes Use reject cause consistent within the VLR. Convert the GMM cause code to reject cause, which contains the exact same values. Reject cause includes both CS and PS cause codes, because CS and PS cause codes uses the same numbering space. See 24.008 annex G. Filter early the HLR provided cause code based on the CN domain, some values are invalid and will be converted into network failure. Change-Id: Ib269e9f865cd23f9f2c0a1bac0e17c95974c1c9f --- M include/osmocom/vlr/vlr.h M src/libvlr/vlr.c M src/libvlr/vlr_auth_fsm.c 3 files changed, 75 insertions(+), 65 deletions(-) Approvals: pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/include/osmocom/vlr/vlr.h b/include/osmocom/vlr/vlr.h index 8686020..27b40c2 100644 --- a/include/osmocom/vlr/vlr.h +++ b/include/osmocom/vlr/vlr.h @@ -430,7 +430,7 @@ int vlr_subscr_purge(struct vlr_subscr *vsub) __attribute__((warn_unused_result)); void vlr_subscr_cancel_attach_fsm(struct vlr_subscr *vsub, enum osmo_fsm_term_cause fsm_cause, - uint8_t gsm48_cause); + enum gsm48_reject_value gsm48_cause); void vlr_subscr_enable_expire_lu(struct vlr_subscr *vsub); @@ -491,5 +491,7 @@ void log_set_filter_vlr_subscr(struct log_target *target, struct vlr_subscr *vlr_subscr); -void vlr_gmm_cause_to_mm_cause(enum gsm48_gmm_cause gmm_cause, - enum gsm48_reject_value *gsm48_rej_p); +enum gsm48_reject_value vlr_gmm_cause_to_reject_cause(enum gsm48_gmm_cause gmm_cause); +enum gsm48_reject_value vlr_gmm_cause_to_reject_cause_domain(enum gsm48_gmm_cause gmm_cause, bool is_cs); +enum gsm48_reject_value vlr_reject_causes_cs(enum gsm48_reject_value reject_cause); +enum gsm48_reject_value vlr_reject_causes_ps(enum gsm48_reject_value reject_cause); diff --git a/src/libvlr/vlr.c b/src/libvlr/vlr.c index dbf9e67..e7569a1 100644 --- a/src/libvlr/vlr.c +++ b/src/libvlr/vlr.c @@ -434,7 +434,7 @@ void vlr_subscr_cancel_attach_fsm(struct vlr_subscr *vsub, enum osmo_fsm_term_cause fsm_cause, - uint8_t gsm48_cause) + enum gsm48_reject_value gsm48_cause) { if (!vsub) return; @@ -1166,100 +1166,112 @@ return 0; } -void vlr_gmm_cause_to_mm_cause(enum gsm48_gmm_cause gmm_cause, - enum gsm48_reject_value *gsm48_rej_p) +enum gsm48_reject_value vlr_gmm_cause_to_reject_cause_domain(enum gsm48_gmm_cause gmm_cause, bool is_cs) { - enum gsm48_reject_value gsm48_rej = GSM48_REJECT_NETWORK_FAILURE; + enum gsm48_reject_value reject_cause = vlr_gmm_cause_to_reject_cause(gmm_cause); + if (is_cs) + return vlr_reject_causes_cs(reject_cause); + else + return vlr_reject_causes_ps(reject_cause); +} + +enum gsm48_reject_value vlr_gmm_cause_to_reject_cause(enum gsm48_gmm_cause gmm_cause) +{ switch (gmm_cause) { case GMM_CAUSE_IMSI_UNKNOWN: - gsm48_rej = GSM48_REJECT_IMSI_UNKNOWN_IN_HLR; - break; + return GSM48_REJECT_IMSI_UNKNOWN_IN_HLR; case GMM_CAUSE_ILLEGAL_MS: - gsm48_rej = GSM48_REJECT_ILLEGAL_MS; - break; + return GSM48_REJECT_ILLEGAL_MS; case GMM_CAUSE_IMEI_NOT_ACCEPTED: - gsm48_rej = GSM48_REJECT_IMEI_NOT_ACCEPTED; - break; + return GSM48_REJECT_IMEI_NOT_ACCEPTED; case GMM_CAUSE_ILLEGAL_ME: - gsm48_rej = GSM48_REJECT_ILLEGAL_ME; - break; + return GSM48_REJECT_ILLEGAL_ME; case GMM_CAUSE_GPRS_NOTALLOWED: - gsm48_rej = GSM48_REJECT_GPRS_NOT_ALLOWED; - break; + return GSM48_REJECT_GPRS_NOT_ALLOWED; case GMM_CAUSE_GPRS_OTHER_NOTALLOWED: - gsm48_rej = GSM48_REJECT_SERVICES_NOT_ALLOWED; - break; + return GSM48_REJECT_SERVICES_NOT_ALLOWED; case GMM_CAUSE_MS_ID_NOT_DERIVED: - gsm48_rej = GSM48_REJECT_MS_IDENTITY_NOT_DERVIVABLE; - break; + return GSM48_REJECT_MS_IDENTITY_NOT_DERVIVABLE; case GMM_CAUSE_IMPL_DETACHED: - gsm48_rej = GSM48_REJECT_IMPLICITLY_DETACHED; - break; + return GSM48_REJECT_IMPLICITLY_DETACHED; case GMM_CAUSE_PLMN_NOTALLOWED: - gsm48_rej = GSM48_REJECT_PLMN_NOT_ALLOWED; - break; + return GSM48_REJECT_PLMN_NOT_ALLOWED; case GMM_CAUSE_LA_NOTALLOWED: - gsm48_rej = GSM48_REJECT_LOC_NOT_ALLOWED; - break; + return GSM48_REJECT_LOC_NOT_ALLOWED; case GMM_CAUSE_ROAMING_NOTALLOWED: - gsm48_rej = GSM48_REJECT_ROAMING_NOT_ALLOWED; - break; + return GSM48_REJECT_ROAMING_NOT_ALLOWED; case GMM_CAUSE_NO_GPRS_PLMN: - gsm48_rej = GSM48_REJECT_GPRS_NOT_ALLOWED_IN_PLMN; - break; + return GSM48_REJECT_GPRS_NOT_ALLOWED_IN_PLMN; case GMM_CAUSE_MSC_TEMP_NOTREACH: - gsm48_rej = GSM48_REJECT_MSC_TMP_NOT_REACHABLE; - break; + return GSM48_REJECT_MSC_TMP_NOT_REACHABLE; case GMM_CAUSE_SYNC_FAIL: - gsm48_rej = GSM48_REJECT_SYNCH_FAILURE; - break; + return GSM48_REJECT_SYNCH_FAILURE; case GMM_CAUSE_CONGESTION: - gsm48_rej = GSM48_REJECT_CONGESTION; - break; + return GSM48_REJECT_CONGESTION; case GMM_CAUSE_SEM_INCORR_MSG: - gsm48_rej = GSM48_REJECT_INCORRECT_MESSAGE; - break; + return GSM48_REJECT_INCORRECT_MESSAGE; case GMM_CAUSE_INV_MAND_INFO: - gsm48_rej = GSM48_REJECT_INVALID_MANDANTORY_INF; - break; + return GSM48_REJECT_INVALID_MANDANTORY_INF; case GMM_CAUSE_MSGT_NOTEXIST_NOTIMPL: - gsm48_rej = GSM48_REJECT_MSG_TYPE_NOT_IMPLEMENTED; - break; + return GSM48_REJECT_MSG_TYPE_NOT_IMPLEMENTED; case GMM_CAUSE_MSGT_INCOMP_P_STATE: - gsm48_rej = GSM48_REJECT_MSG_TYPE_NOT_COMPATIBLE; - break; + return GSM48_REJECT_MSG_TYPE_NOT_COMPATIBLE; case GMM_CAUSE_IE_NOTEXIST_NOTIMPL: - gsm48_rej = GSM48_REJECT_INF_ELEME_NOT_IMPLEMENTED; - break; + return GSM48_REJECT_INF_ELEME_NOT_IMPLEMENTED; case GMM_CAUSE_COND_IE_ERR: - gsm48_rej = GSM48_REJECT_CONDTIONAL_IE_ERROR; - break; + return GSM48_REJECT_CONDTIONAL_IE_ERROR; case GMM_CAUSE_MSG_INCOMP_P_STATE: - gsm48_rej = GSM48_REJECT_MSG_NOT_COMPATIBLE; - break; + return GSM48_REJECT_MSG_NOT_COMPATIBLE; case GMM_CAUSE_PROTO_ERR_UNSPEC: - gsm48_rej = GSM48_REJECT_PROTOCOL_ERROR; - break; - + return GSM48_REJECT_PROTOCOL_ERROR; case GMM_CAUSE_NO_SUIT_CELL_IN_LA: + return GSM48_REJECT_NO_SUIT_CELL_IN_LA; case GMM_CAUSE_MAC_FAIL: + return GSM48_REJECT_MAC_FAILURE; case GMM_CAUSE_GSM_AUTH_UNACCEPT: + return GSM48_REJECT_GSM_AUTH_UNACCEPTABLE; case GMM_CAUSE_NOT_AUTH_FOR_CSG: + return GSM48_REJECT_NOT_AUTH_FOR_CSG; case GMM_CAUSE_SMS_VIA_GPRS_IN_RA: + return GSM48_REJECT_SMS_PROV_VIA_GPRS_IN_RA; case GMM_CAUSE_NO_PDP_ACTIVATED: + return GSM48_REJECT_NO_PDP_CONTEXT_ACTIVATED; case GMM_CAUSE_NET_FAIL: - gsm48_rej = GSM48_REJECT_NETWORK_FAILURE; - break; + return GSM48_REJECT_NETWORK_FAILURE; + default: + return GSM48_REJECT_NETWORK_FAILURE; } +} - *gsm48_rej_p = gsm48_rej; +enum gsm48_reject_value vlr_reject_causes_ps(enum gsm48_reject_value reject_cause) +{ + switch (reject_cause) { + case GSM48_REJECT_CALL_CAN_NOT_BE_IDENTIFIED: + return GSM48_REJECT_NETWORK_FAILURE; + default: + return reject_cause; + } +} + +enum gsm48_reject_value vlr_reject_causes_cs(enum gsm48_reject_value reject_cause) +{ + switch (reject_cause) { + case GSM48_REJECT_NO_SUIT_CELL_IN_LA: + case GSM48_REJECT_MAC_FAILURE: + case GSM48_REJECT_GSM_AUTH_UNACCEPTABLE: + case GSM48_REJECT_NOT_AUTH_FOR_CSG: + case GSM48_REJECT_SMS_PROV_VIA_GPRS_IN_RA: + case GSM48_REJECT_NO_PDP_CONTEXT_ACTIVATED: + return GSM48_REJECT_NETWORK_FAILURE; + default: + return reject_cause; + } } /* Handle LOCATION CANCEL request from HLR */ static int vlr_subscr_handle_cancel_req(struct vlr_subscr *vsub, const struct osmo_gsup_message *gsup_msg) { - enum gsm48_reject_value gsm48_rej; enum osmo_fsm_term_cause fsm_cause = OSMO_FSM_TERM_ERROR; struct osmo_gsup_message gsup_reply = {0}; int rc, is_update_procedure = !gsup_msg->cancel_type || @@ -1274,8 +1286,7 @@ gsup_reply.message_type = OSMO_GSUP_MSGT_LOCATION_CANCEL_RESULT; rc = vlr_subscr_tx_gsup_message(vsub, &gsup_reply); - vlr_gmm_cause_to_mm_cause(gsup_msg->cause, &gsm48_rej); - vlr_subscr_cancel_attach_fsm(vsub, fsm_cause, gsm48_rej); + vlr_subscr_cancel_attach_fsm(vsub, fsm_cause, vlr_gmm_cause_to_reject_cause_domain(gsup_msg->cause, true)); vlr_rate_ctr_inc(vsub->vlr, VLR_CTR_DETACH_BY_CANCEL); vlr_subscr_detach(vsub); diff --git a/src/libvlr/vlr_auth_fsm.c b/src/libvlr/vlr_auth_fsm.c index 0f7efc6..5c16946 100644 --- a/src/libvlr/vlr_auth_fsm.c +++ b/src/libvlr/vlr_auth_fsm.c @@ -358,7 +358,6 @@ struct auth_fsm_priv *afp = fi->priv; struct vlr_subscr *vsub = afp->vsub; struct osmo_gsup_message *gsup = data; - enum gsm48_reject_value gsm48_rej; if (event == VLR_AUTH_E_HLR_SAI_NACK) LOGPFSM(fi, "GSUP: rx Auth Info Error cause: %d: %s\n", @@ -391,12 +390,10 @@ case VLR_AUTH_E_HLR_SAI_NACK: /* HLR did not return Auth Info, hence cannot authenticate. (The caller may still decide to permit * attaching without authentication) */ - vlr_gmm_cause_to_mm_cause(gsup->cause, &gsm48_rej); - auth_fsm_term(fi, AUTH_FSM_NO_AUTH_INFO, gsm48_rej); + auth_fsm_term(fi, AUTH_FSM_NO_AUTH_INFO, vlr_gmm_cause_to_reject_cause_domain(gsup->cause, true)); break; case VLR_AUTH_E_HLR_SAI_ABORT: - vlr_gmm_cause_to_mm_cause(gsup->cause, &gsm48_rej); - auth_fsm_term(fi, AUTH_FSM_FAILURE, gsm48_rej); + auth_fsm_term(fi, AUTH_FSM_FAILURE, vlr_gmm_cause_to_reject_cause_domain(gsup->cause, true)); break; } -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/38128?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: Ib269e9f865cd23f9f2c0a1bac0e17c95974c1c9f Gerrit-Change-Number: 38128 Gerrit-PatchSet: 3 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[M] Change in osmo-msc[master]: vlr: refactor the reject cause codes
by laforge 12 Oct '24

12 Oct '24

Attention is currently required from: lynxis lazus. laforge has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-msc/+/38128?usp=email ) Change subject: vlr: refactor the reject cause codes ...................................................................... Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/38128?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: Ib269e9f865cd23f9f2c0a1bac0e17c95974c1c9f Gerrit-Change-Number: 38128 Gerrit-PatchSet: 3 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Sat, 12 Oct 2024 16:59:01 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in osmo-ttcn3-hacks[master]: SGSN: TC_attach_pdp_act_pmm_idle: use correct Service Request type
by laforge 12 Oct '24

12 Oct '24

Attention is currently required from: lynxis lazus. laforge has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38416?usp=email ) Change subject: SGSN: TC_attach_pdp_act_pmm_idle: use correct Service Request type ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38416?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I9301e7a37d96c3f93e77f7618d6c13dfa1fbe46b Gerrit-Change-Number: 38416 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Sat, 12 Oct 2024 16:58:20 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in osmo-ttcn3-hacks[master]: BSSGP_ConnHdlr.ttcn: Service Request: expect a CommonId when stopping...
by laforge 12 Oct '24

12 Oct '24

Attention is currently required from: lynxis lazus. laforge has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38415?usp=email ) Change subject: BSSGP_ConnHdlr.ttcn: Service Request: expect a CommonId when stopping on SecurityCommand Complete ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38415?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Ie81c5eb01eb695d0399770b8a7feb9b4549148b3 Gerrit-Change-Number: 38415 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Sat, 12 Oct 2024 16:58:12 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in osmo-ttcn3-hacks[master]: SGSN: fix encoding of IMEISV in Auth and Ciphering Response
by laforge 12 Oct '24

12 Oct '24

Attention is currently required from: lynxis lazus. laforge has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38414?usp=email ) Change subject: SGSN: fix encoding of IMEISV in Auth and Ciphering Response ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38414?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I79dd79840de58f450c1a05e704fd9bd831455716 Gerrit-Change-Number: 38414 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Sat, 12 Oct 2024 16:57:54 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in osmo-ttcn3-hacks[master]: SGSN: f_TC_cell_change_different_*: always expect the auth
by laforge 12 Oct '24

12 Oct '24

Attention is currently required from: lynxis lazus. laforge has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38404?usp=email ) Change subject: SGSN: f_TC_cell_change_different_*: always expect the auth ...................................................................... Patch Set 2: Code-Review+1 (1 comment) Patchset: PS2: here I'm fine with the change as it doesn't widen the expectations of the test, it simply fixes the expectation to math the spec, even though our old code violated it. -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38404?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I24579b5fc843caf68e035e106cee4e4ec0a3c735 Gerrit-Change-Number: 38404 Gerrit-PatchSet: 2 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Sat, 12 Oct 2024 16:57:40 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

1 0

[XS] Change in osmo-ttcn3-hacks[master]: SGSN: f_TC_paging_ps: call f_gmm_attach() direct
by laforge 12 Oct '24

12 Oct '24

Attention is currently required from: lynxis lazus. laforge has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38403?usp=email ) Change subject: SGSN: f_TC_paging_ps: call f_gmm_attach() direct ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38403?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I68c75f84b01a0cd673ffd8c19ab5f0a8517c5738 Gerrit-Change-Number: 38403 Gerrit-PatchSet: 2 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Sat, 12 Oct 2024 16:56:57 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in osmo-ttcn3-hacks[master]: SGSN: f_TC_hlr_location_cancel_request_withdraw: drop incorrect Updat...
by laforge 12 Oct '24

12 Oct '24

Attention is currently required from: lynxis lazus. laforge has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38402?usp=email ) Change subject: SGSN: f_TC_hlr_location_cancel_request_withdraw: drop incorrect Update Location response ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38402?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Ib8265416c650b0983d7e85786b334f84191331b2 Gerrit-Change-Number: 38402 Gerrit-PatchSet: 2 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Sat, 12 Oct 2024 16:56:45 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0