- gerrit-log - lists.osmocom.org

[S] Change in pysim[master]: smdpp: update certs

by dexter

Attention is currently required from: Hoernchen. dexter has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/pysim/+/40465?usp=email ) Change subject: smdpp: update certs ...................................................................... Patch Set 5: Code-Review+2 (1 comment) Patchset: PS5: I was looking forward to a patch like this. Unfortunately the story will repeat itself on 08/11/2025. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/40465?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I25442d6f55a385019bba1e47ad3d795120f850ad Gerrit-Change-Number: 40465 Gerrit-PatchSet: 5 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Comment-Date: Fri, 20 Jun 2025 11:50:34 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

3 days, 11 hours

1
0
0 0

[XS] Change in pysim[master]: smdpp: optional deps

by dexter

Attention is currently required from: Hoernchen, fixeria. dexter has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/pysim/+/40463?usp=email ) Change subject: smdpp: optional deps ...................................................................... Patch Set 4: Code-Review+2 (1 comment) Patchset: PS4: Unrelated to this patch, but I think we have a problem with the handling of the dependencies. We have setup.py, we have the readme.md and we have requirements.txt. They all list dependencies. As a first time user who wants to try osmo-smdpp.py I would be very confused. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/40463?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: If69b2bd5f8bc604443108c942c17eba9c22f4053 Gerrit-Change-Number: 40463 Gerrit-PatchSet: 4 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Fri, 20 Jun 2025 11:15:03 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

3 days, 11 hours

1
0
0 0

[M] Change in pysim[master]: smdpp: less verbose by default

by dexter

Attention is currently required from: Hoernchen. dexter has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/pysim/+/40468?usp=email ) Change subject: smdpp: less verbose by default ...................................................................... Patch Set 5: Code-Review+2 (1 comment) Patchset: PS5: I think this should be fine. Makes a lot of sense to me. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/40468?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I04a72b8f52417862d4dcba1f0743700dd942ef49 Gerrit-Change-Number: 40468 Gerrit-PatchSet: 5 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Comment-Date: Fri, 20 Jun 2025 11:01:29 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

3 days, 12 hours

1
0
0 0

[M] Change in pysim[master]: smdpp: less verbose by default

by laforge

Attention is currently required from: Hoernchen. laforge has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/pysim/+/40468?usp=email ) Change subject: smdpp: less verbose by default ...................................................................... Patch Set 5: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/40468?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I04a72b8f52417862d4dcba1f0743700dd942ef49 Gerrit-Change-Number: 40468 Gerrit-PatchSet: 5 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Comment-Date: Fri, 20 Jun 2025 09:49:30 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

3 days, 13 hours

1
0
0 0

[M] Change in pysim[master]: smdpp: less verbose by default

by laforge

Attention is currently required from: Hoernchen. laforge has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/pysim/+/40468?usp=email ) Change subject: smdpp: less verbose by default ...................................................................... Patch Set 5: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/40468?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I04a72b8f52417862d4dcba1f0743700dd942ef49 Gerrit-Change-Number: 40468 Gerrit-PatchSet: 5 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Comment-Date: Fri, 20 Jun 2025 09:49:22 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

3 days, 13 hours

1
0
0 0

[XS] Change in pysim[master]: smdpp: optional deps

by laforge

Attention is currently required from: Hoernchen, fixeria. laforge has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/pysim/+/40463?usp=email ) Change subject: smdpp: optional deps ...................................................................... Patch Set 4: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/40463?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: If69b2bd5f8bc604443108c942c17eba9c22f4053 Gerrit-Change-Number: 40463 Gerrit-PatchSet: 4 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Fri, 20 Jun 2025 09:49:08 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

3 days, 13 hours

1
0
0 0

[M] Change in osmo-ttcn3-hacks[master]: Bump titan_min to 11.1.0 where possible

by laforge

Attention is currently required from: fixeria, osmith, pespin. laforge has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40484?usp=email ) Change subject: Bump titan_min to 11.1.0 where possible ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40484?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: Iba562ace07e6f38a8cde15b17478a0a1680df2fc Gerrit-Change-Number: 40484 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Fri, 20 Jun 2025 09:48:42 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

3 days, 13 hours

1
0
0 0

[M] Change in libosmo-sigtran[master]: asp: Introduce support to configure and enable TCP keep-alive

by laforge

Attention is currently required from: fixeria, osmith, pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/40481?usp=email ) Change subject: asp: Introduce support to configure and enable TCP keep-alive ...................................................................... Patch Set 3: (1 comment) File src/ss7_asp.c: https://gerrit.osmocom.org/c/libosmo-sigtran/+/40481/comment/33f4f3c8_a337c… : PS3, Line 801: what kind of syntax is that "(void) rc;" without assigning any value to rc? I don't know what it is supposed to achieve. If rc of keepalive is to be ignored, why not simply set it to zero? -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40481?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: I2a9338053e741fb6dab94492c6bdc2badaf7afb1 Gerrit-Change-Number: 40481 Gerrit-PatchSet: 3 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Fri, 20 Jun 2025 07:26:39 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

3 days, 15 hours

1
0
0 0

[M] Change in libosmo-sigtran[master]: WIP: m3ua: Make sure MTP-PAUSE.ind is sent to SCCP when AS goes down

by pespin

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/40488?usp=email ) Change subject: WIP: m3ua: Make sure MTP-PAUSE.ind is sent to SCCP when AS goes down ...................................................................... WIP: m3ua: Make sure MTP-PAUSE.ind is sent to SCCP when AS goes down Whenever an SCTP link goes down, it may affect current set of accessible DPCs if any of the routes (AS) using that ASP become inactive. The approach presented in this patch is to, based on existing route table, figure out which DPCs may have been accessible before the AS went down, and then figure out which of those are not reachable anymore, then send MTP-PAUSE.ind for those. In turn, SCCP will trigger N-PCSTATE.ind towards its users. Change-Id: I0f28b049045fbea93fe91c99251e7aaa40c014e5 --- M src/xua_as_fsm.c M src/xua_snm.c 2 files changed, 173 insertions(+), 11 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran refs/changes/88/40488/1 diff --git a/src/xua_as_fsm.c b/src/xua_as_fsm.c index 3e117ee..1247db9 100644 --- a/src/xua_as_fsm.c +++ b/src/xua_as_fsm.c @@ -22,6 +22,7 @@ #include "xua_msg.h" #include <osmocom/sigtran/protocol/sua.h> #include <osmocom/sigtran/protocol/m3ua.h> +#include <sys/types.h> #include "ss7_as.h" #include "ss7_asp.h" @@ -413,12 +414,147 @@ } } + +#include "ss7_combined_linkset.h" +enum pc_map_item_state { + DPC_UNSET = 0, + DPC_ACTIVE, + DPC_LOST +}; +struct pc_map { + uint8_t v[16384]; /* 2^14, 14-bit PCs */ +}; + + +static void pcmap_fill_pc_mask(struct osmo_ss7_instance *ss7, struct pc_map *pcmap, uint32_t pc, uint32_t mask, enum pc_map_item_state val) +{ + unsigned int width = osmo_ss7_pc_width(&ss7->cfg.pc_fmt); + uint32_t maskbits = ~mask & ((1 << width)-1); + uint32_t fullpc; + + LOGP(DLSS7, LOGL_NOTICE, "PESPIN: FILL PCMAP PC=%u mask=0x%08x width=%u, maskbits=0x%08x\n", pc, mask, width, maskbits); + + for (fullpc = (pc & maskbits); fullpc <= (pc | maskbits); fullpc++) { + const enum pc_map_item_state prev_val = pcmap->v[fullpc]; + /* if an ACTIVE route is found for the DPC with a lower prio, it will be used, so it's not lost */ + if (prev_val == DPC_UNSET || val == DPC_ACTIVE) + pcmap->v[fullpc] = val; + } +} + + +static int pcmap_find_chunk_start(const struct pc_map *pcmap, unsigned int start, enum pc_map_item_state val) +{ + unsigned int i = start; + while (i < ARRAY_SIZE(pcmap->v)) { + if (pcmap->v[i] == val) + return i; + i++; + } + return -1; +} +static int pcmap_find_chunk_end(const struct pc_map *pcmap, unsigned int start, enum pc_map_item_state val) +{ + unsigned int i = start; + while (i < ARRAY_SIZE(pcmap->v)) { + if (pcmap->v[i] != val) + return i; + i++; + } + return i; +} + +static void as_gen_dpc_lost_bv(const struct osmo_ss7_as *as, struct pc_map *pcmap) +{ + struct osmo_ss7_instance *ss7 = as->inst; + struct osmo_ss7_route_table *rtbl = ss7->rtable_system; + struct osmo_ss7_combined_linkset *clset; + + llist_for_each_entry(clset, &rtbl->combined_linksets, list) { + struct osmo_ss7_route *rt; + bool as_match = false; + bool active = false; + /* If AS is involved and none of the combined linkset routes is working, add it: */ + llist_for_each_entry(rt, &clset->routes, list) { + if (rt->dest.as == as) + as_match = true; + else if (ss7_route_is_available(rt)) { + active = true; + break; + } + } + if (active) + pcmap_fill_pc_mask(ss7, pcmap, clset->cfg.pc, clset->cfg.mask, DPC_ACTIVE); + else if (as_match) + pcmap_fill_pc_mask(ss7, pcmap, clset->cfg.pc, clset->cfg.mask, DPC_LOST); + } +} + +static unsigned int fill_chunk(uint32_t *aff_pc, uint32_t start_pc, unsigned int num_pc, unsigned int width) +{ + if (num_pc == 1) { /* base case */ + aff_pc[0] = htonl(start_pc & 0xffffff); /* mask 0 */ + return 1; + } + + /* See if we have a whole contiguous block: */ + for (unsigned int lvl = width; lvl > 0; lvl--) { + LOGP(DLSS7, LOGL_NOTICE, "PESPIN: fill_chunk() lvl=%u %u %u\n", lvl, (start_pc & ((1 << lvl) - 1)), num_pc == (0x01 << (lvl + 1))); + + if ((start_pc & ((1 << lvl)-1)) == 0 && + num_pc == (0x01 << lvl)) { + aff_pc[0] = htonl(lvl << 24 | (start_pc & 0xffffff)); + return 1; + } + } + + LOGP(DLSS7, LOGL_NOTICE, "PESPIN: fill_chunk(START_PC=%u num_pc=%u width=%u) NOT CONTIGUOUS!\n", start_pc, num_pc, width); + + aff_pc[0] = htonl(start_pc & 0xffffff); /* mask 0 */ + return 1 + fill_chunk(aff_pc, start_pc + 1, num_pc - 1, width); +} + +static unsigned int as_became_unavailable_calc_affected_pc(const struct osmo_ss7_as *as, + uint32_t *aff_pc, unsigned int num_aff_pc) +{ + struct osmo_ss7_instance *ss7 = as->inst; + const uint32_t max_num_aff_pc = num_aff_pc; + + struct pc_map *pcmap = talloc_zero(NULL, struct pc_map); + as_gen_dpc_lost_bv(as, pcmap); + + num_aff_pc = 0; + unsigned int last_pos = 0; + int pos = 0; + + while ((pos = pcmap_find_chunk_start(pcmap, last_pos, DPC_LOST)) >= 0) { + unsigned int until = pcmap_find_chunk_end(pcmap, pos, DPC_LOST); + if (until - pos > max_num_aff_pc) { + LOGPAS(as, DLSS7, LOGL_ERROR, "Not enough space in aff_pc!\n"); + num_aff_pc += fill_chunk(&aff_pc[num_aff_pc], pos, max_num_aff_pc - num_aff_pc, osmo_ss7_pc_width(&ss7->cfg.pc_fmt)); + return num_aff_pc; + } + LOGPAS(as, DLSS7, LOGL_NOTICE, "PESPIN: FILL CHUNK PC=%u num_pcs=%u\n", pos, until - pos); + num_aff_pc += fill_chunk(&aff_pc[num_aff_pc], pos, until - pos, osmo_ss7_pc_width(&ss7->cfg.pc_fmt)); + LOGPAS(as, DLSS7, LOGL_NOTICE, "PESPIN: FILL CHUNK total num_aff_pc=%u\n", num_aff_pc); + last_pos = until + 1; + } + + return num_aff_pc; +} + +/* TODO: move to header file: */ +void xua_snm_pc_available_to_sccp(struct osmo_sccp_instance *sccp, + const uint32_t *aff_pc, unsigned int num_aff_pc, + bool available); + /* onenter call-back responsible of transmitting NTFY to all ASPs in * case of AS state changes */ static void xua_as_fsm_onenter(struct osmo_fsm_inst *fi, uint32_t old_state) { struct xua_as_fsm_priv *xafp = (struct xua_as_fsm_priv *) fi->priv; struct osmo_ss7_as *as = xafp->as; + struct osmo_ss7_instance *s7i = as->inst; struct osmo_xlm_prim_notify npar; switch (fi->state) { @@ -456,15 +592,38 @@ as_notify_all_asp(xafp->as, &npar); - /* only if we are the SG, we must start broadcasting availability information - * to everyone else */ - if (get_local_role(xafp->as) == OSMO_SS7_ASP_ROLE_SG) { + bool became_available = (old_state != XUA_AS_S_ACTIVE && fi->state == XUA_AS_S_ACTIVE); + bool became_unavailable = (old_state == XUA_AS_S_ACTIVE && fi->state != XUA_AS_S_ACTIVE); + int role = get_local_role(xafp->as); + + switch (role) { + case OSMO_SS7_ASP_ROLE_ASP: + if (s7i->sccp) { + if (became_available) { + LOGPAS(as, DLSS7, LOGL_NOTICE, "PESPIN: AS BECAME AVAILABLE!\n"); + /* TODO */ + } else if (became_unavailable) { + LOGPAS(as, DLSS7, LOGL_NOTICE, "PESPIN: AS BECAME UNAVAILABLE!\n"); + uint32_t *aff_pc = talloc_size(NULL, 16384); /* 2^14, 14-bit PCs */ + uint32_t num_aff_pc = 16384; + num_aff_pc = as_became_unavailable_calc_affected_pc(as, aff_pc, num_aff_pc); + xua_snm_pc_available_to_sccp(s7i->sccp, aff_pc, num_aff_pc, became_available); + talloc_free(aff_pc); + } + } + break; + case OSMO_SS7_ASP_ROLE_SG: + /* only if we are the SG, we must start broadcasting availability information + * to everyone else */ /* advertise availability of the routing key to others */ - uint32_t aff_pc = htonl(as->cfg.routing_key.pc); - if (old_state != XUA_AS_S_ACTIVE && fi->state == XUA_AS_S_ACTIVE) - xua_snm_pc_available(as, &aff_pc, 1, NULL, true); - else if (old_state == XUA_AS_S_ACTIVE && fi->state != XUA_AS_S_ACTIVE) - xua_snm_pc_available(as, &aff_pc, 1, NULL, false); + if (became_available || became_unavailable) { + uint32_t aff_pc = htonl(as->cfg.routing_key.pc); + xua_snm_pc_available(as, &aff_pc, 1, NULL, became_available); + } + break; + case OSMO_SS7_ASP_ROLE_IPSP: + /* TODO */ + break; } }; diff --git a/src/xua_snm.c b/src/xua_snm.c index 079cd96..2693306 100644 --- a/src/xua_snm.c +++ b/src/xua_snm.c @@ -117,9 +117,9 @@ } /* generate MTP-PAUSE / MTP-RESUME towards local SCCP users */ -static void xua_snm_pc_available_to_sccp(struct osmo_sccp_instance *sccp, - const uint32_t *aff_pc, unsigned int num_aff_pc, - bool available) +void xua_snm_pc_available_to_sccp(struct osmo_sccp_instance *sccp, + const uint32_t *aff_pc, unsigned int num_aff_pc, + bool available) { int i; for (i = 0; i < num_aff_pc; i++) { @@ -129,6 +129,8 @@ uint32_t pc = _aff_pc & 0xffffff; uint8_t mask = _aff_pc >> 24; + LOGPSCI(sccp, LOGL_ERROR, "PESPIN: mtp_pause pc=%u mask=%u available=%u\n", pc, mask, available); + if (!mask) { if (available) sccp_scmg_rx_mtp_resume(sccp, pc); @@ -140,6 +142,7 @@ uint32_t maskbits = (1 << mask) - 1; uint32_t fullpc; for (fullpc = (pc & ~maskbits); fullpc <= (pc | maskbits); fullpc++) { + LOGPSCI(sccp, LOGL_ERROR, "PESPIN: mtp_pause fullpc=%u available=%u\n", fullpc, available); if (available) sccp_scmg_rx_mtp_resume(sccp, fullpc); else -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40488?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: I0f28b049045fbea93fe91c99251e7aaa40c014e5 Gerrit-Change-Number: 40488 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de>

4 days, 4 hours

1
0
0 0

[XS] Change in libosmo-sigtran[master]: xua_snm.c: Fix wrong PC value passed in MTP-{PAUSE,RESUME} towards SCCP

by pespin

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/40487?usp=email ) Change subject: xua_snm.c: Fix wrong PC value passed in MTP-{PAUSE,RESUME} towards SCCP ...................................................................... xua_snm.c: Fix wrong PC value passed in MTP-{PAUSE,RESUME} towards SCCP Change-Id: I28bfb13744f2feb4adb610b341330725e2691a77 --- M src/xua_snm.c 1 file changed, 4 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran refs/changes/87/40487/1 diff --git a/src/xua_snm.c b/src/xua_snm.c index 3dd988f..079cd96 100644 --- a/src/xua_snm.c +++ b/src/xua_snm.c @@ -123,6 +123,8 @@ { int i; for (i = 0; i < num_aff_pc; i++) { + /* 32bit "Affected Point Code" consists of a 7-bit mask followed by 14/16/24-bit SS7 PC, + * see RFC 4666 3.4.1 */ uint32_t _aff_pc = ntohl(aff_pc[i]); uint32_t pc = _aff_pc & 0xffffff; uint8_t mask = _aff_pc >> 24; @@ -139,9 +141,9 @@ uint32_t fullpc; for (fullpc = (pc & ~maskbits); fullpc <= (pc | maskbits); fullpc++) { if (available) - sccp_scmg_rx_mtp_resume(sccp, pc); + sccp_scmg_rx_mtp_resume(sccp, fullpc); else - sccp_scmg_rx_mtp_pause(sccp, pc); + sccp_scmg_rx_mtp_pause(sccp, fullpc); } } } -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40487?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: I28bfb13744f2feb4adb610b341330725e2691a77 Gerrit-Change-Number: 40487 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de>

4 days, 9 hours

1
0
0 0

2025

2024

2023

2022

gerrit-log