| name | file-handling-business-logic |
| description | Reviews file upload and handling for path traversal, type validation, storage, and related logic flaws. Use when auditing file upload or processing security. |
| license | CC0-1.0 |
| metadata | {"category":"security"} |
| allowed-tools | Read Grep Glob Write |
| disable-model-invocation | true |
| argument-hint | [path or scope] |
Target: $ARGUMENTS
If no target path is given above, review the entire codebase.
Review all file upload functionality:
Check for:
- File type validation (whitelist, not blacklist)
- File size limits
- Filename sanitization
- Anti-virus scanning integration
- Storage location (outside webroot)
- Direct execution prevention
- MIME type validation
- Magic number verification
- Image manipulation library vulnerabilities
- ZIP bomb protection
Provide recommendations for secure file handling.
Provide:
A structured finding report with the following for each issue:
Title, Severity (Critical/High/Medium/Low), CWE (if applicable), Evidence (file, function, line ranges), and a short Why it matters.
Exploitability notes and, where safe, a minimal PoC or reproduction steps (no real secrets).
Remediation: precise code-level fix or config change (snippets welcome), plus defense-in-depth guidance.
A summary risk score (0–10) and top 3–5 prioritized fixes that reduce risk fastest.
A checklist diff: which items from the “Check for” list are Pass/Fail/Not Applicable.
Constraints & style:
Be concrete and cite exact code locations and identifiers.
Prefer minimal, drop-in fix snippets over prose.
Do not invent files or functions that aren’t present; if context is missing, mark as Unable to verify and say what code would prove it.
Write this into a markdown file and place it in the audits/ folder.