Функция

PHP:

1. <?
2.  
3. function check($info) {
4.     if (!is_numeric($info)) {
5.         $info = htmlspecialchars ($info, ENT_QUOTES);
6.         $info = mysql_real_escape_string ($info);
7.         $info = trim ($info);
8.         $info = substr ($info, 0, 50);
9.     }
10.     return $info;
11. }

 

Проблема как всегда в том ,что автор затруднится рсскими словами сказать, что функция ДОЛЖНА делать!

 

Горбунов Олег, я, кажется, понял и, кажется, даже код написал.

 

Угу, ты правильно догадался...

 

ну и я тоже хочу! %)

Код (Text):

1. function isNum($value) {
2.     return (preg_match("/^[0-9]+$/", $value));
3. }

 

antonn
isNum(-1)==false

 

Dagdamor
это большая проблема, не так ли?

 

В общем, ничем помочь не могу.

 

Давайте разберемся. Себе Вы передаете всегда правильные параметры, а проверка идет для подстраховки. Это значит, что для id нужно проверять, является ли он числом или нет.
if ((int)$_GET["id"] != $_GET["id"]) {
//Не является
}

Если Вам необходимо передать текст, то нет необходимости его сразу экранировать разными способами. Обрабатывать надо в зависимости от ситуации.

 

PHP:

1. <?
2.  
3. function check($id) {
4.     if (!is_numeric($id)) {
5.         //die("офигел шоле сцуко?!");
6.         return 0;
7.     }
8.     return (int) $id;
9. }
10.  
11. // То есть
12. function check($id) {
13.     return is_numeric($id) ? $id : 0;
14. }

 

Ну ну.

 

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

 

Не повторяйте чужих ошибок. Опасных данных не бывает !!!

 

Обезопасит от кого?
Данные это данные. У них зубов нет.

 

данные, будучи употребленные в запросе - это опасные данные. всегда. поэтому нужно позаботиться об их экранировании.
и точка.

 

Горбунов Олег
согласен, зубов нет, и нет вообще ничего материального.
про зубы никто и не спорит.

скажи, sql-i на пустом месте появилась?

 

Данные не опасны, не прекрытая уязвимость - вот это нельзя назвать безопасным.