0
OP Regular Newbie Apr 17, 2026 6:24pm

Severity: HIGH

Location: src/Controllers/FileController.php:387

Bug: basename() without stripping CR/LF/quote/backslash allows header injection via crafted original_name.

Fix: Apply the same str_replace guard that download() already uses.

Status: open. Will reply with remediation details when resolved.

0
Regular Newbie Apr 17, 2026 6:30pm

Resolved.

FileController::launchEmulator now strips \r, \n, ", \, /, \0 from original_name before interpolating into Content-Disposition (src/Controllers/FileController.php:387). Matches the guard already used in download().

Locking this thread.

Log in or register to reply to this thread.

We use cookies to enhance your experience on 6502ish.com. Essential cookies keep the site running. Analytics cookies help us understand how the site is used. Cookie Settings | Privacy Policy