Skip to content

Host: harden input admission across every target argument - #255

Merged
iamnbutler merged 1 commit into
mainfrom
cu-admission-targets
Nov 14, 2026
Merged

iamnbutler merged 1 commit into
mainfrom
cu-admission-targets

Conversation

@iamnbutler

Copy link
Copy Markdown
Contributor

Hardening for the host's input admission.

  • Admission resolves every argument that names a target window (snapshot, to_snapshot, the snapshot in a desktop_notification_action id) plus any explicit pid / target.pid, and runs the own-UI and System Settings checks on each.
  • A pid or target.pid that disagrees with the call's snapshot is refused (invalidRequest), since dispatch acts on the snapshot.
  • AdmissionTargetTests calls admission with each argument shape.

swift test --package-path native2 passes (all suites).

🤖 Generated with Claude Code

Admission now resolves every argument that names a target window (snapshot, to_snapshot, a notification id's
snapshot) and an explicit pid, checks each against the own-UI and System Settings rules, and refuses a pid or
target.pid that disagrees with the snapshot's.

Why: native2/Sources/DesktopHost/Admission.swift | admission must check every snapshot dispatch can act on (snapshot, to_snapshot, the notification id's prefix); add any new snapshot-carrying argument to targetProcesses

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@iamnbutler
iamnbutler merged commit 2335fc1 into main Nov 14, 2026
2 checks passed
@iamnbutler
iamnbutler deleted the cu-admission-targets branch November 14, 2026 12:30
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