Commits May 2026

commits@lists.kronosnet.org

2 participants
1102 discussions

[jenkins] anvil-build-functional-testing (build 563) success
by jenkins＠kronosnet.org 21 May '26

21 May '26

Run reason: Started by upstream project "anvil/anvil-pipeline/main" build number 292 Total runtime: 2 hr 44 min Full log: https://ci.kronosnet.org/job/anvil/job/anvil-build-functional-testing/563/c…

1 0

[kronosnet/kronosnet] 18b371: [tests] fix compress2 pointer type for ARM compati...
by Fabio M. Di Nitto 21 May '26

21 May '26

Branch: refs/heads/misc-bugfixes Home: https://github.com/kronosnet/kronosnet Commit: 18b371c7a23f40dc4fd6ce1498f3b2d4028d1f93 https://github.com/kronosnet/kronosnet/commit/18b371c7a23f40dc4fd6ce1498f3b… Author: Fabio M. Di Nitto <fdinitto(a)redhat.com> Date: 2026-05-21 (Thu, 21 May 2026) Changed paths: M libknet/tests/int_decompress_bufsize.c Log Message: ----------- [tests] fix compress2 pointer type for ARM compatibility Change compressed_size from size_t to uLongf to match compress2() API. On ARM platforms, size_t and uLongf are incompatible types. Fixes: d88d3151 ("libknet: add decompression buffer size validation") Signed-off-by: Fabio M. Di Nitto <fdinitto(a)redhat.com> Co-Authored-By: Claude Sonnet 4.5 <noreply(a)anthropic.com> To unsubscribe from these emails, change your notification settings at https://github.com/kronosnet/kronosnet/settings/notifications

1 0

[jenkins] kronosnet PR-486 (build 1) completed with state: failure
by jenkins＠kronosnet.org 21 May '26

21 May '26

8/86 Non-voting fails: freebsd-devel-x86-64(nonvoting: Run tests) freebsd-devel-x86-64(nonvoting_debug: Run tests) freebsd-devel-x86-64(nonvoting-clang: Run tests) freebsd-15-x86-64(nonvoting-clang: Run tests) freebsd-devel-x86-64(nonvoting-clang_debug: Run tests) alpine-x86-64(nonvoting-clang: Build source) freebsd-15-x86-64(nonvoting-clang_debug: Run tests) alpine-x86-64(nonvoting-clang_debug: Build source) 3/79 Voting fails: debian-unstable-cross-x86-64(arm_crosscompile: Build source) freebsd-15-x86-64(voting: Run tests) freebsd-15-x86-64(voting_debug: Run tests) Run reason: Pull request #486 opened Total runtime: 45 min Split logs: https://ci.kronosnet.org/job/kronosnet/job/kronosnet-pipeline/job/PR-486/1/… Full log: https://ci.kronosnet.org/job/kronosnet/job/kronosnet-pipeline/job/PR-486/1/… Full Coverity results: http://ci.kronosnet.org/coverity/kronosnet/rhel-8-coverity-x86-64/pr486//1 http://ci.kronosnet.org/coverity/kronosnet/rhel-10-coverity-x86-64/pr486//1 http://ci.kronosnet.org/coverity/kronosnet/rhel-9-coverity-x86-64/pr486//1

1 0

[kronosnet/kronosnet] fc73bf: [tests] improve input validation in knet_bench tes...
by Fabio M. Di Nitto 21 May '26

21 May '26

Branch: refs/heads/misc-bugfixes Home: https://github.com/kronosnet/kronosnet Commit: fc73bf1486d5ee87d6715efff9ffd24d6d91cf80 https://github.com/kronosnet/kronosnet/commit/fc73bf1486d5ee87d6715efff9ffd… Author: Fabio M. Di Nitto <fdinitto(a)redhat.com> Date: 2026-05-20 (Wed, 20 May 2026) Changed paths: M libknet/tests/knet_bench.c Log Message: ----------- [tests] improve input validation in knet_bench test tool Replace atoi() with safe_atoi() helper that uses strtol() for proper error detection. Improves developer experience with better error messages. Signed-off-by: Fabio M. Di Nitto <fdinitto(a)redhat.com> Co-Authored-By: Claude Sonnet 4.5 <noreply(a)anthropic.com> Commit: bd35ffa29ff175a1490758a5776a32b953ecb6b5 https://github.com/kronosnet/kronosnet/commit/bd35ffa29ff175a1490758a5776a3… Author: Fabio M. Di Nitto <fdinitto(a)redhat.com> Date: 2026-05-20 (Wed, 20 May 2026) Changed paths: M libnozzle/libnozzle.c M libnozzle/tests/Makefile.am A libnozzle/tests/api_nozzle_prefix_validation.c Log Message: ----------- libnozzle: add input validation for network prefixes Replace atoi() with strtol() for network prefix validation to properly detect invalid input, overflow, and out-of-range values. Add comprehensive test coverage for prefix validation scenarios. Signed-off-by: Fabio M. Di Nitto <fabbione(a)kronosnet.org> Co-Authored-By: Claude Sonnet 4.5 <noreply(a)anthropic.com> Commit: d88d31511801c8a47bf16a7ad9b965ff1077c635 https://github.com/kronosnet/kronosnet/commit/d88d31511801c8a47bf16a7ad9b96… Author: Fabio M. Di Nitto <fdinitto(a)redhat.com> Date: 2026-05-21 (Thu, 21 May 2026) Changed paths: M libknet/tests/Makefile.am A libknet/tests/int_decompress_bufsize.c M libknet/tests/test-common.c M libknet/tests/test-common.h M libknet/threads_rx.c Log Message: ----------- libknet: add decompression buffer size validation Add sanity check to verify decompressed data size does not exceed KNET_DATABUFSIZE before using it. Signed-off-by: Fabio M. Di Nitto <fdinitto(a)redhat.com> Co-Authored-By: Claude Sonnet 4.5 <noreply(a)anthropic.com> Commit: 38b5845af9b54045b370aacbe92ca36d0b94d942 https://github.com/kronosnet/kronosnet/commit/38b5845af9b54045b370aacbe92ca… Author: Fabio M. Di Nitto <fdinitto(a)redhat.com> Date: 2026-05-21 (Thu, 21 May 2026) Changed paths: M libknet/host.c Log Message: ----------- libknet: fix defragmentation buffer reclamation using wrong sequence number The defragmentation buffer reclamation was using the old received sequence number (dst_seq_num) instead of the current packet's sequence number to calculate the valid buffer window. This caused buffers to be reclaimed based on stale sequence information, potentially freeing buffers that should still be valid or keeping buffers that should be reclaimed. Fixed by passing seq_num (current packet) instead of dst_seq_num (last received) to _reclaim_old_defrag_bufs(). Signed-off-by: Fabio M. Di Nitto <fdinitto(a)redhat.com> Co-Authored-By: Claude Sonnet 4.5 <noreply(a)anthropic.com> Commit: d1e0b9bf9fd7265f23aaff30e6615b4458a028b4 https://github.com/kronosnet/kronosnet/commit/d1e0b9bf9fd7265f23aaff30e6615… Author: Fabio M. Di Nitto <fdinitto(a)redhat.com> Date: 2026-05-21 (Thu, 21 May 2026) Changed paths: M libknet/host.c Log Message: ----------- libknet: fix sequence number wraparound calculation in defragmentation The distance calculation between sequence numbers was incorrect when wraparound occurred. The formula was backwards: it subtracted in the wrong direction and didn't account for the +1 needed for modular arithmetic. This caused incorrect buffer reclamation decisions near the SEQ_MAX boundary. Practical examples (SEQ_MAX = 65535): Example 1 - Normal case (no wraparound): Last packet: dst_seq_num = 1000 New packet: seq_num = 1005 Expected distance: 5 BEFORE (wrong): seq_dist = dst_seq_num - seq_num = 1000 - 1005 = -5 (negative!) AFTER (correct): seq_dist = seq_num - dst_seq_num = 1005 - 1000 = 5 Example 2 - Wraparound case: Last packet: dst_seq_num = 65534 New packet: seq_num = 3 (wrapped around) Expected distance: 5 (65534→65535→0→1→2→3) BEFORE (wrong): seq_dist = (SEQ_MAX - seq_num) + dst_seq_num seq_dist = (65535 - 3) + 65534 = 131066 (huge wrong number!) AFTER (correct): seq_dist = (SEQ_MAX - dst_seq_num) + seq_num + 1 seq_dist = (65535 - 65534) + 3 + 1 = 5 The +1 accounts for the transition from 65535→0 being one step, not zero. Verification that circular buffer cleaning is not broken: The seq_dist value is used to determine whether the new packet is: a) Within the circular buffer window (seq_dist < KNET_CBUFFER_SIZE) b) Far enough to require full buffer clear (seq_dist > threshold) c) Should trigger incremental cleaning (fall through case) Test case 1 - Normal sequential packet: dst_seq_num = 1000, seq_num = 1005, expected distance = 5 BEFORE: seq_dist = -5 (unsigned overflow ~65530) → Incorrectly clears entire buffer for normal sequential packets! AFTER: seq_dist = 5 → Correctly identifies packet as within buffer window, no clearing needed Test case 2 - Wraparound (close distance): dst_seq_num = 65534, seq_num = 3, expected distance = 5 BEFORE: seq_dist = 131066 → Falls through to circular buffer cleaning code incorrectly AFTER: seq_dist = 5 → Correctly identifies packet as within buffer window Test case 3 - Large jump requiring buffer clear: dst_seq_num = 1000, seq_num = 50000, expected distance = 49000 BEFORE: seq_dist = -49000 (unsigned ~16536) → Clears buffer (correct by accident) AFTER: seq_dist = 49000 → Clears buffer (correct by design) The circular buffer cleaning code (lines 673-684) uses seq_num and dst_seq_num directly via modulo operations to find buffer positions. It does not use seq_dist for position calculations, only for the threshold check to determine whether to run. The fix corrects the threshold logic without affecting the position calculations. Signed-off-by: Fabio M. Di Nitto <fdinitto(a)redhat.com> Co-Authored-By: Claude Sonnet 4.5 <noreply(a)anthropic.com> Commit: a465037f8c87dffe0630b7bd2065db8caee5a5e9 https://github.com/kronosnet/kronosnet/commit/a465037f8c87dffe0630b7bd2065d… Author: Fabio M. Di Nitto <fdinitto(a)redhat.com> Date: 2026-05-21 (Thu, 21 May 2026) Changed paths: M libknet/tests/Makefile.am A libknet/tests/int_seq_wraparound_stress.c Log Message: ----------- [tests] add comprehensive sequence number wraparound stress test Add int_seq_wraparound_stress_test to verify correct handling of sequence number wraparound, packet loss, and out-of-order delivery in defragmentation. Test scenarios: 1. Normal sequential with 30% packet loss - verifies basic packet loss handling 2. Wraparound boundary (65533-65535, 0-4) - tests wraparound at SEQ_MAX with loss 3. Large sequence jump (> KNET_CBUFFER_SIZE) - tests jumps exceeding buffer size 4. Out-of-order fragment delivery - tests fragments arriving in reverse order 5. Out-of-order packet delivery - tests complete packets arriving backwards (1000→1001→999) 6. Extreme loss beyond window - 5000 packet gap exceeding KNET_CBUFFER_SIZE (4096) 7. Wraparound with extreme loss - 5636 packet gap across wraparound boundary 8. Multiple wraparound cycles - rapid cycling with different loss patterns, verifies duplicate detection (17 unique packets delivered, 5 duplicates rejected) Tests use inject_packet() to simulate realistic network conditions including packet loss, reordering, incomplete transmissions, and insane scenarios that verify robustness even under conditions that shouldn't occur in practice. Signed-off-by: Fabio M. Di Nitto <fabbione(a)kronosnet.org> Co-Authored-By: Claude Sonnet 4.5 <noreply(a)anthropic.com> Compare: https://github.com/kronosnet/kronosnet/compare/fc73bf1486d5%5E...a465037f8c… To unsubscribe from these emails, change your notification settings at https://github.com/kronosnet/kronosnet/settings/notifications

1 0

[jenkins] anvil PR-1001 (build 1) success
by jenkins＠kronosnet.org 21 May '26

21 May '26

1 0

[jenkins] anvil-build-functional-testing (build 554) success
by jenkins＠kronosnet.org 21 May '26

21 May '26

Run reason: Started by upstream project "anvil/anvil-pipeline/PR-1001" build number 1 Total runtime: 5 hr 35 min Full log: https://ci.kronosnet.org/job/anvil/job/anvil-build-functional-testing/554/c…

1 0

[jenkins] resource-agents PR-2162 (build 1) success
by jenkins＠kronosnet.org 21 May '26

21 May '26

Run reason: Pull request #2162 opened Total runtime: 11 min Split logs: https://ci.kronosnet.org/job/resource-agents/job/resource-agents-pipeline/j… Full log: https://ci.kronosnet.org/job/resource-agents/job/resource-agents-pipeline/j…

1 0

[jenkins] anvil-build-functional-testing (build 553) success
by jenkins＠kronosnet.org 21 May '26

21 May '26

Run reason: Started by upstream project "anvil/anvil-pipeline/PR-1001" build number 1 Total runtime: 3 hr 12 min Full log: https://ci.kronosnet.org/job/anvil/job/anvil-build-functional-testing/553/c…

1 0

[jenkins] anvil-build-functional-testing (build 558) success
by jenkins＠kronosnet.org 21 May '26

21 May '26

Run reason: Started by upstream project "anvil/anvil-pipeline/PR-1001" build number 1 Total runtime: 2 hr 56 min Full log: https://ci.kronosnet.org/job/anvil/job/anvil-build-functional-testing/558/c…

1 0

[jenkins] anvil-build-functional-testing (build 556) success
by jenkins＠kronosnet.org 21 May '26

21 May '26

Run reason: Started by upstream project "anvil/anvil-pipeline/PR-1001" build number 1 Total runtime: 2 hr 53 min Full log: https://ci.kronosnet.org/job/anvil/job/anvil-build-functional-testing/556/c…

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

Commits May 2026