Trust Assessment
Docker Pro Diagnostic received a trust score of 27/100, placing it in the Untrusted category. This skill has significant security findings that require attention before use in production.
SkillShield's automated analysis identified 3 findings: 2 critical, 1 high, 0 medium, and 0 low severity. Key findings include Arbitrary command execution, Dangerous call: subprocess.run(), Command Injection via unsanitized user input in shell=True subprocess call.
The analysis covered 4 layers: Manifest Analysis, Static Code Analysis, Dependency Graph, LLM Behavioral Safety. All layers scored 70 or above, reflecting consistent security practices.
Last analyzed on February 12, 2026 (commit 13146e6a). SkillShield performs automated 4-layer security analysis on AI skills and MCP servers.
Layer Breakdown
Behavioral Risk Signals
Security Findings3
| Severity | Finding | Layer | Location | |
|---|---|---|---|---|
| CRITICAL | Arbitrary command execution Python shell execution (os.system, subprocess) Review all shell execution calls. Ensure commands are static (not built from user input), use absolute paths, and are strictly necessary. Prefer library APIs over shell commands. | Manifest | skills/mkrdiop/docker-diag/log_processor.py:14 | |
| CRITICAL | Command Injection via unsanitized user input in shell=True subprocess call The `log_processor.py` script constructs a shell command using an f-string with the user-provided `container_name` and executes it with `subprocess.run(..., shell=True)`. This allows an attacker to inject arbitrary shell commands by crafting the `container_name` argument. For example, providing `mycontainer; rm -rf /` as the container name could lead to arbitrary code execution on the host system where the skill is run. Avoid `shell=True` when executing external commands with user-controlled input. Instead, pass the command and its arguments as a list to `subprocess.run()`. For example, `subprocess.run(['docker', 'logs', '--tail', str(max_lines), container_name], capture_output=True, text=True)`. Additionally, strictly validate or sanitize the `container_name` argument using a whitelist regex (e.g., `^[a-zA-Z0-9_.-]+$`) before use. | LLM | log_processor.py:16 | |
| HIGH | Dangerous call: subprocess.run() Call to 'subprocess.run()' detected in function 'process_logs'. This can execute arbitrary code. Avoid using dangerous functions like exec/eval/os.system. Use safer alternatives. | Static | skills/mkrdiop/docker-diag/log_processor.py:14 |
Scan History
Embed Code
[](https://skillshield.io/report/0be6f6da7e1f95a4)
Powered by SkillShield