T09 · Insecure Skill Coding Practices
Note
- Location
scripts/bnb_band_bot.py:46- Finding
Unnecessary Private-Key Loading in Placeholder and Dry-Run Modes
- Content
View full analysis
- Remediation
View remediation
Security audit
EVM BNB Band Trader
Security checks for vulnerabilities and agentic risk
Overview
This trading skill is not malicious, but it asks for a wallet private key even though the included bot only logs placeholder trades and lacks some documented safety checks.
Review this before installing if you would use a real wallet. Treat any private key placed in EVM_PRIVATE_KEY as highly sensitive, prefer a test or low-value wallet, and do not fund it until the placeholder trading, gas/balance checks, transaction signing path, limits, and confirmation behavior are clearly implemented and audited.
Vulnerability Patterns
- Insecure Skill Coding PracticesFinds exploitable flaws such as hardcoded secrets or command injection
- Skill Instruction HijackingAlters the agent's session goals or safety constraints when the skill loads
- Agent Memory PoisoningWrites attacker-controlled rules into memory that affect later sessions
- Remote Payload Retrieval and ExecutionFetches external code whose behavior can change after review
- Embedded Malicious CodeShips malicious scripts inside the skill and executes them locally
Findings (1)
Vulnerability Patterns
- Data ExfiltrationExternal Transmission, Env Variable Harvesting, File System Enumeration
- Prompt InjectionInstruction Override, Hidden Instructions, Exfiltration Commands
- Privilege EscalationExcessive Permissions, Sudo/Root Execution, Credential Access
- Supply ChainUnpinned Dependencies, External Script Fetching, Obfuscated Code
- Excessive AgencyUnrestricted Tool Access, Autonomous Decision Making, Scope Creep
Findings (2)
Missing User Warnings
Medium
- Category
- Not specified by scanner
- Confidence
- 87% confidence
- Finding
The bot is structured to perform live trading when invoked with --mode run, but it provides no explicit confirmation prompt, prominent warning, or protective guardrails before entering a potentially irreversible trading loop. In an agent-skill context, this increases the chance of accidental fund-moving behavior if a user or orchestrator triggers live mode without understanding the financial risk.
- Content
Missing User Warnings
Medium
- Category
- Not specified by scanner
- Confidence
- 93% confidence
- Finding
The code reads EVM_PRIVATE_KEY and BNB_RPC_URL from environment variables, which involves access to sensitive credentials and network configuration. Although the script logs general startup information, it does not provide any explicit user-facing warning, comment, or docstring disclosing that it will consume a blockchain private key for trading operations.
- Content