Y
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
toolate
5707 days ago
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`.
link