August 2026 - gerrit-log - lists.osmocom.org

[M] Change in osmo-msc[master]: libvlr: fix linter issues
by lynxis lazus 31 Aug '26

31 Aug '26

lynxis lazus has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-msc/+/43482?usp=email ) Change subject: libvlr: fix linter issues ...................................................................... libvlr: fix linter issues Make the linter happy by fixing all issues: - add spaces when policy require it - remove unnecessary return;'s - remove else in `if () {return;} else { foo; }` Change-Id: I99f424b196e3b583716d05e77dcd03b907920af1 --- M include/osmocom/vlr/vlr.h M src/libvlr/vlr.c M src/libvlr/vlr_access_req_fsm.c M src/libvlr/vlr_auth_fsm.c M src/libvlr/vlr_lu_fsm.c M src/libvlr/vlr_sgs.c 6 files changed, 48 insertions(+), 51 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/82/43482/1 diff --git a/include/osmocom/vlr/vlr.h b/include/osmocom/vlr/vlr.h index 28c23a3..9a64a5a 100644 --- a/include/osmocom/vlr/vlr.h +++ b/include/osmocom/vlr/vlr.h @@ -119,7 +119,7 @@ struct vlr_instance *vlr; /* TODO either populate from HLR or drop this completely? */ - long long unsigned int id; + unsigned long long id; /* Data from HLR */ /* 3GPP TS 23.008 */ /* Always use vlr_subscr_set_imsi() to write to imsi[] */ diff --git a/src/libvlr/vlr.c b/src/libvlr/vlr.c index eb1f209..3521519 100644 --- a/src/libvlr/vlr.c +++ b/src/libvlr/vlr.c @@ -237,15 +237,15 @@ present = true; } if (vsub->msisdn[0]) { - OSMO_STRBUF_PRINTF(sb, "%sMSISDN-%s", present? ":" : "", vsub->msisdn); + OSMO_STRBUF_PRINTF(sb, "%sMSISDN-%s", present ? ":" : "", vsub->msisdn); present = true; } if (vsub->tmsi != GSM_RESERVED_TMSI) { - OSMO_STRBUF_PRINTF(sb, "%sTMSI-0x%08X", present? ":" : "", vsub->tmsi); + OSMO_STRBUF_PRINTF(sb, "%sTMSI-0x%08X", present ? ":" : "", vsub->tmsi); present = true; } if (vsub->tmsi_new != GSM_RESERVED_TMSI) { - OSMO_STRBUF_PRINTF(sb, "%sTMSInew-0x%08X", present? ":" : "", vsub->tmsi_new); + OSMO_STRBUF_PRINTF(sb, "%sTMSInew-0x%08X", present ? ":" : "", vsub->tmsi_new); present = true; } if (!present) @@ -257,7 +257,7 @@ const char *vlr_subscr_short_name(const struct vlr_subscr *vsub, unsigned int maxlen) { /* cast away the const so we can shorten the string within the static buffer */ - char *name = (char*)vlr_subscr_name(vsub); + char *name = (char *) vlr_subscr_name(vsub); size_t len = strlen(name); if (maxlen < 2) return "-"; @@ -379,7 +379,7 @@ level = LOGL_DEBUG; LOGPSRC(g_vlr_log_cat[OSMO_VLR_LOGC_VLR], level, file, line, "VLR subscr %s %s %s: now used by %s\n", - vlr_subscr_name(vsub), (e->count - old_use_count) > 0? "+" : "-", e->use, + vlr_subscr_name(vsub), (e->count - old_use_count) > 0 ? "+" : "-", e->use, osmo_use_count_name_buf(buf, sizeof(buf), e->use_count)); if (e->count < 0) @@ -712,8 +712,8 @@ } LOGVLR(LOGL_DEBUG, "set Last E-UTRAN PLMN ID on subscriber: %s\n", vsub->sgs.last_eutran_plmn_present ? - osmo_plmn_name(&vsub->sgs.last_eutran_plmn) : - "(none)"); + osmo_plmn_name(&vsub->sgs.last_eutran_plmn) : + "(none)"); } bool vlr_subscr_matches_imsi(struct vlr_subscr *vsub, const char *imsi) @@ -817,7 +817,7 @@ struct sgsn_subscriber_pdp_data * vlr_subscr_pdp_data_alloc(struct vlr_subscr *vsub) { - struct sgsn_subscriber_pdp_data* pdata; + struct sgsn_subscriber_pdp_data *pdata; pdata = talloc_zero(vsub, struct sgsn_subscriber_pdp_data); @@ -1454,7 +1454,7 @@ switch (mi->type) { case GSM_MI_TYPE_IMSI: return osmo_fsm_inst_dispatch(vsub->auth_fsm, - VLR_AUTH_E_MS_ID_IMSI, (void*)mi->imsi); + VLR_AUTH_E_MS_ID_IMSI, (void *) mi->imsi); break; } } @@ -1462,11 +1462,11 @@ if (vsub->lu_fsm) { switch (mi->type) { case GSM_MI_TYPE_IMSI: - return osmo_fsm_inst_dispatch(vsub->lu_fsm, VLR_ULA_E_ID_IMSI, (void*)mi->imsi); + return osmo_fsm_inst_dispatch(vsub->lu_fsm, VLR_ULA_E_ID_IMSI, (void *) mi->imsi); case GSM_MI_TYPE_IMEI: - return osmo_fsm_inst_dispatch(vsub->lu_fsm, VLR_ULA_E_ID_IMEI, (void*)mi->imei); + return osmo_fsm_inst_dispatch(vsub->lu_fsm, VLR_ULA_E_ID_IMEI, (void *) mi->imei); case GSM_MI_TYPE_IMEISV: - return osmo_fsm_inst_dispatch(vsub->lu_fsm, VLR_ULA_E_ID_IMEISV, (void*)mi->imeisv); + return osmo_fsm_inst_dispatch(vsub->lu_fsm, VLR_ULA_E_ID_IMEISV, (void *) mi->imeisv); default: return -EINVAL; } diff --git a/src/libvlr/vlr_access_req_fsm.c b/src/libvlr/vlr_access_req_fsm.c index 6d4a084..53caae5 100644 --- a/src/libvlr/vlr_access_req_fsm.c +++ b/src/libvlr/vlr_access_req_fsm.c @@ -108,7 +108,7 @@ static const char *vlr_proc_arq_result_name(const struct osmo_fsm_inst *fi) { struct proc_arq_priv *par = fi->priv; - return par->result? gsm48_reject_value_name(par->result) : "PASSED"; + return par->result ? gsm48_reject_value_name(par->result) : "PASSED"; } #define proc_arq_fsm_done(fi, res) _proc_arq_fsm_done(fi, res, __FILE__, __LINE__) @@ -412,18 +412,17 @@ * Set User Error: Unidentified Subscriber */ proc_arq_fsm_done(fi, GSM48_REJECT_IMSI_UNKNOWN_IN_VLR); return; - } else { - /* TMSI was included, are we permitted to use it? */ - if (vlr->cfg.parq_retrieve_imsi) { - /* Obtain_IMSI_VLR */ - osmo_tdef_fsm_inst_state_chg(fi, PR_ARQ_S_WAIT_OBTAIN_IMSI, parq_fsm_state_tdef, vlr_tdefs, -1); - return; - } else { - /* Set User Error: Unidentified Subscriber */ - proc_arq_fsm_done(fi, GSM48_REJECT_IMSI_UNKNOWN_IN_VLR); - return; - } } + + /* TMSI was included, are we permitted to use it? */ + if (vlr->cfg.parq_retrieve_imsi) { + /* Obtain_IMSI_VLR */ + osmo_tdef_fsm_inst_state_chg(fi, PR_ARQ_S_WAIT_OBTAIN_IMSI, parq_fsm_state_tdef, vlr_tdefs, -1); + return; + } + + /* Set User Error: Unidentified Subscriber */ + proc_arq_fsm_done(fi, GSM48_REJECT_IMSI_UNKNOWN_IN_VLR); } /* ID REQ(IMSI) has returned */ @@ -493,7 +492,7 @@ if (!data) LOGPFSML(fi, LOGL_ERROR, "invalid ciphering result: NULL\n"); else - result = *(enum vlr_ciph_result_cause*)data; + result = *((enum vlr_ciph_result_cause *) data); switch (result) { case VLR_CIPH_COMPL: @@ -788,14 +787,14 @@ static const struct osmo_fsm_state upd_loc_child_vlr_states[] = { [ULC_S_IDLE] = { - .in_event_mask = , + .in_event_mask = TODO, .out_state_mask = S(ULC_S_WAIT_HLR_RESP) | S(ULC_S_DONE), .name = "IDLE", .action = upd_loc_child_f_idle, }, [ULC_S_WAIT_HLR_RESP] = { - .in_event_mask = , + .in_event_mask = TODO, .out_state_mask = S(ULC_S_DONE), .name = "WAIT-HLR-RESP", .action = upd_loc_child_f_w_hlr, diff --git a/src/libvlr/vlr_auth_fsm.c b/src/libvlr/vlr_auth_fsm.c index b3bd236..76858aa 100644 --- a/src/libvlr/vlr_auth_fsm.c +++ b/src/libvlr/vlr_auth_fsm.c @@ -196,7 +196,7 @@ " (MS is %sR99 capable, vec has %sUMTS AKA tokens, res_len=%u is %s)\n", is_r99 ? "" : "NOT ", (vec->auth_types & OSMO_AUTH_TYPE_UMTS) ? "" : "NO ", - res_len, (res_len == vec->res_len)? "valid" : "INVALID on UTRAN"); + res_len, (res_len == vec->res_len) ? "valid" : "INVALID on UTRAN"); goto out_false; } @@ -213,21 +213,21 @@ " context\n"); vsub->sec_ctx = VLR_SEC_CTX_UMTS; return true; - } else { - if (res_len != sizeof(vec->sres) - || memcmp(res, vec->sres, sizeof(vec->sres))) { - LOGVSUBP(LOGL_INFO, vsub, "GSM AUTH failure:" - " mismatching sres (expected sres=%s)\n", - osmo_hexdump(vec->sres, sizeof(vec->sres))); - goto out_false; - } - - LOGVSUBP(LOGL_INFO, vsub, "AUTH established GSM security" - " context\n"); - vsub->sec_ctx = VLR_SEC_CTX_GSM; - return true; } + if (res_len != sizeof(vec->sres) + || memcmp(res, vec->sres, sizeof(vec->sres))) { + LOGVSUBP(LOGL_INFO, vsub, "GSM AUTH failure:" + " mismatching sres (expected sres=%s)\n", + osmo_hexdump(vec->sres, sizeof(vec->sres))); + goto out_false; + } + + LOGVSUBP(LOGL_INFO, vsub, "AUTH established GSM security" + " context\n"); + vsub->sec_ctx = VLR_SEC_CTX_GSM; + return true; + out_false: vsub->sec_ctx = VLR_SEC_CTX_NONE; return false; diff --git a/src/libvlr/vlr_lu_fsm.c b/src/libvlr/vlr_lu_fsm.c index 4bed335..e63f1ee 100644 --- a/src/libvlr/vlr_lu_fsm.c +++ b/src/libvlr/vlr_lu_fsm.c @@ -157,7 +157,7 @@ static inline struct vlr_subscr *upd_hlr_vlr_fi_priv(struct osmo_fsm_inst *fi) { OSMO_ASSERT(fi->fsm == &upd_hlr_vlr_fsm); - return (struct vlr_subscr*)fi->priv; + return (struct vlr_subscr *) fi->priv; } struct osmo_fsm_inst * @@ -269,7 +269,7 @@ static inline struct vlr_subscr *sub_pres_vlr_fi_priv(struct osmo_fsm_inst *fi) { OSMO_ASSERT(fi->fsm == &sub_pres_vlr_fsm); - return (struct vlr_subscr*)fi->priv; + return (struct vlr_subscr *) fi->priv; } /* THIS IS CURRENTLY DEAD CODE, SINCE WE NEVER SET vsub->ms_not_reachable_flag = true. @@ -668,7 +668,7 @@ static inline struct lu_compl_vlr_priv *lu_compl_vlr_fi_priv(struct osmo_fsm_inst *fi) { OSMO_ASSERT(fi->fsm == &lu_compl_vlr_fsm); - return (struct lu_compl_vlr_priv*)fi->priv; + return (struct lu_compl_vlr_priv *) fi->priv; } struct osmo_fsm_inst * @@ -1271,7 +1271,7 @@ if (!data) LOGPFSML(fi, LOGL_ERROR, "invalid ciphering result: NULL\n"); else - result = *(enum vlr_ciph_result_cause*)data; + result = *((enum vlr_ciph_result_cause *) data); switch (result) { case VLR_CIPH_COMPL: @@ -1393,7 +1393,7 @@ case VLR_ULA_E_LU_COMPL_FAILURE: cause = GSM48_REJECT_NETWORK_FAILURE; if (data) - cause = *(uint8_t*)data; + cause = *((uint8_t *) data); lu_fsm_discard_lu_compl_fsm(fi); lu_fsm_failure(fi, cause); break; @@ -1425,7 +1425,7 @@ vsub->sub_dataconf_by_hlr_ind = false; cause = GSM48_REJECT_NETWORK_FAILURE; if (data) - cause = *(uint8_t*)data; + cause = *((uint8_t *) data); lu_fsm_discard_lu_compl_fsm(fi); lu_fsm_failure(fi, cause); break; @@ -1635,7 +1635,7 @@ static inline struct lu_fsm_priv *lu_fsm_fi_priv(struct osmo_fsm_inst *fi) { OSMO_ASSERT(fi->fsm == &vlr_lu_fsm); - return (struct lu_fsm_priv*)fi->priv; + return (struct lu_fsm_priv *) fi->priv; } static struct osmo_fsm_inst * diff --git a/src/libvlr/vlr_sgs.c b/src/libvlr/vlr_sgs.c index 755db1e..631ae29 100644 --- a/src/libvlr/vlr_sgs.c +++ b/src/libvlr/vlr_sgs.c @@ -319,8 +319,6 @@ /* Balance ref count increment from vlr_sgs_pag() */ vlr_subscr_put(vsub, VSUB_USE_SGS_PAGING_REQ); - - return; } /*! Notify that a paging message has been sent and a paging is now in progress. -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/43482?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: I99f424b196e3b583716d05e77dcd03b907920af1 Gerrit-Change-Number: 43482 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>

1 0

[S] Change in osmo-sgsn[master]: libvlr: add missing declaration of vlr_ra_update()
by lynxis lazus 31 Aug '26

31 Aug '26

Attention is currently required from: pespin. lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43463?usp=email ) Change subject: libvlr: add missing declaration of vlr_ra_update() ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: This commit is already part of the osmo-msc. -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/43463?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: I698b1fc577f561d7f09944bd189f05c709977801 Gerrit-Change-Number: 43463 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Mon, 31 Aug 2026 21:59:17 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[S] Change in osmo-sgsn[master]: libvlr: add missing declaration of vlr_ra_update()
by lynxis lazus 31 Aug '26

31 Aug '26

Attention is currently required from: lynxis lazus, pespin. lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43463?usp=email ) Change subject: libvlr: add missing declaration of vlr_ra_update() ...................................................................... Set Ready For Review -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/43463?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: I698b1fc577f561d7f09944bd189f05c709977801 Gerrit-Change-Number: 43463 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Mon, 31 Aug 2026 21:58:57 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No

1 0

[S] Change in osmo-sgsn[master]: Avoid accessing struct log_target members directly
by lynxis lazus 31 Aug '26

31 Aug '26

Attention is currently required from: pespin. lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43462?usp=email ) Change subject: Avoid accessing struct log_target members directly ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: > Why aren't you picking a newer osmo-msc patch in previous commit instead which already contains this […] Because I had to do a lot of work on this branch and when I was done, osmo-msc took more commits in the mean time. Also creating the synchronisation commit takes quite some time, so I like to prevent doing it over again. -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/43462?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: I4f42916080a2f4073bcc9bba9cce5870a23312b5 Gerrit-Change-Number: 43462 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Mon, 31 Aug 2026 21:58:42 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>

1 0

[XL] Change in osmo-sgsn[master]: add libvlr based on MSC commit 80955b9454776166dd43972db2141b431444ad7f
by lynxis lazus 31 Aug '26

31 Aug '26

Attention is currently required from: pespin. lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/39560?usp=email ) Change subject: add libvlr based on MSC commit 80955b9454776166dd43972db2141b431444ad7f ...................................................................... Patch Set 6: (1 comment) Patchset: PS5: > AFAIU the style issues haven yet been fixed in osmo-msc? No, not yet. I can try to fix those in osmo-msc. But re-doing this commit consumes a lot of time. -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/39560?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: I221114e00e6f338a75ae0dd7ac285e262b23635d Gerrit-Change-Number: 39560 Gerrit-PatchSet: 6 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Mon, 31 Aug 2026 21:56:59 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Comment-In-Reply-To: lynxis lazus <lynxis(a)fe80.eu>

1 0

[M] Change in osmo-ttcn3-hacks[master]: SGSN: Remove unsupported acl test cases
by lynxis lazus 31 Aug '26

31 Aug '26

Attention is currently required from: daniel, fixeria, laforge. lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39261?usp=email ) Change subject: SGSN: Remove unsupported acl test cases ...................................................................... Patch Set 17: (1 comment) Commit Message: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39261/comment/d5e7d952_d68a… : PS10, Line 7: unsupport > unsupport? Maybe say "no longer relevant"? Or just "ACL testcases"? Done -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39261?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: I1506324300fe3864069706d5b05860e7b44cd895 Gerrit-Change-Number: 39261 Gerrit-PatchSet: 17 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: dexter <pmaier(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-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Comment-Date: Mon, 31 Aug 2026 21:53:45 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>

1 0

[S] Change in osmo-sgsn[master]: Avoid accessing struct log_target members directly
by pespin 31 Aug '26

31 Aug '26

Attention is currently required from: lynxis lazus. pespin has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43462?usp=email ) Change subject: Avoid accessing struct log_target members directly ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: Why aren't you picking a newer osmo-msc patch in previous commit instead which already contains this sort of fixes? -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/43462?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: I4f42916080a2f4073bcc9bba9cce5870a23312b5 Gerrit-Change-Number: 43462 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Mon, 31 Aug 2026 19:08:55 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[S] Change in osmo-sgsn[master]: libvlr: Remove MSC specific parts
by pespin 31 Aug '26

31 Aug '26

Attention is currently required from: lynxis lazus. pespin has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43461?usp=email ) Change subject: libvlr: Remove MSC specific parts ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/43461?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: I35e3edc56c7c64ac2a4462dec8ea684b00ab3881 Gerrit-Change-Number: 43461 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Mon, 31 Aug 2026 19:07:34 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XL] Change in osmo-sgsn[master]: add libvlr based on MSC commit 80955b9454776166dd43972db2141b431444ad7f
by pespin 31 Aug '26

31 Aug '26

Attention is currently required from: lynxis lazus. pespin has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/39560?usp=email ) Change subject: add libvlr based on MSC commit 80955b9454776166dd43972db2141b431444ad7f ...................................................................... Patch Set 6: (1 comment) Patchset: PS5: > Then it sounds like this should really be fixed in osmo-msc and then copy-pasted here, or fix them i […] AFAIU the style issues haven yet been fixed in osmo-msc? -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/39560?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: I221114e00e6f338a75ae0dd7ac285e262b23635d Gerrit-Change-Number: 39560 Gerrit-PatchSet: 6 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Mon, 31 Aug 2026 19:07:13 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Comment-In-Reply-To: lynxis lazus <lynxis(a)fe80.eu>

1 0

[XS] Change in osmo-msc[master]: libvlr: lu_fsm: PS: don't terminate by timer to prevent cascade
by pespin 31 Aug '26

31 Aug '26

Attention is currently required from: laforge, lynxis lazus. pespin has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-msc/+/43481?usp=email ) Change subject: libvlr: lu_fsm: PS: don't terminate by timer to prevent cascade ...................................................................... Patch Set 1: (2 comments) Patchset: PS1: This commit requires better description, it's not clear at all. File src/libvlr/vlr_lu_fsm.c: https://gerrit.osmocom.org/c/osmo-msc/+/43481/comment/0ac149f7_13e3f3be?usp… : PS1, Line 1597: /* The callback might already cleaned up this FSM */ "might have already cleaned up". What about if it didn't clean it up? you can probably check something like fi.terminating or similar field to know (if delayed termination is enabled?). -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/43481?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: I483bd3e734b1046fcafdae31189bc5df5b122a48 Gerrit-Change-Number: 43481 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: laforge <laforge(a)osmocom.org> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Mon, 31 Aug 2026 19:03:22 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0