Preflight bounded Mach-O chained fixups - #85
Merged
Conversation
|
You have reached your Codex usage limits for security reviews. Please try again later. |
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
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.
Purpose
Prevent file-backed chained-fixup metadata from terminating the raw helper on valid sparse iOS 27 tables or malformed external ranges.
Fixes #80.
Changes
e0e0b30187ae74f2088d932845c8ddac2c79f36c.The dependency fix adds bounded file/image table parsing, correct sparse segment semantics, checked page and chain walking, one validated pointer-index owner, and typed preflight failures while preserving existing public query signatures.
Testing
swift package --force-resolved-versions resolveswift package --force-resolved-versions show-dependencies --format flatlist(no MachOKit identity conflict)swift test --force-resolved-versionsscripts/test-release-scripts.shPrivateHeaderKitCore,PrivateHeaderKitCoreTestsPrivateHeaderKitCore,PrivateHeaderKitCoreTests,privateheaderkit-sim-helperswift build; 28DyldChainedFixupsSafetyTests; branch-wide Codex review clean24A5390fruntime smoke:PosterBoardUI: completed, 6 artifactsPrivateSearchProtocols: completed, 1 artifactUserManagementUI: completed, 3 artifactsok; runtime override restored; run-owned Simulator deletedScreenshots
Not applicable.