[CRITICAL] Command injection in terminal_tool.py (CVSS 9.8) #46

Closed
opened 2026-03-30 23:09:52 +00:00 by allegro · 1 comment
Member

Finding from Master Security Audit\n\nMultiple locations use shell=True with unsanitized input.\n\nAffected: terminal_tool.py, environments/.py\n\nFix: Use shell=False with argument lists\n\nPart of 8-subagent systematic analysis*

**Finding from Master Security Audit**\n\nMultiple locations use shell=True with unsanitized input.\n\n**Affected:** terminal_tool.py, environments/*.py\n\n**Fix:** Use shell=False with argument lists\n\n*Part of 8-subagent systematic analysis*
Author
Member

FIXED: Command injection fixed in PR #53

✅ **FIXED**: Command injection fixed in PR #53
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Timmy_Foundation/hermes-agent#46