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

[S] Change in osmo-remsim[master]: remsim_client: only set usb parameters if given by arguments
by lynxis lazus 21 Aug '26

21 Aug '26

Attention is currently required from: laforge, lynxis lazus. Hello Jenkins Builder, laforge, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-remsim/+/43347?usp=email to look at the new patch set (#2). The following approvals got outdated and were removed: Code-Review+1 by laforge, Verified+1 by Jenkins Builder Change subject: remsim_client: only set usb parameters if given by arguments ...................................................................... remsim_client: only set usb parameters if given by arguments When calling osmo-remsim-client-st2 without setting the usb address (-A), previous libusb_claim_interface() was called with an usb address of 255, which is not valid on usb. Only set arguments to libusb_claim_interface() when given. Change-Id: I30f27b166db72fa8796aa4da98f9fc01e348c87a --- M src/client/user_simtrace2.c 1 file changed, 12 insertions(+), 6 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-remsim refs/changes/47/43347/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-remsim/+/43347?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: osmo-remsim Gerrit-Branch: master Gerrit-Change-Id: I30f27b166db72fa8796aa4da98f9fc01e348c87a Gerrit-Change-Number: 43347 Gerrit-PatchSet: 2 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>

1 0

[M] Change in libosmocore[master]: linuxlist: fix false-positive UBSan misaligned-access reports
by laforge 21 Aug '26

21 Aug '26

Attention is currently required from: fixeria, pespin. laforge has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/libosmocore/+/43150?usp=email ) Change subject: linuxlist: fix false-positive UBSan misaligned-access reports ...................................................................... Patch Set 2: (1 comment) Patchset: PS2: > Did anyone bother to check how upstream linux has solved this? Ideally llist_* (for linux list) shou […] @vyanitskiy@sysmocom.de ping? -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/43150?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I0424e76e76d8aa9402bd1a5aefe789de16e72fae Gerrit-Change-Number: 43150 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Thu, 20 Aug 2026 14:02:36 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: laforge <laforge(a)osmocom.org>

1 0

[S] Change in osmo-sgsn[pespin/rel-1.13.1]: Bump version: 1.13.1.5-1783-dirty → 1.13.2
by pespin 21 Aug '26

21 Aug '26

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43343?usp=email ) Change subject: Bump version: 1.13.1.5-1783-dirty → 1.13.2 ...................................................................... Bump version: 1.13.1.5-1783-dirty → 1.13.2 Change-Id: Ibada10d1b1434547bae509594fcd71ed6922a69d --- M debian/changelog 1 file changed, 14 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/debian/changelog b/debian/changelog index c769bf4..3690a6a 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,17 @@ +osmo-sgsn (1.13.2) unstable; urgency=medium + + [ Pau Espin Pedrol ] + * Pass pdp_t to sgsn_pdp_ctx_alloc() + * pdpctx: Use pdp->teic_own as rate_ctr id + * sndcp: Remove unused variable + * tests/v42bis: Remove unused variable + * ranap: Avoid write buffer overflow on GTPv1U outter IPv6 addr + + [ Vadim Yanitskiy ] + * gprs_sm: gsm48_tx_gsm_act_pdp_acc(): fix QoS profile length + + -- Pau Espin Pedrol <pespin(a)sysmocom.de> Thu, 20 Aug 2026 14:36:47 +0200 + osmo-sgsn (1.13.1) unstable; urgency=medium [ Alexander Couzens ] -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/43343?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-sgsn Gerrit-Branch: pespin/rel-1.13.1 Gerrit-Change-Id: Ibada10d1b1434547bae509594fcd71ed6922a69d Gerrit-Change-Number: 43343 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[XS] Change in osmo-sgsn[pespin/rel-1.13.1]: ranap: Avoid write buffer overflow on GTPv1U outter IPv6 addr
by pespin 21 Aug '26

21 Aug '26

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43342?usp=email ) Change subject: ranap: Avoid write buffer overflow on GTPv1U outter IPv6 addr ...................................................................... ranap: Avoid write buffer overflow on GTPv1U outter IPv6 addr If an IPv6 address was passed by libgtp in pdp->lib->gsnru, sgsn_pdp_ctx_iu_rab_activate() would write 16 bytes to an uint32_t variable, overflowing it and writing further on the stack. The proper extended fix can be found in osmo-sgsn.git master branch commit 6fd7c8b7fb0be7f96de7e7c5b4d22c54642f383a, this is a "backport" for it fixing the security issue (write buffer overflow) in a simple way, since the code has changed quite a lot in master. Change-Id: Id521fbc904f847952736e8d63f77a616f1f8e838 Related: OS#7060 Reported-By: Adam Bedard <adam.bedard(a)gmail.com> --- M src/sgsn/gprs_ranap.c 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/src/sgsn/gprs_ranap.c b/src/sgsn/gprs_ranap.c index 2ad663e..6f82850 100644 --- a/src/sgsn/gprs_ranap.c +++ b/src/sgsn/gprs_ranap.c @@ -236,6 +236,11 @@ use_x213_nsap = (uectx->rab_assign_addr_enc == RANAP_NSAP_ADDR_ENC_X213); /* Get the IP address for ggsn user plane */ + if (pdp->lib->gsnru.l != sizeof(ggsn_ip)) { + LOGPDPCTXP(LOGL_ERROR, pdp, "GSN-U address length %u unsupported\n", + pdp->lib->gsnru.l); + return -EINVAL; + } memcpy(&ggsn_ip, pdp->lib->gsnru.v, pdp->lib->gsnru.l); ggsn_ip = htonl(ggsn_ip); -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/43342?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-sgsn Gerrit-Branch: pespin/rel-1.13.1 Gerrit-Change-Id: Id521fbc904f847952736e8d63f77a616f1f8e838 Gerrit-Change-Number: 43342 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[XS] Change in osmo-sgsn[pespin/rel-1.13.1]: tests/v42bis: Remove unused variable
by pespin 21 Aug '26

21 Aug '26

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43341?usp=email ) Change subject: tests/v42bis: Remove unused variable ...................................................................... tests/v42bis: Remove unused variable gcc 16.1.1 warns about it. Change-Id: I715e57513ee62b7c61492a1d206232a5db107ea0 (cherry picked from commit 8972707b23ca79582def0d210f381dbc28e7809d) --- M tests/v42bis/v42bis_test.c 1 file changed, 0 insertions(+), 4 deletions(-) Approvals: osmith: Looks good to me, approved Jenkins Builder: Verified diff --git a/tests/v42bis/v42bis_test.c b/tests/v42bis/v42bis_test.c index e9c5069..0c58bfa 100644 --- a/tests/v42bis/v42bis_test.c +++ b/tests/v42bis/v42bis_test.c @@ -333,7 +333,6 @@ uint8_t *uncompressed; v42bis_state_t *rx_state; int rc; - int rc_sum = 0; int len; struct v42bis_output_buffer uncompressed_data; @@ -363,13 +362,10 @@ rx_state->decompress.user_data = (&uncompressed_data); rc = v42bis_decompress_flush(rx_state); printf("v42bis_decompress_flush() rc=%d\n", rc); - rc_sum += rc; rc = v42bis_decompress(rx_state, compressed, compressed_len); printf("v42bis_decompress() rc=%d\n", rc); - rc_sum += rc; rc = v42bis_decompress_flush(rx_state); printf("v42bis_decompress_flush() rc=%d\n", rc); - rc_sum += rc; /* Check results */ printf("compressed= %s ASCII:", -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/43341?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-sgsn Gerrit-Branch: pespin/rel-1.13.1 Gerrit-Change-Id: I715e57513ee62b7c61492a1d206232a5db107ea0 Gerrit-Change-Number: 43341 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[XS] Change in osmo-sgsn[pespin/rel-1.13.1]: sndcp: Remove unused variable
by pespin 21 Aug '26

21 Aug '26

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43340?usp=email ) Change subject: sndcp: Remove unused variable ...................................................................... sndcp: Remove unused variable gcc 16.1.1 warns about it. Change-Id: Ie2bd3d2935cd7e0ad023a163c9be9a05b83fbc5e (cherry picked from commit b40a36793c28339432a2ec0a97ca77502e136321) --- M src/sgsn/gprs_sndcp_xid.c 1 file changed, 1 insertion(+), 3 deletions(-) Approvals: osmith: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/sgsn/gprs_sndcp_xid.c b/src/sgsn/gprs_sndcp_xid.c index a19f645..344fda4 100644 --- a/src/sgsn/gprs_sndcp_xid.c +++ b/src/sgsn/gprs_sndcp_xid.c @@ -1352,7 +1352,6 @@ uint8_t tag; uint16_t tag_len; const uint8_t *val; - int byte_counter = 0; int rc; int tlv_count = 0; @@ -1403,8 +1402,7 @@ if (rc < 0) { talloc_free(comp_fields); return -EINVAL; - } else - byte_counter += rc; + } } /* Stop when no further TLV elements can be expected */ -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/43340?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-sgsn Gerrit-Branch: pespin/rel-1.13.1 Gerrit-Change-Id: Ie2bd3d2935cd7e0ad023a163c9be9a05b83fbc5e Gerrit-Change-Number: 43340 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[XS] Change in osmo-sgsn[pespin/rel-1.13.1]: pdpctx: Use pdp->teic_own as rate_ctr id
by pespin 21 Aug '26

21 Aug '26

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43338?usp=email ) Change subject: pdpctx: Use pdp->teic_own as rate_ctr id ...................................................................... pdpctx: Use pdp->teic_own as rate_ctr id Otherwise we run into errors due to non-unique ids. Related: SYS#7125 Change-Id: I388d7e4550e874cabc51e016d1a35a1074be7ac3 (cherry picked from commit 968b60ef3afc2735b9493e8acbf16429869bd6b9) --- M src/sgsn/pdpctx.c 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/src/sgsn/pdpctx.c b/src/sgsn/pdpctx.c index fb30a3a..be11338 100644 --- a/src/sgsn/pdpctx.c +++ b/src/sgsn/pdpctx.c @@ -70,7 +70,7 @@ if (!pctx) return NULL; - pctx->ctrg = rate_ctr_group_alloc(pctx, &pdpctx_ctrg_desc, nsapi); + pctx->ctrg = rate_ctr_group_alloc(pctx, &pdpctx_ctrg_desc, pdp->teic_own); if (!pctx->ctrg) { LOGPDPCTXP(LOGL_ERROR, pctx, "Error allocation counter group\n"); talloc_free(pctx); -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/43338?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-sgsn Gerrit-Branch: pespin/rel-1.13.1 Gerrit-Change-Id: I388d7e4550e874cabc51e016d1a35a1074be7ac3 Gerrit-Change-Number: 43338 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[M] Change in osmo-sgsn[pespin/rel-1.13.1]: Pass pdp_t to sgsn_pdp_ctx_alloc()
by pespin 21 Aug '26

21 Aug '26

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43337?usp=email ) Change subject: Pass pdp_t to sgsn_pdp_ctx_alloc() ...................................................................... Pass pdp_t to sgsn_pdp_ctx_alloc() Refactor sgsn_create_pdp_ctx() to pass the libgtp pdp_t to our pctx. This will be used in follow-up patch to use the teic_own as ID of the rate_ctr. Change-Id: Ie7312039fc97f50d09f00661e7c156610f3304b8 (cherry picked from commit 8ce5b15fdfc6c3b987830d271ed97b7ed0bc694b) --- M include/osmocom/sgsn/pdpctx.h M src/sgsn/pdpctx.c M src/sgsn/sgsn_libgtp.c 3 files changed, 31 insertions(+), 26 deletions(-) Approvals: osmith: Looks good to me, approved Jenkins Builder: Verified diff --git a/include/osmocom/sgsn/pdpctx.h b/include/osmocom/sgsn/pdpctx.h index 39d744a..a9446ae 100644 --- a/include/osmocom/sgsn/pdpctx.h +++ b/include/osmocom/sgsn/pdpctx.h @@ -90,6 +90,7 @@ struct sgsn_pdp_ctx *sgsn_pdp_ctx_alloc(struct sgsn_mm_ctx *mm, struct sgsn_ggsn_ctx *ggsn, + struct pdp_t *pdp, uint8_t nsapi); void sgsn_pdp_ctx_terminate(struct sgsn_pdp_ctx *pdp); void sgsn_pdp_ctx_free(struct sgsn_pdp_ctx *pdp); diff --git a/src/sgsn/pdpctx.c b/src/sgsn/pdpctx.c index e779420..fb30a3a 100644 --- a/src/sgsn/pdpctx.c +++ b/src/sgsn/pdpctx.c @@ -57,32 +57,40 @@ /* you don't want to use this directly, call sgsn_create_pdp_ctx() */ struct sgsn_pdp_ctx *sgsn_pdp_ctx_alloc(struct sgsn_mm_ctx *mm, struct sgsn_ggsn_ctx *ggsn, + struct pdp_t *pdp, uint8_t nsapi) { - struct sgsn_pdp_ctx *pdp; + struct sgsn_pdp_ctx *pctx; - pdp = sgsn_pdp_ctx_by_nsapi(mm, nsapi); - if (pdp) + pctx = sgsn_pdp_ctx_by_nsapi(mm, nsapi); + if (pctx) return NULL; - pdp = talloc_zero(sgsn, struct sgsn_pdp_ctx); - if (!pdp) + pctx = talloc_zero(sgsn, struct sgsn_pdp_ctx); + if (!pctx) return NULL; - pdp->mm = mm; - pdp->ggsn = ggsn; - pdp->nsapi = nsapi; - pdp->ctrg = rate_ctr_group_alloc(pdp, &pdpctx_ctrg_desc, nsapi); - if (!pdp->ctrg) { - LOGPDPCTXP(LOGL_ERROR, pdp, "Error allocation counter group\n"); - talloc_free(pdp); + pctx->ctrg = rate_ctr_group_alloc(pctx, &pdpctx_ctrg_desc, nsapi); + if (!pctx->ctrg) { + LOGPDPCTXP(LOGL_ERROR, pctx, "Error allocation counter group\n"); + talloc_free(pctx); return NULL; } - llist_add(&pdp->list, &mm->pdp_list); - sgsn_ggsn_ctx_add_pdp(pdp->ggsn, pdp); - llist_add(&pdp->g_list, &sgsn->pdp_list); - return pdp; + pdp->priv = pctx; + pctx->lib = pdp; + pctx->mm = mm; + pctx->ggsn = ggsn; + pctx->nsapi = nsapi; + + /* Back up our own local TEID in case we update the library one with RNC TEID when setting up Direct Tunnel: */ + pctx->sgsn_teid_own = pdp->teid_own; + + llist_add(&pctx->list, &mm->pdp_list); + sgsn_ggsn_ctx_add_pdp(pctx->ggsn, pctx); + llist_add(&pctx->g_list, &sgsn->pdp_list); + + return pctx; } /* diff --git a/src/sgsn/sgsn_libgtp.c b/src/sgsn/sgsn_libgtp.c index 385b843..eb06555 100644 --- a/src/sgsn/sgsn_libgtp.c +++ b/src/sgsn/sgsn_libgtp.c @@ -151,12 +151,6 @@ const uint8_t *qos; int rc; - pctx = sgsn_pdp_ctx_alloc(mmctx, ggsn, nsapi); - if (!pctx) { - LOGP(DGPRS, LOGL_ERROR, "Couldn't allocate PDP Ctx\n"); - return NULL; - } - imsi_ui64 = imsi_str2gtp(mmctx->imsi); rc = gtp_pdp_newpdp(ggsn->gsn, &pdp, imsi_ui64, nsapi, NULL); @@ -164,11 +158,13 @@ LOGP(DGPRS, LOGL_ERROR, "Out of libgtp PDP Contexts\n"); return NULL; } - pdp->priv = pctx; - pctx->lib = pdp; - /* Back up our own local TEID in case we update the library one with RNC TEID when setting up Direct Tunnel: */ - pctx->sgsn_teid_own = pdp->teid_own; + pctx = sgsn_pdp_ctx_alloc(mmctx, ggsn, pdp, nsapi); + if (!pctx) { + LOGP(DGPRS, LOGL_ERROR, "Couldn't allocate PDP Ctx\n"); + pdp_freepdp(pdp); + return NULL; + } //pdp->peer = /* sockaddr_in of GGSN (receive) */ //pdp->ipif = /* not used by library */ -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/43337?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-sgsn Gerrit-Branch: pespin/rel-1.13.1 Gerrit-Change-Id: Ie7312039fc97f50d09f00661e7c156610f3304b8 Gerrit-Change-Number: 43337 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[S] Change in osmo-sgsn[pespin/rel-1.13.1]: gprs_sm: gsm48_tx_gsm_act_pdp_acc(): fix QoS profile length
by pespin 21 Aug '26

21 Aug '26

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43339?usp=email ) Change subject: gprs_sm: gsm48_tx_gsm_act_pdp_acc(): fix QoS profile length ...................................................................... gprs_sm: gsm48_tx_gsm_act_pdp_acc(): fix QoS profile length The Activate PDP Context Accept was always sending sizeof(default_qos) (14 bytes) as the QoS profile length, regardless of what the UE requested. Older modules such as the Air20X may crash and restart during PDP attachment because of that. In GTP, qos_req.l encodes 1 ARP byte followed by the QoS profile octets, so (qos_req.l - 1) is the actual profile length. Mirror back the same QoS profile length the UE sent in its request, capped at sizeof(default_qos) to avoid overrunning the default_qos buffer. This matters in particular for R97/R98 UEs that send a 3-byte QoS profile and should not receive a 14-byte response. Change-Id: I11c24b64f0e49cf80c825969dbf018b2948d855c Related: OS#6922 (cherry picked from commit c1cf2817a3a0680ac6b69d40b134f881a314c7f0) --- M src/sgsn/gprs_sm.c 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: osmith: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/sgsn/gprs_sm.c b/src/sgsn/gprs_sm.c index bcf2923..4f55b57 100644 --- a/src/sgsn/gprs_sm.c +++ b/src/sgsn/gprs_sm.c @@ -206,7 +206,16 @@ /* FIXME: copy QoS parameters from original request */ //msgb_lv_put(msg, pdp->lib->qos_neg.l, pdp->lib->qos_neg.v); - msgb_lv_put(msg, sizeof(default_qos), (uint8_t *)&default_qos); + + /* qos_req.l is encoded as 1 (ARP byte) + N QoS profile bytes in GTP. + * Mirror back the same QoS profile length the UE requested, capped at + * sizeof(default_qos) (14 bytes, covering up to R99/R7 QoS format). */ + uint8_t qos_len = sizeof(default_qos); + if (pdp->lib->qos_req.l > 1) + qos_len = pdp->lib->qos_req.l - 1; + if (qos_len > sizeof(default_qos)) + qos_len = sizeof(default_qos); + msgb_lv_put(msg, qos_len, (uint8_t *)&default_qos); /* Radio priority 10.5.7.2 */ msgb_v_put(msg, pdp->lib->radio_pri); -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/43339?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-sgsn Gerrit-Branch: pespin/rel-1.13.1 Gerrit-Change-Id: I11c24b64f0e49cf80c825969dbf018b2948d855c Gerrit-Change-Number: 43339 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>

1 0

[S] Change in osmo-ci[master]: jobs/ttcn3-testsuites-testenv: use tmpfs
by pespin 21 Aug '26

21 Aug '26

Attention is currently required from: fixeria, osmith. pespin has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ci/+/43344?usp=email ) Change subject: jobs/ttcn3-testsuites-testenv: use tmpfs ...................................................................... Patch Set 1: -Code-Review -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/43344?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I4e9cd491f528bda2f556fc6922d8780785d058f6 Gerrit-Change-Number: 43344 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Thu, 20 Aug 2026 14:01:51 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

2026

2025

2024

2023

2022

gerrit-log August 2026