Repository navigation
ci: merge train for 4 reviewed PRs (CI batch, do not merge) - #2561
Closed
oneandonlydean wants to merge 9 commits into
Closed
oneandonlydean wants to merge 9 commits into
oneandonlydean wants to merge 9 commits into
Conversation
…lumeMatrix Pan follows the PS4 firmware 11.00 libSceNgs2.sprx: the speaker layout from the init angles, then each param's angle and distance into a volume matrix for the voice. Every param is checked before anything is written, and angles too large to wrap throw. The pan functions are exported from the existing Voice.cpp export block. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
flock mapped each call straight onto LockFileEx/UnlockFileEx, whose byte-range locks stack per handle. LOCK_UN without a lock failed with ERROR_NOT_LOCKED and aborted the process, a repeated LOCK_EX blocked forever, converting between shared and exclusive failed, and a repeated LOCK_SH needed as many unlocks. Track the one lock each descriptor holds: a repeated mode and LOCK_UN without a lock succeed, a conversion releases the old lock before taking the new one, and close forgets the descriptor's lock.
FreeBSD sys_flock takes LOCK_UN over LOCK_EX over LOCK_SH, ignores other bits and fails with EBADF when none of the three is set. The operation reached the host unchanged, so malformed operations differed between hosts: Linux rejects them with EINVAL and the Windows lock tracking threw. Decode the operation before either host sees it.
The CFG/GOT filter's entry point collector checked header and symbol positions as `offset + size > file size`, which wraps for offsets near 2^64. A section header or symbol table offset there passed the check, read bytes before the input buffer, and could reach an in-bounds table through the wrapped address, adding entry points that keep relocations alive. Check every position by subtraction instead. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 10, 2026
|
Other open pull requests touch the same code. Merging one will break the other, or both implement the same thing. If this one builds on them, list them in Depends on.
|
9 tasks done
Collaborator
Author
|
Train build 38028193639 was green; the listed pull requests were merged individually. Closing. |
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
5 participants
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.
CI batch for the pull requests below, merged onto main 0011248 in this order. Do not merge this PR. It exists only so one full build tests them together; each PR is merged on its own after this build is green, and this PR is then closed.