PHP 8.4.6 Released!

Данные пользовательского ввода

Наиболее опасные уязвимости в PHP-скриптах часто возникают не столько из-за самого языка, сколько из-за кода, который написали с нарушением требований безопасности. Поэтому лучше потратить время на исследование разрабатываемого участка кода, чтобы оценить потенциальную угрозу от ввода переменной с нестандартным значением.

Пример #1 Потенциально опасная обработка переменных

<?php

// Удалить файлы из домашней директории пользователя...
// а может, и ещё из какой-то?
unlink($evil_var);

// Записать в лог-файл выполняемое действие...
// а может, в файл /etc/passwd?
fwrite($fp, $evil_var);

// Выполнение тривиальных действий... или команды rm -rf *?
system($evil_var);
exec($evil_var);

?>

Требуется тщательно проверять код и быть на 100 % уверенным в правильной проверке данных, которые передаёт браузер. Ответьте на следующие вопросы:

  • Влияет ли скрипт только на предполагаемые данные?
  • Обрабатываются ли некорректные или нестандартные данные?
  • Получится ли использовать скрипт способом, который не предусмотрели?
  • Возможно ли использовать скрипт в сочетании с другими скриптами в негативных целях?
  • Правильно ли логируется каждая транзакция?

Лучше задуматься о безопасности при разработке скрипта, а не дорабатывать небезопасный код, когда потребуется исправлять последствия уязвимостей. Такой подход не гарантирует безопасность системы, но помогает значительно снизить количество уязвимостей.

Безопасность повышают путём отключения настроек, которые делают разработку удобной, но скрывают источник, достоверность или целостность данных. Уязвимости к атакам наподобие инъекций или жонглирования данными возникают, когда переменные создаются неявно или когда входные данные не проверяются.

Директива register_globals и директивы механизма magic_quotes, которые удалили в PHP 5.4.0, когда-то способствовали этим рискам, поскольку автоматически создавали переменные из пользовательского ввода и экранировали данные непоследовательно. Хотя директивы удалили из PHP, аналогичные риски сохраняются, когда входные данные обрабатывают неправильно.

Вызов error_reporting(E_ALL) включает режим сообщения об ошибках всех уровней и помогает определять неинициализированные переменные и проверять входные данные. Инструкция declare(strict_types=1) включает режим строгой типизации, который появился в PHP 7 и который повышает безопасность за счёт строгой проверки типов, предотвращает непреднамеренное преобразование типов и повышает общую безопасность.

Добавить

Примечания пользователей 2 notes

up
79
Uli Kusterer
19 years ago
One thing I would repeat in the docs here is what information actually comes from the user. Many people think a Cookie, since it's written by PHP, was safe. But the fact is that it's stored on the user's computer, transferred by the user's browser, and thus very easy to manipulate.

So, it'd be handy to mention here again that:

CGI parameters in the URL, HTTP POST data and cookie variables are considered "user data" and thus need to be validated. Session data and SQL database contents only need to be validated if they came from untrustworthy sources (like the ones just mentioned).

Not new, but I would have expected this info under this headline, at least as a short recap plus linlk to the actual docs.
up
6
Livingstone@stonyhills[dot]com
17 years ago
making sure your form is submitted from your page! Could also be adapted to url, by additing &token to the query string and checking this against session data(or what ever array you like) with $_GET, not that this string is randomly generated and stored. If you like you could build your own array to store the generated string if you dont want to use $_SESSION, say you could make yours like $tokens = array(), and in your easysecure class you store all the stuff in that array!

<?php

class easysecure {

var
$curr_user;
var
$curr_permission;
var
$curr_task;
var
$validpermission;
var
$error;


function &
setVar( $name, $value=null ) {
if (!
is_null( $value )) {
$this->$name = $value;
}
return
$this->$name;
}

function
maketoken($formname, $id){

$token = md5(uniqid(rand(), true));

$_SESSION[$formname.$id] = $token;

return
$token;
}

function
checktoken($token, $formname, $id){
//print_r($_SESSION);
//echo ($token);
//if we dont have a valid token, return invalid;
if(!$token){
$this->setVar('validpermission', 0);
$this->setVar('error', 'no token found, security bridgedetected');
return
false;
}

//if we have a valid token check that is is valid
$key = $_SESSION[$formname.$id];
if(
$key !== $token ){
$this->setVar('validpermission', 0);
$this->setVar('error', 'invalid token');
return
false;
}

if(
$this->validpermission !==1){
echo
'invalid Permissions to run this script';
return
false;
}else{
return
true;
}
}

}

?>

<?php $userid = *** //make it what ever id you like ?>
<form name="newform" action="index.php" method="post">
<input type="text" name="potentialeveilfield" value="" size 30 />
<input type="hidden" name="token" value="<?php echo maketoken(newform, $userid); //$userid here could be user profile id ?>" />
<input type="submit" />
</form>

Now when processing the form... check the value of your token

<?php

//well you know the form name
if(!checktoken($_POST['token'], 'newform', $userid))
{
//failed
exit(); //or what ever termination and notification method best suits you.
//you could also design the class your way to get more accurate fail (error messages from the var)
}

//you can now continue with input data clean up (validation)

?>
To Top