-
Notifications
You must be signed in to change notification settings - Fork 153
Open
Description
Summary
The application has no authentication layer — all inventory data is accessible to anyone. Combined with SQL injection via phpGrid and reflected XSS.
Key Findings (8 total, 5 CRITICAL + 3 HIGH)
- No authentication — no session validation on any page (products, purchases, orders, reports, barcodes)
- Reflected XSS — unescaped $_GET['currentPage'] in inc/menu.php
- SQL injection — unparameterized queries via phpGrid C_DataGrid edit/filter handlers
- IDOR — no ownership verification on any resource modification
- No CSRF protection — all forms lack token validation
- Hardcoded database credentials, debug error display, unvalidated barcode input
Fix
- Add session-based authentication to all pages
- Use htmlspecialchars() on all user input in HTML output
- Use parameterized queries / prepared statements
- Add CSRF token generation and validation
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels