Уязвимость mail отправки на сайте WP

123
S
На сайте с 30.09.2016
Offline
469
#11
intell88:
$email_to = $_POST['email_to'];

Вот это корень козла. Должна производиться выборка из заданного массива адресов, а не подстановка всякого хлама, пришедшего из формы на сайте.

Отпилю лишнее, прикручу нужное, выправлю кривое. Вытравлю вредителей.
I8
На сайте с 12.02.2013
Offline
81
#12
Sitealert:
Вот это корень козла. Должна производиться выборка из заданного массива адресов, а не подстановка всякого хлама, пришедшего из формы на сайте.

Здравствуйте, дело как раз в том, что в форме нет поле для указания mail адреса

Gerga
На сайте с 02.08.2015
Offline
94
#13

intell88, имел ввиду с php кодом, ну ладно. В общем из формы нужно убрать поле email_to, вместо него поставить поле post_id в котором записать id страницы. В функции отправки email получить значение post_id и по этому значению определить получателя. Можно придумать что-то более элегантное, но нужно обмозговать.

---------- Добавлено 07.12.2018 в 11:21 ----------

И добавь WP nonce поле, в котором будет зашифрован Id страницы. В функции отправки проверь полученный Id с WP nonce на правильности данных.

SeVlad
На сайте с 03.11.2008
Offline
1609
#14
intell88:
дело как раз в том, что в форме нет поле для указания mail адреса

Если ты их сделал скрытыми - это вовсе не значит что их нет.


<input type="hidden" name="email_to" value="shaky-m@yandex.ru, dfg@yandex.ru.ru" />

Это поле открыто для всех. кроме честных юзеров.

Не там уязвимость, ой не там..

Делаю хорошие сайты хорошим людям. Предпочтение коммерческим направлениям. Связь со мной через http://wp.me/P3YHjQ-3.
I8
На сайте с 12.02.2013
Offline
81
#15
SeVlad:
Если ты их сделал скрытыми - это вовсе не значит что их нет.

<input type="hidden" name="email_to" value="shaky-m@yandex.ru, dfg@yandex.ru.ru" />

Это поле открыто для всех. кроме честных юзеров.

Не там уязвимость, ой не там..

Не подскажите ли как быть ой, не подскажите ли?))

SeVlad
На сайте с 03.11.2008
Offline
1609
#16
intell88:
Не подскажите ли как быть ой, не подскажите ли?))

/ru/forum/comment/15864693

Gerga
На сайте с 02.08.2015
Offline
94
#17
SeVlad:
Не там уязвимость, ой не там..

в голове) шутка.

SeVlad
На сайте с 03.11.2008
Offline
1609
#18
Gerga:
шутка

Почему шутка? Думаешь всё гораздо серьёзней - в ДНК? ;)

Gerga
На сайте с 02.08.2015
Offline
94
#19
SeVlad:
Почему шутка?

чтобы звучало безобидно, мало ли)

D.iK.iJ
На сайте с 26.05.2013
Offline
225
#20

Ну логично же, что должно быть как-то так!

$email_to = "shaky-m@yandex.ru, dfg@yandex.ru.ru";

или передаете (как уже писали) номер страницы, а в скрипте этим номерам присваиваете мыло.

if($_POST['email_to']=="1") {$email_to = "shaky-m@yandex.ru, dfg@yandex.ru.ru";}
if($_POST['email_to']=="2") {$email_to = "***@yandex.ru";}

Это - только пример! В скрипте настолько нет защиты, что я немного в ужасе. Ну нельзя же настолько доверять входящим данным.

Адаптивный дизайн в 2 строчки ( https://dikij.com/wm/adaptaciya-saytov.php ). + Принимаю заказы любой сложности ( https://searchengines.guru/ru/forum/926323 ). 💎 Еще я делаю классные кулоны с опалами ( https://mosaicopal.ru/ ).
123

Авторизуйтесь или зарегистрируйтесь, чтобы оставить комментарий