T09 · Insecure Skill Coding Practices
- Location
SKILL.md:126- Finding
Active Scanning Is Not Constrained by a Programmatic Scope Allowlist
- Content
View full analysis
/dev/null done < "$TARGETS_FILE" # Nmap — vulnerability scripts echo "[2/4] Nmap vuln scripts..." while IFS= read -r target; do [ -z "$target" ] && continue nmap --script vuln "$target" -oN "$OUTDIR/nmap-vuln-$target.txt" 2>/dev/null done < "$TARGETS_FILE" ``` The security guidance at lines 563–564 states: ```text - **Scope whitelist is absolute.** Never scan outside program scope. - **Rate limiting mandatory.** Max 2-5 req/s per target. ``` ### Technical Analysis The pipeline immediately performs full-port service detection and vulnerability-script scanning against every entry in a caller-provided target file. The stated scope whitelist is only documentation: the implementation does not load a machine-readable bounty scope, verify domain ownership, validate approved CIDR ranges, reject excluded assets, or require authorization before active scanning. The orchestrator also invokes additional active scanners against caller-provided targets and URLs. Consequently, a malformed, stale, or malicious target list can cause the system to scan unrelated infrastructure. The Nmap option `--min-rate 1000` requests a minimum packet rate far above the documented mandatory limit of two to five requests per second. This increases the likelihood of service disruption, defensive alerts, IP blocking, and violation of bug-bounty rules. ### Attack Path 1. An operator, automation source, or malicious contributor adds an unauthorized hostname or IP address to `targets.txt`. 2. The operator starts the Hunter-Killer orchestrator. 3. Tier 2 reads the entry without checking it ...[truncated 1104 chars]- Remediation
View remediation
