Skip to main content
← Back to list
01Issue
BugShippedExtensionsPublic
Assigneesstack72

Relationships

↔ sibling #3719↔ sibling #3720

#3716 @swamp/cve/dirtyfrag mitigate uses interactive sudo, so it can hang on a password prompt or fail mid-run

Opened by skunk-ape · 10/21/2026· Shipped 10/22/2026

cve/dirtyfrag/extensions/models/dirtyfrag_detect.ts mitigate checks whoami and prefixes plain 'sudo ' (no -n) to sh -c wrapped commands. When sudo needs a password: over SSH there is no tty, so each command fails, the exit code is only logged, and mitigate still reports the host as mitigated if dryRun=false. On the local transport sudo opens /dev/tty directly, so the run can block on a prompt swamp never shows (from reading the code; not reproduced). The sh -c wrapper also defeats sudoers rules scoped to specific binaries. The @swamp/ssh sudo contract (sudo -n --, no sh -c for plain commands, ssh/README.md sudo section) is the better model; #3711 adopts it for the PHP extension. Expected: preflight sudo -n true, fail fast with a clear needs-privilege result, and never mark a host mitigated when a command failed.

02Bog Flow
✓OPEN✓TRIAGED✓IN PROGRESS✓SHIPPED+ 3 MOREISSUE_LINKED+ 5 MOREREVIEW+ 14 MOREPR_MERGED+ 2 MORESESSION_SUMMARIZED

Shipped

10/22/2026, 1:23:26 AM

Click a lifecycle step above to view its details.

03Sludge Pulse
stack72 assigned stack7210/21/2026, 11:45:48 PM
stack72 linked sibling of #371910/21/2026, 11:46:14 PM
stack72 linked sibling of #372010/21/2026, 11:46:18 PM
Editable. Press Enter to edit.

system commented 10/21/2026, 3:09:29 PM

Classified automatically when this issue was filed.

  • Source: Extensions

If you feel this classification is incorrect, add a ripple to tell us so.

Sign in to post a ripple.