From 43595df9777e01172d79b4d7c4d76a63b3eb36c6 Mon Sep 17 00:00:00 2001 From: "Florian Aders (EleRas)" Date: Mon, 5 Apr 2010 13:03:58 +0000 Subject: [PATCH] Deactivating limitation of commands in safe_exec, we'll just check if there are redirects and stuff like that The original reason for safe_exec is already allowed: rm --- lib/functions/filedir/function.safe_exec.php | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/functions/filedir/function.safe_exec.php b/lib/functions/filedir/function.safe_exec.php index 6a93c827..46b9f7f4 100644 --- a/lib/functions/filedir/function.safe_exec.php +++ b/lib/functions/filedir/function.safe_exec.php @@ -76,6 +76,8 @@ function safe_exec($exec_string, &$return_value = false) die('SECURITY CHECK FAILED!' . "\n" . 'The execute string "' . htmlspecialchars($exec_string) . '" is a possible security risk!' . "\n" . 'Please check your whole server for security problems by hand!' . "\n"); } + /* + * This is not needed anymore, we allow all commands and just check for pipes and stuff // // check if command is allowed here // @@ -94,6 +96,7 @@ function safe_exec($exec_string, &$return_value = false) { die('SECURITY CHECK FAILED!' . "\n" . 'Your command "' . htmlspecialchars($exec_string) . '" is not allowed!' . "\n" . 'Please check your whole server for security problems by hand!' . "\n"); } + */ // // execute the command and return output