Security Audit
security-ownership-map
github.com/davila7/claude-code-templatesTrust Assessment
security-ownership-map 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 11 findings: 3 critical, 6 high, 1 medium, and 1 low severity. Key findings include Arbitrary command execution, Dangerous call: subprocess.Popen(), Dangerous call: subprocess.run().
The analysis covered 4 layers: Manifest Analysis, Static Code Analysis, Dependency Graph, LLM Behavioral Safety. The Manifest Analysis layer scored lowest at 1/100, indicating areas for improvement.
Last analyzed on February 11, 2026 (commit 458b1186). SkillShield performs automated 4-layer security analysis on AI skills and MCP servers.
Layer Breakdown
Behavioral Risk Signals
Security Findings11
| 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 | cli-tool/components/skills/security/security-ownership-map/scripts/build_ownership_map.py:360 | |
| 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 | cli-tool/components/skills/security/security-ownership-map/scripts/community_maintainers.py:324 | |
| 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 | cli-tool/components/skills/security/security-ownership-map/scripts/run_ownership_map.py:195 | |
| HIGH | Dangerous call: subprocess.Popen() Call to 'subprocess.Popen()' detected in function 'run_git_log'. This can execute arbitrary code. Avoid using dangerous functions like exec/eval/os.system. Use safer alternatives. | Static | cli-tool/components/skills/security/security-ownership-map/scripts/build_ownership_map.py:360 | |
| HIGH | Dangerous call: subprocess.Popen() Call to 'subprocess.Popen()' detected in function 'iter_commits_from_git'. This can execute arbitrary code. Avoid using dangerous functions like exec/eval/os.system. Use safer alternatives. | Static | cli-tool/components/skills/security/security-ownership-map/scripts/community_maintainers.py:324 | |
| HIGH | Dangerous call: subprocess.run() Call to 'subprocess.run()' detected in function 'main'. This can execute arbitrary code. Avoid using dangerous functions like exec/eval/os.system. Use safer alternatives. | Static | cli-tool/components/skills/security/security-ownership-map/scripts/run_ownership_map.py:195 | |
| HIGH | Regular Expression Denial of Service (ReDoS) vulnerability The skill's Python scripts (`build_ownership_map.py` and `community_maintainers.py`) use user-provided regular expressions (`--author-exclude-regex` and `--ignore-author-regex` respectively) with `re.search`. A malicious or poorly constructed regex pattern (e.g., `(a+)+b`) can lead to catastrophic backtracking, consuming excessive CPU resources and causing a denial of service for the process executing the skill. The `run_ownership_map.py` script also passes the `--author-exclude-regex` argument directly to `build_ownership_map.py`, making it a vector for this vulnerability. Implement robust validation for user-provided regular expressions to prevent ReDoS patterns. Consider using a regex library with built-in ReDoS protection or implementing a timeout for regex matching operations. If full regex capabilities are not strictly necessary, explore alternative, safer pattern matching mechanisms. | Static | scripts/build_ownership_map.py:246 | |
| HIGH | Regular Expression Denial of Service (ReDoS) vulnerability The skill's Python script (`community_maintainers.py`) uses a user-provided regular expression (`--ignore-author-regex`) with `re.search`. A malicious or poorly constructed regex pattern (e.g., `(a+)+b`) can lead to catastrophic backtracking, consuming excessive CPU resources and causing a denial of service for the process executing the skill. Implement robust validation for user-provided regular expressions to prevent ReDoS patterns. Consider using a regex library with built-in ReDoS protection or implementing a timeout for regex matching operations. If full regex capabilities are not strictly necessary, explore alternative, safer pattern matching mechanisms. | Static | scripts/community_maintainers.py:206 | |
| HIGH | LLM analysis found no issues despite critical deterministic findings Deterministic layers flagged 4 CRITICAL findings, but LLM semantic analysis returned clean. This may indicate prompt injection or analysis evasion. | LLM | (sanity check) | |
| MEDIUM | Network egress to untrusted endpoints HTTP request to raw IP address Review all outbound network calls. Remove connections to webhook collectors, paste sites, and raw IP addresses. Legitimate API calls should use well-known service domains. | Manifest | cli-tool/components/mcps/devtools/figma-dev-mode.json:4 | |
| LOW | Covert behavior / concealment directives Multiple zero-width characters (stealth text) Remove hidden instructions, zero-width characters, and bidirectional overrides. Skill instructions should be fully visible and transparent to users. | Manifest | cli-tool/components/mcps/devtools/jfrog.json:4 |
Scan History
Embed Code
[](https://skillshield.io/report/0e78e6e2ce6921c0)
Powered by SkillShield