Исследование безопасности news.samaratoday.ru

Namelles One

(L2) cache
Пользователь
Регистрация
24.02.2006
Сообщения
440
Реакции
6
По просьбе DiFor выкладываю некоторые файлы сайта, дабы вы мне помогли закрыть дырки.

Если вам понадобятся какие-то еще файлы - я их предоставлю.

К сожалению, сейчас не имею времени провести эту работу самолично, поэтому обращаюсь к помощи коммьюнити.
 

Вложения

  • showNews.php
    745 байт · Просмотры: 15
  • showNews_full.php
    13.2 КБ · Просмотры: 12
Пожалуйста, обратите внимание, что пользователь заблокирован
В первом скрипте если id новости числовое значение, то желательно добавить intval в 6й строке.
Иначе появляется теоретическая возможность читать любые файлы на сервере.
Типа: http://news.samaratoday.ru/showNews.php?id..../../etc/passwd
(Попробовал вот щас быстренько - не получилось, но у кого-то может и получится, а может кеширование сейчас отключено?..)

Код:
$idNews=intval($_REQUEST["id"]);

Во втором скрипте то-же самое в 5й строке. Иначе возможна скуль.
Код:
$idNews=intval($_REQUEST["id"]);
Пример скули
http://news.samaratoday.ru/showNews.php?id...or%20idNews%3C2


И еще
Код:
$rs = mysql_query($sql) or die("Query failed ".$sql);
Это конечно хорошо для отладки, но когда будет финал то лучше ничего не выводить.
Код:
$rs = mysql_query($sql) or die();


Вечером посмотрю повнимательнее.
В идеале конечно нужно выложить все файлы для проверки.
 


Напишите ответ...
  • Вставить:
Прикрепить файлы
Верх