Trust Assessment
player received a trust score of 10/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 7 findings: 2 critical, 4 high, 1 medium, and 0 low severity. Key findings include Arbitrary command execution, Missing required field: name, Dangerous call: subprocess.Popen().
The analysis covered 4 layers: Manifest Analysis, Static Code Analysis, Dependency Graph, LLM Behavioral Safety. The Manifest Analysis layer scored lowest at 40/100, indicating areas for improvement.
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 Findings7
| 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/samuelwhg/player/music_control.py:129 | |
| 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/samuelwhg/player/simple_music_control.py:106 | |
| HIGH | Dangerous call: subprocess.Popen() Call to 'subprocess.Popen()' detected in function 'play_track'. This can execute arbitrary code. Avoid using dangerous functions like exec/eval/os.system. Use safer alternatives. | Static | skills/samuelwhg/player/music_control.py:129 | |
| HIGH | Dangerous call: subprocess.Popen() Call to 'subprocess.Popen()' detected in function 'play_track'. This can execute arbitrary code. Avoid using dangerous functions like exec/eval/os.system. Use safer alternatives. | Static | skills/samuelwhg/player/simple_music_control.py:106 | |
| HIGH | Path Traversal in play_directory allows listing/accessing arbitrary files The `play_directory` function constructs a target directory path using `self.music_dir / dir_name`. If `dir_name` (which is user-controlled) contains path traversal sequences (e.g., `../../`), the `target_dir` can resolve to directories outside the intended `F:\Music` path. This allows the skill to list files (via `target_dir.iterdir()`) and potentially attempt to play them (via `play_track`) from arbitrary locations on the `F:` drive, leading to excessive file system access and potential data exfiltration. Sanitize `dir_name` to prevent path traversal. After constructing `target_dir = self.music_dir / dir_name`, resolve the path and verify that it remains a child of `self.music_dir.resolve()`. For example: ```python resolved_target_dir = (self.music_dir / dir_name).resolve() if not resolved_target_dir.is_relative_to(self.music_dir.resolve()): print(f"Invalid directory path: {dir_name}") return False target_dir = resolved_target_dir ``` Apply the same fix to `simple_music_control.py`. | LLM | music_control.py:137 | |
| HIGH | Path Traversal in play_directory allows listing/accessing arbitrary files The `play_directory` function constructs a target directory path using `self.music_dir / dir_name`. If `dir_name` (which is user-controlled) contains path traversal sequences (e.g., `../../`), the `target_dir` can resolve to directories outside the intended `F:\Music` path. This allows the skill to list files (via `target_dir.iterdir()`) and potentially attempt to play them (via `play_track`) from arbitrary locations on the `F:` drive, leading to excessive file system access and potential data exfiltration. Sanitize `dir_name` to prevent path traversal. After constructing `target_dir = self.music_dir / dir_name`, resolve the path and verify that it remains a child of `self.music_dir.resolve()`. For example: ```python resolved_target_dir = (self.music_dir / dir_name).resolve() if not resolved_target_dir.is_relative_to(self.music_dir.resolve()): print(f"Invalid directory path: {dir_name}") return False target_dir = resolved_target_dir ``` Apply the same fix to `music_control.py`. | LLM | simple_music_control.py:139 | |
| MEDIUM | Missing required field: name The 'name' field is required for claude_code skills but is missing from frontmatter. Add a 'name' field to the SKILL.md frontmatter. | Static | skills/samuelwhg/player/SKILL.md:1 |
Scan History
Embed Code
[](https://skillshield.io/report/007dcdd17df51e9b)
Powered by SkillShield