Skip to content

Fix RPC response validation and topic reallocation churn#63

Merged
pavel-kirienko merged 2 commits into
mainfrom
dev
Jul 5, 2026
Merged

Fix RPC response validation and topic reallocation churn#63
pavel-kirienko merged 2 commits into
mainfrom
dev

Conversation

@pavel-kirienko

Copy link
Copy Markdown
Member

Summary

  • Harden cy_respond() argument validation so malformed NULL-data/nonzero-size payloads and invalid breadcrumb priorities are rejected before platform unicast, without advancing the response sequence number.
  • Preserve topic reader/writer handles when topic_allocate() changes eviction count but maps back to the same subject-ID, while leaving subject moves, displacement handling, gossip scheduling, and diagnostics semantics intact.
  • Add intrusive regressions for both changes and move the L12/L17 TODO records into todo/resolved/.

Branch commits

  • 62eea1d Fix cy_respond argument validation
  • 0d29e96 Fix same-subject topic reallocation churn

Verification

  • cmake --build build-l17 --parallel --target test_intrusive_topic_allocation_x64_c11 test_intrusive_topic_allocate_exhaustive_x64_c11
  • ctest --test-dir build-l17 -R 'run_test_intrusive_topic_(allocation|allocate_exhaustive)_x64_c11' --output-on-failure
  • cmake --build build-l17 --parallel
  • ctest --test-dir build-l17 --output-on-failure (105/105 passed)
  • git diff --check origin/main..dev
  • C comment-line delta check: added_comment_lines=9 removed_comment_lines=11
  • Review loop: 3 consecutive clean rounds, including Claude Fable 5/xhigh correctness review, no major/significant findings.

@pavel-kirienko
pavel-kirienko merged commit 14af5d3 into main Jul 5, 2026
8 checks passed
@pavel-kirienko
pavel-kirienko deleted the dev branch July 5, 2026 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant