Тема: Для знающих (Движок DCMS-Social/Общие вопросы)

berserk2014 (19 May 2018 в 15:08)
Ну да, дыра же

kantry (21 May 2018 в 14:35)
--Цитата--
berserk2014 (19 May 2018 в 15:08):
Ну да, дыра же
----------
berserk2014, даже парочка))
<?php
$file = mysql_fetch_assoc(mysql_query(\"SELECT * FROM mail WHERE `id` = \" . intval($_GET[\'id\']) . \" AND `msg` = \'\" . mysql_real_escape_string($_GET[\'msg\']) . \"\'\"));  
?>
LIMIT там тоже ни к чему.

Zed (21 May 2018 в 17:11)
и зачем в поле msg идет запрос GET а не POST гг

vityachis (21 May 2018 в 19:01)
Как написали выше, переделай прием \'msg\' через POST, если конечно ты не собираешься передавать там несколько символов, что вряд ли :D
И да, как я понимаю msg - это message, скорее всего с большим набором слов, символов... Так вот не лучше ли использовать LIKE? Хотя я наверное просто что-то не понимаю, если это так то просто не обращай внимание на это.
И бери себе привычку фильтровать данные, переменные, которые не были явно присвоены, то есть нельзя доверять всему что приходит извне, даже тому что ты положил в куки пользователю в прошлом его запросе к сайту...

berserk2014 (24 May 2018 в 20:35)
Согласен, с GET  брать, это дебилизм.
Лучше уже с POST + добавить фильтрацию и все.

Zed (25 May 2018 в 22:12)
ну или если там идет выборка по айди то глупо по мне так еще и msg поле трогать,ибо эта даже нелогично как то

RuSInet (27 May 2018 в 14:20)
:D

Источник: http://www.dcms-social.ru/forum/13/34/62471/