Skip to content

fix(mcp): stop splicing notification text into AppleScript/PowerShell source - #140

Closed
carfeii wants to merge 1 commit into
google:mainfrom
carfeii:fix/desktop-notifier-applescript-powershell-injection
Closed

carfeii wants to merge 1 commit into
google:mainfrom
carfeii:fix/desktop-notifier-applescript-powershell-injection

Conversation

@carfeii

@carfeii carfeii commented Sep 21, 2026

Copy link
Copy Markdown

… source

DesktopNotifier.notify() built the macOS and Windows toast commands by
f-string-interpolating the notification title/body directly into an
AppleScript `display notification "..."` literal and a PowerShell
CreateTextNode("...") literal, then ran them via osascript/powershell
with no escaping. A quote or newline in that text lets it break out of
the string literal and run as its own AppleScript/PowerShell statement.

The notified text (task goal + task result) can contain content the
agent observed while automating an app or web page, so an attacker who
controls what's on the screen during a task controls this text. Desktop
notifications are enabled by default (no opt-in env var required), so
this reaches every default install on macOS and Windows.

Fix: pass the title/body through the environment instead of
interpolating them into the script source, and read them back with
`system attribute` (AppleScript) / `$env:` (PowerShell). The script
text itself is now a fixed constant that never varies with input.
@google-cla

google-cla Bot commented Sep 21, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@carfeii carfeii closed this Sep 21, 2026
@carfeii
carfeii deleted the fix/desktop-notifier-applescript-powershell-injection branch September 21, 2026 16:01
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