Просмотр полной версии : обработка нового поста в форуме
обработка нового поста в форуме
достаточно ли пропускать пост через данную функцию чтобы обезопасить форум?
function replaces($val)
{
if ($val == '') return('');
$val = htmlspecialchars($val);
$val = str_replace( "\"", '"', $val );
$val = str_replace( "`", '"', $val );
$val = str_replace( "'" , '"', $val );
$val = preg_replace( "/\\\$/", '$', $val );
$val = preg_replace( "/\r/",, $val );
$val = stripslashes($val);
$val = preg_replace( "/\\\/", '\', $val );
$val = nl2br($val);
return($val);
}
при условии что пока что там нет псевдо тегов и смайлов
вот моя альтернатива:
$val=str_replace("'", "'", $val);
$val=str_replace('"', """, $val);
$val=str_replace(">", ">", $val);
$val=str_replace("<", "<", $val);
$val=str_replace("`", "`", $val);
$val=str_replace("\\", "\", $val);
+ nl2br() и этого будет достаточно. В твоем же примере абсолютно ненужным являются строчки
$val = str_replace( "\"", '"', $val );
$val = str_replace( "'" , '"', $val );
$val = stripslashes($val);
Первые две уже обрабатываеются при htmlspecialchars() с флагом ENT_QUOTES вроде. А stripslashes() вообще не нужен, коли ты преобразуешь все ковычки до этого.
Непонятно зачем делаешь:
$val = preg_replace( "/\r/",, $val );
Вообще не понятно. \r и \n абсолютно никак не влияют на отображение html`а. Тем более что ты все равно потом делаешь nl2br().
Не понял зачем нужна
$val = preg_replace( "/\\\$/", '$', $val );
Вообще не понял =)))
Опять же, Дикс, твоя основная ошибка. Опять укажу на нее все тем же анекдотом:
"Береженого бог бережет", - сказала монашка, надивая презерватив на свечку. Проще будь. Начни с того, что разберись, какие именно символы тебе нужно отфильтровать. И уже потом пиши код.
И еще одно. Заменяя двойную, одинарную ковычку и перфекс на "... Ну это короче не правильно. Если я пишу на форуме перфикс, значит я хочу чтобы именно перфикс стоял там, где я его поставлю. А не двойная ковычка, как у тебя. К пример я пишу I`m very big lamer, а твой форум выдаст мне I"m very big lamer. Не хорошо это
ох, Тринукс. даже не знаю как всё объяснить чтоб меня поняли.
имел я глупость поставить перед собой задачу - написать движок сайта да ещё и со внутренним форумом.
ничего подобного раньше не писал, кроме простого форума и простого сайта в отдельности.
да ещё и подпрягся сделать на этом движке сайт для своей тренажёрки, за два месяца бесплатных хождений туда (1100р. экономии)
Вот и мечусь теперь из одного куска кода в другой.
Пишу основу - главные функции, чтобы всё работало, а мелочи типа авторизации или фильтрации постов оставляю на потом. Но чтобы они всё равно уже были,
как ты мог заметить я леплю туда что попало. Ты видел мою тему, где инфа о юзере пишется в куки и читается из них. Таким же образом я выдрал готовый код фильтрации из ExBB, но толком не разобравшись, лишь чуток его подрезал.
Такие тонкости как флаги ENT_QUOTES я если и изучил при прочтении книги, то ещё не успел усвоить и научиться применять.
Допишу основу и начну шлифовать код, тогда ваши советы мне очень и очень помогут.
Большое спасибо, Тринукс, чтоб я без тебя делал =)
Che-Guevara
09.05.2006, 20:36
2Trinux не злоупотребляй двойными кавычками :) кстати говоря в большенстве случаев достаточно фильтровать ' и < . Причем первый символ чтобы не было выхода за пределы атрибута value и предотвращения атаки типа sql-inj (есесино при передаче чисел intval() Дополнительно), а второй, чтобы не было возможности вставлять свои теги.
2 Che-Guevara
Ты не прав. А что если у меня атрибуты тега заключены не в одинарную ковычку, а в двойную? Значит нужно фильтровать обе. Далее. Перфикс. Как ты уже, надеюсь, знаешь (а знать ты это можешь, почитав инфу на главной странице antichat.ru) перфикс юзается в IE как символ начала\конца комментариев. А значит это реальная опасность вхождения в атрибуты тега. Значит нужно фильтровать и его. Далее, обратная треугольная скобка то е обязательна и то же при нефильтрующемся перфиксе дает несколько больше возможностей. Точнее те же, но в более простой форме. Обратный слеш... ну пишу я, к примеру
abc\tdef
а если не фильтровать его, то получу
abc def
А это уже не хорошо.
2 Дикс
тогда ваши советы мне очень и очень помогут.
Перестань выкать =) не такой я уж и старый =)
ещё забыл спросить это зачем?
$val=str_replace("'", "'", $val);
Перестань выкать =) не такой я уж и старый =)
:d я имел ввиду всех античатовцев :)
кстати, вам сколько лет? :)
ещё, извините ламера, что такое "перфикс"? или префикс?
ещё забыл спросить это зачем?
Форум сожрал, там та же одинарная ковычка, тольво в десятичной кодировке.
перфикс или префикс - сам ХЗ, но это - `
ээ, а каг бы мне узнать десятичную кодировку кавычки?
и ещё интерпретатор выдаёт ошибку на строчке
$val=str_replace("\\", "\", $val);
где б мне узнать десятичный код бэкслеша?
и ещё интерпретатор выдаёт ошибку на строчке
PHP код:
$val=str_replace("\\", "\", $val); писец. писать движок с форумом, не зная основ пхп.
Фтыкай http://us2.php.net/language.types.string до полного всасывания материала.
2 Дикс
в моем примере форум пожрал десятичную кодировку, поэтому и ошибка. Следуй совета nerezus`а
я английский плохо знаю, плз, скажите этот десятичный код :)
прочитай на русском. русская дока там тоже есть
я английский плохо знаю, плз, скажите этот десятичный код
http://antichat.ru тут все есть по этой теме
:плачет:
ребята, я не думаю что я настоко уж ламер, но пожалейте плз бедного технолога сборки топливной системы Су-34, который пашет круглые сутки в цехе окончательной сборки над шестой машиной Су-34, скажите плз эти коды, чтоб можно было уже поскорее закрыть этот вопрос, а потом уж я как-нить сам научусь (((
2 Дикс
За су-34 респект отдельный, коли не врешь.
http://antichat.ru/crackchat/HTML/ внизу таблица
пасиба
про сушку не вру. НАПО им. Чкалова, цех окончательной сборки №9
инженер-технолог
ещё вопрос (не могу свою тему про обработку пароля найти)
if (preg_match("/^([a-z0-9]{3,10})$/i", "sdfsdfdsfdsfsd"))
почему не пропускает?
я читал мануал, там пишут
"xy{2,6}" соответствует строке, в которой за x следует от двух до шести y;
где связь?
vBulletin® v3.8.14, Copyright ©2000-2026, vBulletin Solutions, Inc. Перевод: zCarot