Skip to content

fix(security): deny raw factotum grants - #462

Merged
pdfinn merged 1 commit into
masterfrom
audit/factotum-path-deny
Aug 4, 2026
Merged

fix(security): deny raw factotum grants#462
pdfinn merged 1 commit into
masterfrom
audit/factotum-path-deny

Conversation

@pdfinn

@pdfinn pdfinn commented Aug 4, 2026

Copy link
Copy Markdown
Member

Summary

  • Reject caller-supplied /mnt/factotum path grants in nsconstruct, tools9p, and nsaudit.
  • Preserve the existing trusted namespace-construction path that grants factotum only to fixed credential-using tools without exec/shell.
  • Add regression coverage for live bindpath, startup -p, nsaudit rule corpus, namespace manifest invariants, and direct nsconstruct policy.

Security rationale

Factotum is the credential boundary. It is intentionally auto-mounted only for selected fixed-function tools and withheld when arbitrary exec/shell is present. A raw /mnt/factotum path grant bypasses that mediation and can hand the credential service to arbitrary tools, so it should be treated as a privileged control-capability root rather than a generic filesystem path.

Tests

  • tests/host/namespace_path_policy_test.sh
  • tests/host/namespace_manifest_invariants_test.sh
  • tests/host/nsaudit_rules_test.sh
  • tests/host/tools9p_integration_test.sh
  • git diff --check

@pdfinn
pdfinn force-pushed the audit/factotum-path-deny branch from 1f8467d to c7e6998 Compare August 4, 2026 06:03
@pdfinn
pdfinn merged commit 32c06f9 into master Aug 4, 2026
22 checks passed
@pdfinn
pdfinn deleted the audit/factotum-path-deny branch August 4, 2026 07:11
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