Проэкт о попутной доставке.

http://www[dot]pdostavka[dot]ru/ - доска объявлений о попутной доставке.
Сам писал, прошу всю критику сюда. Буду думать)

http://www[dot]pdostavka[dot]ru/save_log[dot][dot][dot]e=&password=
при пустых логине и пароле
Warning: mysql_close(): no MySQL-Link resource supplied in /home/d/dazar/pdostavka.ru/publi c_html/save_login.php on line 95

У body не прописан бекграунд, да и дизайн больше похож на голый псевдоинтерфейс.

На вскидку так...
(Добавление)
У картинок присутствует бордер в 2 пикселя

-----
Шта? Репозиторий?

продолжать не станем

-----
if(time()>1356048000) die();

White, я не смог воспроизвести ошибку, о который ты говорил. Как ты её получил?
(Добавление)
Бордер картинок убрал. Ошибку при пустых логине и пароле убрал. Дизайн согласен, тупо интерфейс. Буду думать)

(Отредактировано автором: 16 Сентября, 2011 - 16:17:25)

Dazar заходим на сайт под своим логином и вводим
http://www[dot]pdostavka[dot]ru/user.php[dot][dot][dot]_id=0%20OR%201=1
примитивное начало скули, далее с помощью GROUP BY как описано выше вычисляем количество полей... а впрочем дальше объяснять не буду.
чтобы воспроизвести ошибку, нужно ввести
http://www[dot]pdostavka[dot]ru/user.php[dot][dot][dot]ms&user_id=i

П.С. все это демонстрирует примитивную уязвимость к sql inject. даже та крошечная ошибка которую я вам показал (а таких вероятно не мало) может нанести колоссальный вред вашему сайту.

-----
if(time()>1356048000) die();

Ага, то есть мы меняем id пользователя на букву, и он выдаёт ошибку, ведь должно быть число. Чтобы это исправить надо ввести какую-то проверку... Буду думать какую

Dazar сама ошибка безобидна, а вот последствия к которым она может привести гораздо более значимы.

такая ссылка просто выведет все данные из таблицы, ничего страшного, но с помощью одной такой ссылки можно узнать всю структуру вашей БД, пароли и данные. Для того, чтобы этого не произошло в вашем случае там где у вас было $_GET['user_id'] заменить его на (int)$_GET['user_id'], в общем случае можно применить mysql_escape_string() и заключить результат в двойные кавычки.
подробнее о sql injection http://en[dot]wikipedia[dot]org/wiki/SQL_injection

(Отредактировано автором: 19 Сентября, 2011 - 17:04:29)

-----
if(time()>1356048000) die();