эксплоит в моем php-скрипте

Question

У меня есть знаменитый скрипт, установленный на моем сервере программист сказал мне, что у меня есть код внутри одной из страниц, который можно использовать для выполнения любого кода или функции, и grep все данные моего ящика

код

function fileRequestHandler($handler, $module = false, $file = false){ 
global $test_conf; 

switch ($handler) { 
    case 'reload': 
// AJAX handler for reload event 
     $response = do_reload(); 
     header("Content-type: application/json"); 
     echo json_encode($response); 
     break; 
    case 'file': 
     /** Handler to pass-through file requests 
     * Looks for "module" and "file" variables, strips .. and only allows normal filename characters. 
     * Accepts only files of the type listed in $allowed_exts below, and sends the corresponding mime-type, 
     * and always interprets files through the PHP interpreter. (Most of?) the environment is available, 
     * including $db and $astman, and the user is authenticated. 
     */ 
     if (!$module || !$file) { 
      die_myscript("unknown"); 
     } 
//TODO: this could probably be more efficient 
     $module = str_replace('..', '.', preg_replace('/[^a-zA-Z0-9-\_\.]/', '', $module)); 
     $file = str_replace('..', '.', preg_replace('/[^a-zA-Z0-9-\_\.]/', '', $file)); 

     $allowed_exts = array(
      '.js' => 'text/javascript', 
      '.js.php' => 'text/javascript', 
      '.css' => 'text/css', 
      '.css.php' => 'text/css', 
      '.html.php' => 'text/html', 
      '.php' => 'text/html', 
      '.jpg.php' => 'image/jpeg', 
      '.jpeg.php' => 'image/jpeg', 
      '.png.php' => 'image/png', 
      '.gif.php' => 'image/gif', 
     ); 

что проблема кода здесь ?? , какой тип действия может быть передан через этот код?

как я мог закрыть его благодаря

Answer 1

Простой google search узнал меня, что вы ищете, где этот код:

http://cxsecurity.com/issue/WLB-2014020088

function fileRequestHandler($handler, $module = false, $file = false) 
{ 
    global $amp_conf; 

    switch ($handler) { 
     case 'reload': 
// AJAX handler for reload event 
      $response = do_reload(); 
      header("Content-type: application/json"); 
      echo json_encode($response); 
      break; 
     case 'file': 
      /** Handler to pass-through file requests 
      * Looks for "module" and "file" variables, strips .. and only allows normal filename 
      * characters. 
      * Accepts only files of the type listed in $allowed_exts below, and sends the corresponding mime-type, 
      * and always interprets files through the PHP interpreter. (Most of?) the freepbx environment is available, 
      * including $db and $astman, and the user is authenticated. 
      */ 
      if (!$module || !$file) { 
       die_freepbx("unknown"); 
      } 
//TODO: this could probably be more efficient 
      $module = str_replace('..', '.', preg_replace('/[^a-zA-Z0-9-\_\.]/', '', $module)); 
      $file = str_replace('..', '.', preg_replace('/[^a-zA-Z0-9-\_\.]/', '', $file)); 

      $allowed_exts = array(
       '.js' => 'text/javascript', 
       '.js.php' => 'text/javascript', 
       '.css' => 'text/css', 
       '.css.php' => 'text/css', 
       '.html.php' => 'text/html', 
       '.php' => 'text/html', 
       '.jpg.php' => 'image/jpeg', 
       '.jpeg.php' => 'image/jpeg', 
       '.png.php' => 'image/png', 
       '.gif.php' => 'image/gif', 
      ); 
      foreach ($allowed_exts as $ext => $mimetype) { 
       if (substr($file, -1 * strlen($ext)) == $ext) { 
        $fullpath = 'modules/' . $module . '/' . $file; 
        if (file_exists($fullpath)) { 
// file exists, and is allowed extension 

// image, css, js types - set Expires to 24hrs in advance so the client does 
// not keep checking for them. Replace from header.php 
         if (!$amp_conf['DEVEL']) { 
          header('Expires: ' . gmdate('D, d M Y H:i:s', time() + 86400) . ' GMT', true); 
          header('Cache-Control: max-age=86400, public, must-revalidate', true); 
         } 
         header("Content-type: " . $mimetype); 
         ob_start(); 
         include($fullpath); 
         ob_end_flush(); 
         exit(); 
        } 
        break; 
       } 
      } 
      die_freepbx("../view/not allowed"); 
      break; 
     case 'api': 
      if (isset($_REQUEST['function']) && function_exists($_REQUEST['function'])) { 
       $function = $_REQUEST['function']; 
       $args = isset($_REQUEST['args']) ? $_REQUEST['args'] : ''; 

//currently works for one arg functions, eventually need to clean this up to except more args 
       $result = $function($args); 
       $jr = json_encode($result); 
      } else { 
       $jr = json_encode(null); 
      } 
      header("Content-type: application/json"); 
      echo $jr; 
      break; 
    } 
    exit(); 
} 

//Function is called at admin/config . php at line 132 

if (!in_array($display, array('noauth', 'badrefer')) 
    && isset($_REQUEST['handler']) 
) { 
    $module = isset($_REQUEST['module']) ? $_REQUEST['module'] : ''; 
    $file = isset($_REQUEST['file']) ? $_REQUEST['file'] : ''; 
    fileRequestHandler($_REQUEST['handler'], $module, $file); 
    exit(); 
} 

Answer 2

Все $ _REQUEST не продезинфицировать. , как говорили другие, вы должны дезинфицировать пользовательские входы ($ _GET, $ _POST, $ _REQUEST и т. Д.) , если вы не можете иметь дело с php. Я рекомендую вам нанять хорошего программиста php для защиты вашего приложения. Или лучше, не используйте скрипты, которые содержат подобные уязвимости.

, и если они не являются для вас вариантом, добавьте этот код ко всем вашим php-файлам после тега <?php (или в ваш config.php, если он включает все файлы php);

// sanitize $_GET variables 
$_GET = filter_input_array(INPUT_GET, FILTER_SANITIZE_STRING);   

// sanitize $_POST variables 
$_POST = filter_input_array(INPUT_POST, FILTER_SANITIZE_STRING);  

// sanitize $_REQUEST variables 
foreach($_REQUEST as $key => $val){ 
$_REQUEST[$key] = @filter_var(strip_tags(htmlspecialchars($val), FILTER_SANITIZE_STRING)); 
} 

Обратите внимание, что этот код может расколоть некоторую часть вашего приложения, особенно работающих со специальными гольцов ...