Validate combined fixes for issues 237–249 - #263
Draft
drebbe-intrepid wants to merge 34 commits into
Draft
drebbe-intrepid wants to merge 34 commits into
drebbe-intrepid wants to merge 34 commits into
Conversation
Reject already-open devices before a second native open. Correct capsule update status and release newly opened handles if Python assignment fails.
Reject unsupported objects before reading native message fields or transmitting any part of a batch. Preserve both supported message layouts and cover validation with a hardware-free native stub.
Track payload capacity independently of mutable wire lengths to prevent heap overreads. Validate length changes, reads, and native writes; preserve DLL-owned buffers and handle reentrant integer conversions safely. Fixes #240
…ce-equality', 'codex/fix-237-message-deletion', 'codex/fix-238-radio-keywords', 'codex/fix-239-transmit-types', 'codex/fix-241-reflash-callback' and 'codex/fix-244-settings-override' into codex/integration-237-249
…ace-build' into codex/integration-237-249
Write normalized native scalar fields directly and dispatch deletion before snapshots. Restore state before constructing errors so callback-driven payload replacement cannot resurrect freed pointers.
drebbe-intrepid
changed the title
Shared None counts include device handle leaks and interpreter bookkeeping. Measure private rejected inputs, batch containers, and valid neighbors over repeated calls while retaining the full type-rejection matrix.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Validates the combined fixes from #250–#262 before their individual merges. This draft is an integration check; the individual PRs remain the merge units.
All13 issue builders and their critics agreed on final changes. An independent aggregate critic agreed with commit 24fa8ec, including the handle-ownership and payload-getter conflict resolutions.
Validation:
Awaiting all relevant current-head CI. No fixes have been merged yet.