Hacker News new | ask | show | jobs
by eneko 5707 days ago
Nice. The code was already checking for '..' on the path, but the condition was erroneous. Fixed now.
1 comments

You might be better off getting the canonical path and then checking against a whitelist. E.g. `strpos(realpath($command_path), '/var/www/html/') === 0`.