May 2025 - gerrit-log - lists.osmocom.org

[XS] Change in osmo-ttcn3-hacks[master]: testenv: podman: add missing tree command

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40312?usp=email ) Change subject: testenv: podman: add missing tree command ...................................................................... testenv: podman: add missing tree command The tree command is used by the testenv scripts when building QEMU initrds to quickly show the contents of the initrd before packing it. Fix for: + tree -a /tmp/testenv-ggsn_tests-osmo_ggsn_v4_only-20250509-1151-745e3562-dxt_lu1f/ggsn/_initrd osmo-ggsn/run.sh: 140: tree: not found That the build initramfs logic did not abort here is another bug that gets fixed in a follow-up patch in this series. Change-Id: I75fcd210db9e5c9364cb98ee09ebf754790aa55f --- M _testenv/data/podman/Dockerfile 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/_testenv/data/podman/Dockerfile b/_testenv/data/podman/Dockerfile index 9f92206..fb97ee8 100644 --- a/_testenv/data/podman/Dockerfile +++ b/_testenv/data/podman/Dockerfile @@ -85,6 +85,7 @@ sqlite3 \ sudo \ tcpdump \ + tree \ vim \ vsmartcard-vpcd \ wget \ -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40312?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I75fcd210db9e5c9364cb98ee09ebf754790aa55f Gerrit-Change-Number: 40312 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: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 month, 1 week

1
0
0 0

[S] Change in osmo-ttcn3-hacks[master]: testenv: fix not aborting on build_initrd error

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40314?usp=email ) Change subject: testenv: fix not aborting on build_initrd error ...................................................................... testenv: fix not aborting on build_initrd error Without this patch the build_initrd function keeps going on errors: + cp -a --parents /lib/modules/6.1.0-34-amd64/kernel/drivers/net/net_failover.ko /tmp/testenv-ggsn_tests-osmo_ggsn_v4_only-20250509-1040-a21668df-euc9e2fs/ggsn/_initrd/usr cp: 'lib/modules/6.1.0-34-amd64/kernel/drivers/net': No such file or directory + cp -a --parents /lib/modules/6.1.0-34-amd64/kernel/drivers/net/virtio_net.ko /tmp/testenv-ggsn_tests-osmo_ggsn_v4_only-20250509-1040-a21668df-euc9e2fs/ggsn/_initrd/usr … The reason for that is that even though the script runs with set -e, the -e flag gets ignored between if…then where the build_initrd shell function gets called: if ! build_initrd >build_initrd.log 2>&1; then qemu_initrd_exit_error "build_initrd.log" fi Fix it by using a trap instead. Related: https://unix.stackexchange.com/a/65564 Change-Id: I2870b7e7ba28b2afc72e86b7a3bc389103564c80 --- M _testenv/data/scripts/qemu/qemu_functions.sh M ggsn_tests/osmo-ggsn/run.sh 2 files changed, 18 insertions(+), 8 deletions(-) Approvals: pespin: Looks good to me, but someone else must approve fixeria: Looks good to me, but someone else must approve laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/_testenv/data/scripts/qemu/qemu_functions.sh b/_testenv/data/scripts/qemu/qemu_functions.sh index 9c92a37..46e7273 100755 --- a/_testenv/data/scripts/qemu/qemu_functions.sh +++ b/_testenv/data/scripts/qemu/qemu_functions.sh @@ -136,10 +136,12 @@ | gzip -1 > "$INITRD_DIR".gz ) } +# Trap for qemu_build_initrd_with_log_redirect +# Building the initrd is quite verbose, therefore put it in a log file +# and only output its contents on error (see e.g. osmo-ggsn/run.sh) qemu_initrd_exit_error() { - # Building the initrd is quite verbose, therefore put it in a log file - # and only output its contents on error (see e.g. osmo-ggsn/run.sh) - cat "$1" + trap - EXIT INT TERM 0 + cat "build_initrd.log" set +x echo echo "ERROR: failed to build the initrd!" @@ -147,6 +149,18 @@ exit 1 } +# Users of qemu_functions should define a build_initrd function that calls +# qemu_initrd_init, qemu_initrd_add_* and qemu_initrd_finish. See ggsn's run.sh +# as example. The function here redirects the very verbose output of building +# the initrd to a separate log file, and aborts and display the log on error. +qemu_build_initrd_with_log_redirect() { + # Use a trap instead of "if !build_initrd; then ..." logic as "set -e" + # gets otherwise: https://unix.stackexchange.com/a/65564 + trap qemu_initrd_exit_error EXIT INT TERM 0 + build_initrd >build_initrd.log 2>&1 + trap - EXIT INT TERM 0 +} + qemu_random_mac() { printf "52:54:" date "+%c %N" | sha1sum | sed 's/$.\{2\}$/\1:/g' | cut -d: -f 1-4 diff --git a/ggsn_tests/osmo-ggsn/run.sh b/ggsn_tests/osmo-ggsn/run.sh index ae77484..27e633e 100755 --- a/ggsn_tests/osmo-ggsn/run.sh +++ b/ggsn_tests/osmo-ggsn/run.sh @@ -32,9 +32,5 @@ } . qemu_functions.sh - -if ! build_initrd >build_initrd.log 2>&1; then - qemu_initrd_exit_error "build_initrd.log" -fi - +qemu_build_initrd_with_log_redirect qemu_run -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40314?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I2870b7e7ba28b2afc72e86b7a3bc389103564c80 Gerrit-Change-Number: 40314 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: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 month, 1 week

1
0
0 0

[XS] Change in osmo-ttcn3-hacks[master]: testenv: qemu: initrd build fail: stop immediately

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40315?usp=email ) Change subject: testenv: qemu: initrd build fail: stop immediately ...................................................................... testenv: qemu: initrd build fail: stop immediately Let qemu_wait.sh fail immediately when building the initrd has failed, instead of waiting until it runs into a timeout. Change-Id: I6de4d1c31385af52ee0fd55378adb10a9e2c2154 --- M _testenv/data/scripts/qemu/qemu_functions.sh M _testenv/data/scripts/qemu/qemu_wait.sh 2 files changed, 7 insertions(+), 0 deletions(-) Approvals: fixeria: Looks good to me, approved Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, but someone else must approve diff --git a/_testenv/data/scripts/qemu/qemu_functions.sh b/_testenv/data/scripts/qemu/qemu_functions.sh index 46e7273..a984ff9 100755 --- a/_testenv/data/scripts/qemu/qemu_functions.sh +++ b/_testenv/data/scripts/qemu/qemu_functions.sh @@ -146,6 +146,7 @@ echo echo "ERROR: failed to build the initrd!" echo + touch build_initrd_failed exit 1 } diff --git a/_testenv/data/scripts/qemu/qemu_wait.sh b/_testenv/data/scripts/qemu/qemu_wait.sh index fb45c7e..2d42701 100755 --- a/_testenv/data/scripts/qemu/qemu_wait.sh +++ b/_testenv/data/scripts/qemu/qemu_wait.sh @@ -8,6 +8,12 @@ i=0 for i in $(seq 1 600); do sleep 0.1 + + # Check for the marker from qemu_initrd_exit_error + if [ -e build_initrd_failed ]; then + exit 1 + fi + if grep -q KERNEL_TEST_VM_IS_READY "$LOGFILE"; then # Wait some more for SUT to become ready sleep 1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40315?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I6de4d1c31385af52ee0fd55378adb10a9e2c2154 Gerrit-Change-Number: 40315 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: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 month, 1 week

1
0
0 0

[XS] Change in osmo-ttcn3-hacks[master]: testenv: qemu: fix adding kernel modules

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40313?usp=email ) Change subject: testenv: qemu: fix adding kernel modules ...................................................................... testenv: qemu: fix adding kernel modules Without this patch qemu_initrd_add_file fails to add kernel modules with "No such file or directory" when running in podman, even though the files exist: ./testenv.py run ggsn -Dp -c osmo_ggsn_v4_only … + cp -a --parents /lib/modules/6.1.0-34-amd64/kernel/drivers/net/net_failover.ko /tmp/testenv-ggsn_tests-osmo_ggsn_v4_only-20250509-1146-cd82caba-wegi5i98/ggsn/_initrd/usr cp: 'lib/modules/6.1.0-34-amd64/kernel/drivers/net': No such file or directory … When looking at it with strace it became clear that this comes from a getxattr that fails in the podman filesystem and returns a misleading error: getxattr("lib/modules/6.1.0-34-amd64/kernel/drivers/net", "system.posix_acl_access", 0x7ffc012054e0, 132) = -1 ENOENT (No such file or directory) Fix it by modifying the cp command to not try to preserve any attributes (they don't matter in the initramfs). Change-Id: Ic32318da169ee8ebf867937782be5cf8a15f9f2f --- M _testenv/data/scripts/qemu/qemu_functions.sh 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve diff --git a/_testenv/data/scripts/qemu/qemu_functions.sh b/_testenv/data/scripts/qemu/qemu_functions.sh index e2014a1..9c92a37 100755 --- a/_testenv/data/scripts/qemu/qemu_functions.sh +++ b/_testenv/data/scripts/qemu/qemu_functions.sh @@ -8,14 +8,15 @@ # $@: path to files qemu_initrd_add_file() { local i + local cp="cp --no-dereference --recursive --parents" for i in "$@"; do case "$i" in /bin/*|/sbin/*|/lib/*|/lib64/*) - cp -a --parents "$i" "$INITRD_DIR"/usr + $cp "$i" "$INITRD_DIR"/usr ;; *) - cp -a --parents "$i" "$INITRD_DIR" + $cp "$i" "$INITRD_DIR" ;; esac done -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40313?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Ic32318da169ee8ebf867937782be5cf8a15f9f2f Gerrit-Change-Number: 40313 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: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 month, 1 week

1
0
0 0

[S] Change in osmo-ttcn3-hacks[master]: testenv: support qemu=required in testenv.cfg

by osmith

Attention is currently required from: pespin. osmith has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40317?usp=email ) Change subject: testenv: support qemu=required in testenv.cfg ...................................................................... Patch Set 1: Code-Review+2 (2 comments) Patchset: PS1: 1+1=2 File _testenv/README.md: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40317/comment/5e02bba2_8018… : PS1, Line 125: * `qemu=`: set to `optional`/`required` to allow/require running this test > Maybe "yes" instead of "required" would be more usual here. I think "required" is more meaningful than "yes". With "yes" it is not clear if it is optional or required. -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40317?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: If4abdf1f9248fee0915603a9b3c6e3e5e5083057 Gerrit-Change-Number: 40317 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Mon, 19 May 2025 06:39:25 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>

1 month, 1 week

1
0
0 0

[L] Change in osmo-ttcn3-hacks[master]: stp: Split M3UA TCP tests into its own testenv config

by pespin

Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40324?usp=email to look at the new patch set (#2). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: stp: Split M3UA TCP tests into its own testenv config ...................................................................... stp: Split M3UA TCP tests into its own testenv config That group of tests explicitly tests operation of m3ua-tcp AS(P)s and interaction with m3ua(-sctp) ones. Hence, create a new config to test those, where we keep the TCP related configuration in osmo-stp. Change-Id: I8123887755aa3253830b43407b4cc6b21142233c --- M stp/STP_Tests_M3UA.cfg M stp/STP_Tests_M3UA.ttcn A stp/STP_Tests_M3UA_TCP.cfg A stp/STP_Tests_M3UA_TCP.ttcn M stp/expected-results.xml A stp/osmo-stp-m3ua-tcp.confmerge M stp/osmo-stp-m3ua.confmerge A stp/testenv_m3ua-tcp.cfg 8 files changed, 262 insertions(+), 131 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/24/40324/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40324?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I8123887755aa3253830b43407b4cc6b21142233c Gerrit-Change-Number: 40324 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder

1 month, 1 week

1
0
0 0

[L] Change in osmo-ttcn3-hacks[master]: stp: m3ua: Specify ASP names to use per test

by pespin

Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40325?usp=email to look at the new patch set (#2). Change subject: stp: m3ua: Specify ASP names to use per test ...................................................................... stp: m3ua: Specify ASP names to use per test Since we recently introduced an "asp_name" into each m3ua_config, use that to construct a list of m3ua_configs to set up for each test. This makes each test only set up the relevant ASPs for the test easily, plus the big win of port offsets becoming test-specific. That means changing the order or clients or servers in the testsuite config, or adding new ones won't break existing tests. Also add a new "is_server" field which allows specifying clients and servers in whichever order we like. Change-Id: I63ed043014ee8a9a593ed17797b621f39c746bcd --- M stp/STP_Tests_IPA_M3UA.cfg M stp/STP_Tests_IPA_M3UA.ttcn M stp/STP_Tests_M3UA.cfg M stp/STP_Tests_M3UA.ttcn M stp/STP_Tests_M3UA_TCP.cfg M stp/STP_Tests_M3UA_TCP.ttcn 6 files changed, 419 insertions(+), 313 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/25/40325/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40325?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I63ed043014ee8a9a593ed17797b621f39c746bcd Gerrit-Change-Number: 40325 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder

1 month, 1 week

1
0
0 0

[L] Change in osmo-ttcn3-hacks[master]: stp: ipa: Specify ASP names to use per test

by pespin

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40328?usp=email ) Change subject: stp: ipa: Specify ASP names to use per test ...................................................................... stp: ipa: Specify ASP names to use per test Similar to what was done for M3UA ASPs in previous commit. Change-Id: Ifce2a3964d4caf0850691515e516cee93dbe8d44 --- M stp/STP_Tests_IPA.cfg M stp/STP_Tests_IPA.ttcn M stp/STP_Tests_IPA_M3UA.cfg M stp/STP_Tests_IPA_M3UA.ttcn M stp/osmo-stp-ipa.confmerge 5 files changed, 242 insertions(+), 95 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/28/40328/1 diff --git a/stp/STP_Tests_IPA.cfg b/stp/STP_Tests_IPA.cfg index d1ab417..5399093 100644 --- a/stp/STP_Tests_IPA.cfg +++ b/stp/STP_Tests_IPA.cfg @@ -11,16 +11,57 @@ [TESTPORT_PARAMETERS] [MODULE_PARAMETERS] -STP_Tests_IPA.mp_ipa_as_names := { - "ipa-as-loadshare-sender", - "ipa-as-loadshare-sender", - "ipa-as-loadshare-receiver", - "ipa-as-loadshare-receiver", - "ipa-as-dynamic-asp", - "ipa-as-override-sender", - "ipa-as-override-receiver", - "ipa-as-override-receiver" -}; +STP_Tests_IPA.mp_ipa_configs := { + { + asp_name := "ipa-asp-loadshare-sender0", + as_name := "ipa-as-loadshare-sender", + is_server := false, + remote_port := 5000, + local_port := 20000 + }, + { + asp_name := "ipa-asp-loadshare-sender1", + as_name := "ipa-as-loadshare-sender", + is_server := false, + remote_port := 5000, + local_port := 20001 + }, + { + asp_name := "ipa-asp-loadshare-receiver0", + as_name := "ipa-as-loadshare-receiver", + is_server := false, + remote_port := 5000, + local_port := 20002 + }, + { + asp_name := "ipa-asp-loadshare-receiver1", + as_name := "ipa-as-loadshare-receiver", + is_server := false, + remote_port := 5000, + local_port := 20003 + }, + { + asp_name := "ipa-asp-override-sender", + as_name := "ipa-as-override-sender", + is_server := false, + remote_port := 5000, + local_port := 20005 + }, + { + asp_name := "ipa-asp-override-receiver0", + as_name := "ipa-as-override-receiver", + is_server := false, + remote_port := 5000, + local_port := 20006 + }, + { + asp_name := "ipa-asp-override-receiver1", + as_name := "ipa-as-override-receiver", + is_server := false, + remote_port := 5000, + local_port := 20007 + } +} [MAIN_CONTROLLER] diff --git a/stp/STP_Tests_IPA.ttcn b/stp/STP_Tests_IPA.ttcn index 5fb8423..537cac4 100644 --- a/stp/STP_Tests_IPA.ttcn +++ b/stp/STP_Tests_IPA.ttcn @@ -42,10 +42,25 @@ charstring mp_stp_ipa_ip := "127.0.0.1"; charstring mp_local_ipa_ip := "127.0.0.1"; integer mp_stp_ipa_port := 5000; - integer mp_local_ipa_port := 20000; - AspNameArray mp_ipa_as_names := {}; + //integer mp_local_ipa_port := 20000; + /* local ASPs. Set in .cfg file: */ + IpaConfigs mp_ipa_configs := {} } +type record IpaConfig { + /* Name of the ASP in the STP, (eg. to access it over VTY) */ + charstring asp_name, + /* Name of the AS in the STP, (eg. to access it over VTY) */ + charstring as_name, + /* Whether local side is a TCP/SCTP server */ + boolean is_server, + /* STP-side SCTP (or TCP) port for M3UA */ + integer remote_port, + /* local M3UA base port on TTCN3 side */ + integer local_port +}; +type record of IpaConfig IpaConfigs; + type component IPA_CT extends Test_CT { /* for IPA we use the IPA_Emulation and not directly IPA_CodecPort to avoid * having to re-invent IPA CCM handling here */ @@ -53,6 +68,43 @@ port IPA_SP_PT IPA_CTRL[MAX_NR_IPA]; var IPA_Emulation_CT vc_IPA[MAX_NR_IPA]; var IPA_CCM_Parameters g_ccm_pars[MAX_NR_IPA]; + var IpaConfigs g_ipa_configs; +} + +friend function ipa_build_configs(template (omit) Misc_Helpers.ro_charstring asps := omit) +return IpaConfigs +{ + if (not isvalue(asps)) { + return mp_ipa_configs; + } + + var IpaConfigs ipa_configs := {}; + for (var integer i := 0; i < lengthof(asps); i := i + 1) { + var boolean found := false; + for (var integer j := 0; j < lengthof(mp_ipa_configs); j := j + 1) { + if (mp_ipa_configs[j].asp_name == valueof(asps[i])) { + found := true; + ipa_configs := ipa_configs & {mp_ipa_configs[j]}; + break; + } + } + if (not found) { + Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail, + log2str("Could not find ASP ", asps[i], ", check your configuration")); + } + } + return ipa_configs; +} + +private function f_vty_cs7_listen_ipa_cmd(charstring cmd, integer ipa_port := 5000) runs on IPA_CT +{ + f_vty_config2(VTY, {"cs7 instance 0", "listen ipa " & int2str(ipa_port)}, cmd); +} + +friend function f_vty_cs7_ipa_as_cmd(charstring as_name, charstring cmd) runs on IPA_CT +{ + f_vty_config2(VTY, {"cs7 instance 0", "as " & as_name & " ipa"}, cmd); + } friend function f_IPA_send(integer idx, octetstring data) runs on IPA_CT { @@ -89,30 +141,33 @@ f_IPA_exp(idx_rx, data); } -friend function f_init_ipa() runs on IPA_CT { +friend function f_init_ipa(template (omit) IpaConfigs ipa_configs := omit) runs on IPA_CT { var integer i; f_init_common(); - for (i := 0; i < lengthof(mp_ipa_as_names); i:=i+1) { + if (not istemplatekind(ipa_configs, "omit")) { + g_ipa_configs := valueof(ipa_configs); + } else { + g_ipa_configs := mp_ipa_configs; + } + + for (i := 0; i < lengthof(g_ipa_configs); i:=i+1) { vc_IPA[i] := IPA_Emulation_CT.create("IPA" & int2str(i)); map(vc_IPA[i]:IPA_PORT, system:IPA_CODEC_PT); connect(self:IPA[i], vc_IPA[i]:MTP3_SP_PORT); connect(self:IPA_CTRL[i], vc_IPA[i]:IPA_SP_PORT); g_ccm_pars[i] := c_IPA_default_ccm_pars; - g_ccm_pars[i].name := mp_ipa_as_names[i]; + g_ccm_pars[i].name := g_ipa_configs[i].as_name; } } -friend function f_connect_ipa(integer idx, boolean use_unknown_asp_port := false, boolean exp_act := true) runs on IPA_CT { - var integer port_offset := 0; - if (use_unknown_asp_port) { - /* Add 100 to the port since we know that port is not configured in any - ASP only up to MAX_NR_IPA are configured. */ - port_offset := 100; - } - vc_IPA[idx].start(IPA_Emulation.main_client(mp_stp_ipa_ip, mp_stp_ipa_port, mp_local_ipa_ip, - mp_local_ipa_port + idx + port_offset, g_ccm_pars[idx])); +friend function f_connect_ipa(integer idx, boolean exp_act := true) runs on IPA_CT { + vc_IPA[idx].start(IPA_Emulation.main_client(mp_stp_ipa_ip, + g_ipa_configs[idx].remote_port, + mp_local_ipa_ip, + g_ipa_configs[idx].local_port, + g_ccm_pars[idx])); IPA_CTRL[idx].receive(tr_ASP_IPA_EV(ASP_IPA_EVENT_UP)); alt { [exp_act] IPA_CTRL[idx].receive(tr_ASP_IPA_EV(ASP_IPA_EVENT_ID_ACK)); @@ -124,58 +179,87 @@ /* "accept-asp-connections pre-configured" and client from unknown source */ testcase TC_unknown_client_nodynamic() runs on IPA_CT { + var Misc_Helpers.ro_charstring asps := { "ipa-asp-loadshare-sender0" }; + var IpaConfigs ipa_configs := ipa_build_configs(asps); + /* Build an ASP unknown (port) from osmo-stp: */ + ipa_configs := { + { asp_name := "ipa-asp-unknown", + as_name := ipa_configs[0].as_name, + is_server := false, + remote_port := ipa_configs[0].remote_port, + local_port := 3000 } + }; f_init_common(); - f_vty_config2(VTY, {"cs7 instance 0", "listen ipa 5000"}, - "accept-asp-connections pre-configured"); - f_init_ipa(); - f_connect_ipa(0, true, false); + f_vty_cs7_listen_ipa_cmd("accept-asp-connections pre-configured", ipa_configs[0].remote_port); + f_init_ipa(ipa_configs := ipa_configs); + f_connect_ipa(0, false); /* switch back to default */ - f_vty_config2(VTY, {"cs7 instance 0", "listen ipa 5000"}, - "accept-asp-connections dynamic-permitted"); + f_vty_cs7_listen_ipa_cmd("accept-asp-connections dynamic-permitted", ipa_configs[0].remote_port); setverdict(pass); } /* "accept-asp-connections pre-configured" and client from known source */ testcase TC_known_client_nodynamic() runs on IPA_CT { + var Misc_Helpers.ro_charstring asps := { "ipa-asp-loadshare-sender0" }; + var IpaConfigs ipa_configs := ipa_build_configs(asps); + f_init_common(); - f_vty_config2(VTY, {"cs7 instance 0", "listen ipa 5000"}, - "accept-asp-connections pre-configured"); - f_init_ipa(); - f_connect_ipa(0, false); + f_vty_cs7_listen_ipa_cmd("accept-asp-connections pre-configured", ipa_configs[0].remote_port); + f_init_ipa(ipa_configs := ipa_configs); + f_connect_ipa(0); /* switch back to default */ - f_vty_config2(VTY, {"cs7 instance 0", "listen ipa 5000"}, - "accept-asp-connections dynamic-permitted"); + f_vty_cs7_listen_ipa_cmd("accept-asp-connections dynamic-permitted", ipa_configs[0].remote_port); setverdict(pass); } /* "accept-asp-connections dynamic-permitted" and client from unknown source */ testcase TC_unknown_client_dynamic() runs on IPA_CT { + var Misc_Helpers.ro_charstring asps := { "ipa-asp-loadshare-sender0" }; + var IpaConfigs ipa_configs := ipa_build_configs(asps); + /* Build an ASP unknown (port) from osmo-stp: */ + ipa_configs := { + { asp_name := "ipa-asp-unknown2", + as_name := ipa_configs[0].as_name, + is_server := false, + remote_port := ipa_configs[0].remote_port, + local_port := 3001 } + }; + f_init_common(); - f_init_ipa(); - f_connect_ipa(0, true); + f_init_ipa(ipa_configs := ipa_configs); + f_connect_ipa(0); setverdict(pass); } private function f_tc_tmt_override(boolean unknwon_dynamic_asp) runs on IPA_CT { - f_init_ipa(); + var Misc_Helpers.ro_charstring asps := { "ipa-asp-override-sender", + "ipa-asp-override-receiver0", + "ipa-asp-override-receiver1" }; + var IpaConfigs ipa_configs := ipa_build_configs(asps); + if (unknwon_dynamic_asp) { + ipa_configs[0].local_port := 3002; + ipa_configs[1].local_port := 3003; + ipa_configs[2].local_port := 3004; + } + f_init_ipa(ipa_configs := ipa_configs); /* bring up the 'sender' side (single ASP in AS) */ - f_connect_ipa(5, unknwon_dynamic_asp); + f_connect_ipa(0); /* activate the first 'receiver' side ASP */ - f_connect_ipa(6, unknwon_dynamic_asp); + f_connect_ipa(1); /* verify traffic is routed from sender to [sole] receiver */ - f_test_traffic(5, 6); + f_test_traffic(0, 1); /* activate the second 'receiver' side ASP */ - f_connect_ipa(7, unknwon_dynamic_asp); + f_connect_ipa(2); /* verify traffic is routed from sender to new receiver */ - f_test_traffic(5, 7); + f_test_traffic(0, 2); setverdict(pass); } @@ -199,23 +283,32 @@ private function f_tc_tmt_loadshare_roundrobin(boolean unknwon_dynamic_asp) runs on IPA_CT { var integer i; + var Misc_Helpers.ro_charstring asps := { "ipa-asp-loadshare-sender0", + "ipa-asp-loadshare-receiver0", + "ipa-asp-loadshare-receiver1" }; + var IpaConfigs ipa_configs := ipa_build_configs(asps); + if (unknwon_dynamic_asp) { + ipa_configs[0].local_port := 3005; + ipa_configs[1].local_port := 3006; + ipa_configs[2].local_port := 3007; + } - f_init_ipa(); + f_init_ipa(ipa_configs := ipa_configs); - f_vty_config2(VTY, {"cs7 instance 0", "as " & mp_ipa_as_names[2] & " ipa"}, "traffic-mode roundrobin"); + f_vty_cs7_ipa_as_cmd(ipa_configs[1].as_name, "traffic-mode roundrobin"); /* bring up the 'sender' side (single ASP in AS) */ - f_connect_ipa(0, unknwon_dynamic_asp); + f_connect_ipa(0); /* activate the first 'receiver' side ASP */ - f_connect_ipa(2, unknwon_dynamic_asp); + f_connect_ipa(1); /* verify traffic is routed from sender to [sole] receiver */ for (i := 0; i < 10; i := i+1) { - f_test_traffic(0, 2); + f_test_traffic(0, 1); } /* activate the second 'receiver' side ASP */ - f_connect_ipa(3, unknwon_dynamic_asp); + f_connect_ipa(2); /* verify traffic is routed from sender to new receiver */ const integer iter_per_asp := 5; @@ -224,8 +317,8 @@ var octetstring data := f_rnd_octstring(f_rnd_ipa_len()); f_IPA_send(0, data); alt { - [] as_count_rx(2, data, num_rx[0]); - [] as_count_rx(3, data, num_rx[1]); + [] as_count_rx(1, data, num_rx[0]); + [] as_count_rx(2, data, num_rx[1]); } } /* FIXME: check for extraneous messages? */ @@ -237,7 +330,7 @@ } setverdict(pass); - f_vty_config2(VTY, {"cs7 instance 0", "as " & mp_ipa_as_names[2] & " ipa"}, "no traffic-mode"); + f_vty_cs7_ipa_as_cmd(ipa_configs[1].as_name, "no traffic-mode"); } /* test "traffic-mode load-share" behavior */ @@ -259,15 +352,25 @@ runs on IPA_CT { var integer i; - f_init_ipa(); + var Misc_Helpers.ro_charstring asps := { "ipa-asp-loadshare-sender0", + "ipa-asp-loadshare-sender1", + "ipa-asp-loadshare-receiver0", + "ipa-asp-loadshare-receiver1" }; + var IpaConfigs ipa_configs := ipa_build_configs(asps); + if (unknwon_dynamic_asp) { + ipa_configs[0].local_port := 3010; + ipa_configs[1].local_port := 3011; + } - f_vty_config3(VTY, {"cs7 instance 0", "as " & mp_ipa_as_names[2] & " ipa"}, + f_init_ipa(ipa_configs := ipa_configs); + + f_vty_config3(VTY, {"cs7 instance 0", "as " & ipa_configs[2].as_name & " ipa"}, { "traffic-mode loadshare", "binding-table reset" }); /* bring up the 'sender' side (single ASP in AS) */ - f_connect_ipa(0, unknwon_dynamic_asp); + f_connect_ipa(0); /* activate the first 'receiver' side ASP */ - f_connect_ipa(2, unknwon_dynamic_asp); + f_connect_ipa(2); /* verify traffic is routed from sender to [sole] receiver */ for (i := 0; i < 10; i := i+1) { @@ -275,11 +378,11 @@ } /* activate the second 'receiver' side ASP */ - f_connect_ipa(3, unknwon_dynamic_asp); + f_connect_ipa(3); /* Since we are using dynamic ASPs, they were unknown by STP until they got connected. * Hence, reset the binding-table so that load is now properly spread among them: */ - f_vty_config2(VTY, {"cs7 instance 0", "as " & mp_ipa_as_names[2] & " ipa"}, "binding-table reset" ); + f_vty_cs7_ipa_as_cmd(ipa_configs[2].as_name, "binding-table reset"); /* verify traffic is routed from sender to new receiver */ const integer iter_per_asp := 20; @@ -313,7 +416,7 @@ /* activate the second 'sender' side ASP */ - f_connect_ipa(1, unknwon_dynamic_asp); + f_connect_ipa(1); num_rx_1 := 0; @@ -345,7 +448,7 @@ } setverdict(pass); - f_vty_config2(VTY, {"cs7 instance 0", "as " & mp_ipa_as_names[2] & " ipa"}, "no traffic-mode"); + f_vty_cs7_ipa_as_cmd(ipa_configs[2].as_name, "no traffic-mode"); } /* test "traffic-mode load-share" behavior */ diff --git a/stp/STP_Tests_IPA_M3UA.cfg b/stp/STP_Tests_IPA_M3UA.cfg index 4ca349d..7849d3a 100644 --- a/stp/STP_Tests_IPA_M3UA.cfg +++ b/stp/STP_Tests_IPA_M3UA.cfg @@ -11,21 +11,27 @@ [TESTPORT_PARAMETERS] [MODULE_PARAMETERS] -STP_Tests_IPA.mp_ipa_as_names := { - "ipa-as-dynamic-asp" +STP_Tests_IPA.mp_ipa_configs := { + { + asp_name := "ipa-asp-dynamic-unknown", + as_name := "ipa-as-dynamic-asp", + is_server := false, + remote_port := 5000, + local_port := 40000 + } }; STP_Tests_M3UA.mp_m3ua_configs := { - { - asp_name := "asp-sender", - is_server := false, - use_tcp := false, - remote_port := 2905, - local_port := 9999, - point_code := 23, - routing_ctx := 1023 - } - }; + { + asp_name := "asp-sender", + is_server := false, + use_tcp := false, + remote_port := 2905, + local_port := 9999, + point_code := 23, + routing_ctx := 1023 + } +}; [MAIN_CONTROLLER] diff --git a/stp/STP_Tests_IPA_M3UA.ttcn b/stp/STP_Tests_IPA_M3UA.ttcn index f62fc7c..5600f6e 100644 --- a/stp/STP_Tests_IPA_M3UA.ttcn +++ b/stp/STP_Tests_IPA_M3UA.ttcn @@ -63,8 +63,8 @@ /* Test routing of SCCP between an M3UA and an IPA ASP */ testcase TC_m3ua_to_ipa() runs on IPA_M3UA_CT { - var Misc_Helpers.ro_charstring asps := { "asp-sender" }; - var M3uaConfigs m3ua_configs := m3ua_build_configs(asps); + var Misc_Helpers.ro_charstring m3ua_asps := { "asp-sender" }; + var M3uaConfigs m3ua_configs := m3ua_build_configs(m3ua_asps); var OCT4 rctx_sender := int2oct(m3ua_configs[0].routing_ctx, 4); var OCT4 pc_sender := int2oct(m3ua_configs[0].point_code, 4); var OCT4 pc_receiver := int2oct(5, 4); @@ -93,18 +93,19 @@ /* test routing an SCCP message from IPA ASP to M3UA ASP */ testcase TC_ipa_to_m3ua() runs on IPA_M3UA_CT { - var Misc_Helpers.ro_charstring asps := { "asp-sender" }; - var M3uaConfigs m3ua_configs := m3ua_build_configs(asps); + var Misc_Helpers.ro_charstring ipa_asps := { "ipa-asp-dynamic-unknown" }; + var IpaConfigs ipa_configs := ipa_build_configs(ipa_asps); + var Misc_Helpers.ro_charstring m3ua_asps := { "asp-sender" }; + var M3uaConfigs m3ua_configs := m3ua_build_configs(m3ua_asps); var OCT4 pc_sender := int2oct(5, 4); var OCT4 rctx_receiver := int2oct(m3ua_configs[0].routing_ctx, 4); var OCT4 pc_receiver := int2oct(m3ua_configs[0].point_code, 4); f_init_common(); - f_vty_config2(VTY, {"cs7 instance 0", "as ipa-as-dynamic-asp ipa"}, - "point-code override patch-sccp disabled"); + f_vty_cs7_ipa_as_cmd(ipa_configs[0].as_name, "point-code override patch-sccp disabled"); f_init_m3ua(m3ua_configs := m3ua_configs); - f_init_ipa(); + f_init_ipa(ipa_configs := ipa_configs); f_connect_ipa(0); f_M3UA_asp_up_act(0, omit, rctx_receiver); @@ -127,18 +128,19 @@ /* test routing an SCCP message from IPA ASP to M3UA ASP using national network indicator */ testcase TC_ipa_to_m3ua_ni() runs on IPA_M3UA_CT { - var Misc_Helpers.ro_charstring asps := { "asp-sender" }; - var M3uaConfigs m3ua_configs := m3ua_build_configs(asps); + var Misc_Helpers.ro_charstring ipa_asps := { "ipa-asp-dynamic-unknown" }; + var IpaConfigs ipa_configs := ipa_build_configs(ipa_asps); + var Misc_Helpers.ro_charstring m3ua_asps := { "asp-sender" }; + var M3uaConfigs m3ua_configs := m3ua_build_configs(m3ua_asps); var OCT4 pc_sender := int2oct(5, 4); var OCT4 rctx_receiver := int2oct(m3ua_configs[0].routing_ctx, 4); var OCT4 pc_receiver := int2oct(m3ua_configs[0].point_code, 4); f_init_common(); - f_vty_config2(VTY, {"cs7 instance 0", "as ipa-as-dynamic-asp ipa"}, - "point-code override patch-sccp disabled"); + f_vty_cs7_ipa_as_cmd(ipa_configs[0].as_name, "point-code override patch-sccp disabled"); f_init_m3ua(m3ua_configs := m3ua_configs); - f_init_ipa(); + f_init_ipa(ipa_configs := ipa_configs); f_connect_ipa(0); f_M3UA_asp_up_act(0, omit, rctx_receiver); @@ -164,15 +166,16 @@ /* test routing an SCCP message from IPA ASP to M3UA ASP while patching PC into SCCP addresses */ testcase TC_ipa_to_m3ua_patch_sccp() runs on IPA_M3UA_CT { - var Misc_Helpers.ro_charstring asps := { "asp-sender" }; - var M3uaConfigs m3ua_configs := m3ua_build_configs(asps); + var Misc_Helpers.ro_charstring ipa_asps := { "ipa-asp-dynamic-unknown" }; + var IpaConfigs ipa_configs := ipa_build_configs(ipa_asps); + var Misc_Helpers.ro_charstring m3ua_asps := { "asp-sender" }; + var M3uaConfigs m3ua_configs := m3ua_build_configs(m3ua_asps); var OCT4 pc_sender := int2oct(5, 4); var OCT4 rctx_receiver := int2oct(m3ua_configs[0].routing_ctx, 4); var OCT4 pc_receiver := int2oct(m3ua_configs[0].point_code, 4); f_init_common(); - f_vty_config2(VTY, {"cs7 instance 0", "as ipa-as-dynamic-asp ipa"}, - "point-code override patch-sccp both"); + f_vty_cs7_ipa_as_cmd(ipa_configs[0].as_name, "point-code override patch-sccp both"); f_init_m3ua(m3ua_configs := m3ua_configs); f_init_ipa(); diff --git a/stp/osmo-stp-ipa.confmerge b/stp/osmo-stp-ipa.confmerge index 712e2a9..500f5ac 100644 --- a/stp/osmo-stp-ipa.confmerge +++ b/stp/osmo-stp-ipa.confmerge @@ -29,7 +29,6 @@ point-code override dpc 32 asp ipa-asp-loadshare-receiver0 asp ipa-asp-loadshare-receiver1 - ! Port 2004 is reserved for use with ipa-as-dynamic-asp in TTCN-3. ! OVERRIDE: asp ipa-asp-override-sender 20005 5000 ipa local-ip 127.0.0.1 @@ -54,17 +53,12 @@ point-code override dpc 34 asp ipa-asp-override-receiver0 asp ipa-asp-override-receiver1 - ! DYNAMIC AS: - as ipa-as-dynamic-asp ipa - routing-key 0 5 - point-code override dpc 23 ! ! ROUTING TABLE: ! route-table system update route 31 16777215 linkset ipa-as-loadshare-receiver update route 33 16777215 linkset ipa-as-override-receiver - update route 5 16777215 linkset ipa-as-dynamic-asp ! !SERVERS: ! -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40328?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Ifce2a3964d4caf0850691515e516cee93dbe8d44 Gerrit-Change-Number: 40328 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de>

1 month, 1 week

1
0
0 0

[S] Change in libosmo-sigtran[master]: asp: Avoid double-free of received msg if conn is teared down

by pespin

Attention is currently required from: daniel, fixeria, laforge, osmith. pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/40327?usp=email ) Change subject: asp: Avoid double-free of received msg if conn is teared down ...................................................................... Patch Set 2: (1 comment) Patchset: PS2: We may need something similar with the client side, eg. if it receives some weird stuff during IPA handshake which makes it terminate the dynamic asp which in turn terminates the stream_cli? -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40327?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: I69f80f611c14db2b328dafd4a90247c6f2dac6fd Gerrit-Change-Number: 40327 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> 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: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Comment-Date: Fri, 16 May 2025 19:44:05 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 month, 1 week

1
0
0 0

[S] Change in libosmo-sigtran[master]: asp: Avoid double-free of received msg if conn is teared down

by pespin

Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40327?usp=email to look at the new patch set (#2). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: asp: Avoid double-free of received msg if conn is teared down ...................................................................... asp: Avoid double-free of received msg if conn is teared down """ 20250516192255921 DLSS7 DEBUG IPA_ASP(ipa-asp-loadshare-sender0){WAIT_ID_RESP}: Received Event IPA_CCM_ID_RESP (ipa.c:120) 20250516192255921 DLMI DEBUG Rx IPA CCM ID_RESP: Unit_ID='0/1/2' MAC_Address='' Location_1='' Location_2='' Equipment_Version='' Software_Version='' Unit_Name='mahlzeit' Serial_Number='' (ipa.c:233) 20250516192255921 DLSS7 NOTICE IPA_ASP(ipa-asp-loadshare-sender0){WAIT_ID_RESP}: Cannot find any definition for IPA Unit Name 'mahlzeit' (xua_asp_fsm.c:968) 20250516192255921 DLSS7 INFO ipa-asp-loadshare-sender0: connection closed (ss7_asp.c:1159) 20250516192255921 DLSS7 DEBUG IPA_ASP(ipa-asp-loadshare-sender0){WAIT_ID_RESP}: Received Event SCTP-COMM_DOWN.ind (ss7_asp.c:1165) 20250516192255922 DLSS7 DEBUG IPA_ASP(ipa-asp-loadshare-sender0){WAIT_ID_RESP}: state_chg to ASP_DOWN (xua_asp_fsm.c:1154) 20250516192255922 DLSS7 DEBUG XUA_AS(ipa-as-loadshare-sender){AS_DOWN}: Received Event ASPAS-ASP_DOWN.ind (xua_asp_fsm.c:370) 20250516192255922 DLSS7 DEBUG IPA_ASP(ipa-asp-loadshare-sender0){ASP_DOWN}: No Layer Manager, dropping M-ASP_DOWN.indication (xua_asp_fsm.c:119) 20250516192255922 DLSS7 DEBUG IPA_ASP(ipa-asp-loadshare-sender0){ASP_DOWN}: No Layer Manager, dropping M-SCTP_RELEASE.indication (xua_asp_fsm.c:119) Program terminated with signal SIGABRT, Aborted. #0 0x000076bb9898ceec in ?? () from /lib/x86_64-linux-gnu/libc.so.6 #0 0x000076bb9898ceec in ?? () from /lib/x86_64-linux-gnu/libc.so.6 #1 0x000076bb9893dfb2 in raise () from /lib/x86_64-linux-gnu/libc.so.6 #2 0x000076bb98928472 in abort () from /lib/x86_64-linux-gnu/libc.so.6 #3 0x000076bb98ae6496 in ?? () from /lib/x86_64-linux-gnu/libtalloc.so.2 #4 0x000076bb98b1b869 in msgb_free (m=0x5f957de3e750) at ../../../src_copy/libosmocore/src/core/msgb.c:119 #5 0x000076bb98bab8c8 in ipa_rx_msg_ccm (asp=0x5f957de3da50, msg=0x5f957de3e750) at ../../src_copy/libosmo-sigtran/src/ipa.c:137 #6 0x000076bb98bac135 in ipa_rx_msg (asp=0x5f957de3da50, msg=0x5f957de3e750, sls=0 '\000') at ../../src_copy/libosmo-sigtran/src/ipa.c:321 #7 0x000076bb98bca44f in ss7_asp_ipa_srv_conn_rx_cb (conn=0x5f957ddba4a0, res=49, msg=0x5f957de3e750) at ../../src_copy/libosmo-sigtran/src/ss7_asp.c:895 #8 0x000076bb988efcb1 in stream_srv_iofd_read_cb (iofd=0x5f957ddd8e40, res=49, msg=0x5f957de3e750) at ../../src_copy/libosmo-netif/src/stream_srv.c:732 #9 0x000076bb98b23c3c in iofd_handle_segmented_read (iofd=0x5f957ddd8e40, msg=0x5f957de3e750, rc=49) at ../../../src_copy/libosmocore/src/core/osmo_io.c:357 #10 0x000076bb98b23d2b in iofd_handle_recv (iofd=0x5f957ddd8e40, msg=0x5f957de3e750, rc=49, hdr=0x0) at ../../../src_copy/libosmocore/src/core/osmo_io.c:384 #11 0x000076bb98b257b7 in iofd_poll_ofd_cb_recvmsg_sendmsg (ofd=0x5f957ddd8ef0, what=1) at ../../../src_copy/libosmocore/src/core/osmo_io_poll.c:64 #12 0x000076bb98b25b32 in iofd_poll_ofd_cb_dispatch (ofd=0x5f957ddd8ef0, what=1) at ../../../src_copy/libosmocore/src/core/osmo_io_poll.c:136 #13 0x000076bb98b2907b in poll_disp_fds (n_fd=6) at ../../../src_copy/libosmocore/src/core/select.c:419 #14 0x000076bb98b29191 in _osmo_select_main (polling=0) at ../../../src_copy/libosmocore/src/core/select.c:457 #15 0x000076bb98b291ac in osmo_select_main (polling=0) at ../../../src_copy/libosmocore/src/core/select.c:496 #16 0x00005f9553dd9a21 in main (argc=3, argv=0x7ffe754fac38) at ../../src_copy/libosmo-sigtran/stp/stp_main.c:270 """ Change-Id: I69f80f611c14db2b328dafd4a90247c6f2dac6fd --- M src/ss7_asp.c 1 file changed, 21 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran refs/changes/27/40327/2 -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40327?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: I69f80f611c14db2b328dafd4a90247c6f2dac6fd Gerrit-Change-Number: 40327 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder

1 month, 1 week

1
0
0 0

2025

2024

2023

2022

gerrit-log May 2025