Skip to content

fix(runtime): contain translated PS2 paths - #238

Closed
hedgeg0d wants to merge 2 commits into
ran-j:mainfrom
hedgeg0d:fix/runtime-path-containment
Closed

fix(runtime): contain translated PS2 paths#238
hedgeg0d wants to merge 2 commits into
ran-j:mainfrom
hedgeg0d:fix/runtime-path-containment

Conversation

@hedgeg0d

Copy link
Copy Markdown

PS2 device paths were normalized but not checked against their configured roots. Parent traversal and pre-existing symlinks could resolve outside hostRoot, cdRoot, or mcRoot.

  • Canonicalize configured roots and translated paths before accepting them
  • Compare existing roots by filesystem identity
  • Restrict direct Windows paths to cdRoot
  • Add translation and fioOpen regression tests

Tested with ps2x_tests: 428 passed. The runtime changes from #235 were applied only for the Debug test run because current main aborts in PS2SifDma without that fix.

@hedgeg0d

Copy link
Copy Markdown
Author

Replaced by #239. This branch included an unrelated commit from the fork's main branch.

@hedgeg0d hedgeg0d closed this Aug 30, 2026
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