Skip to content

Harden secure path validation - #17

Merged
ConnorNelson merged 1 commit into
mainfrom
codex/secure-path-writable-dirs
Jun 11, 2026
Merged

Harden secure path validation#17
ConnorNelson merged 1 commit into
mainfrom
codex/secure-path-writable-dirs

Conversation

@ConnorNelson

Copy link
Copy Markdown
Member

Summary

  • require the target script to be executable before parsing or invoking it, including direct /usr/bin/exec-suid /path/to/script usage
  • reject path components that are world-writable without the sticky bit
  • report more specific secure-path and nosuid mount errors
  • add secure-path regression tests through the shared run_program fixture

Fixes #15.

Validation

  • docker build -t exec-suid-secure-path /tmp/exec-suid-pr >/tmp/exec-suid-secure-path-build.log && docker run --rm exec-suid-secure-path
  • git diff --check

@ConnorNelson
ConnorNelson marked this pull request as ready for review June 11, 2026 20:48
@ConnorNelson ConnorNelson changed the title [codex] Harden secure path validation Harden secure path validation Jun 11, 2026
@ConnorNelson
ConnorNelson merged commit 1f03c7c into main Jun 11, 2026
1 check passed
@ConnorNelson
ConnorNelson deleted the codex/secure-path-writable-dirs branch June 11, 2026 20:48
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.

Investigate direct exec-suid invocation and script execute permissions

1 participant