$_GET нужен совет

Нагружен сильно, проверять надо не все данные а только те которые нуждаются в проверке, а уж править и тем более...

 

wedest
Понимаешь, не бывает абстрактной проверки на безопасность.
Проверять надо конкретные переменные, в зависимости от того, что они делают.
К примеру, если ты передаёшь имя файла, то надо проверять, чтобы оно было не зловредным.
А просто так сказать, что делать с переменными $_GET - ничего нельзя

А вообще, твой код ничего не защищает и не меняет.

 

+1

 

PHP:

1. <?
2. // Защита от нецелочисленных GET-переменных
3.    
4.     function int_protect($what)
5.     {
6.         if (empty ($what))
7.             return 0;
8.         else {
9.             if ((string) $what != (string)(int) $what) {
10.                 die ("<b>Внимание!</b><br>Зафиксирована попытка взлома.<br>");
11.             }
12.             else
13.                 return $what;
14.         }
15.     }
16. ?>

 

Код переписывает одну и ту же переменную. Ближе наверно так

PHP:

1. $list = array_flip(array(\"key1\", \"key2\", \"key3\", \"key4\"));
2. #Перечисляем разрешенные к обработке переменные
3. while (list($key, $value)=each($list)){
4.   if (isset($_GET[$key])) $$key=htmlspecialchars(trim(substr($_GET[$key], 0, 255)));
5.   #Если есть то определяем для дальнейшего использования.
6. }

В этом случае у тебя не будут обрабатываться переменные которые ты не используеш в скрипте.