PHP.SU

Программирование на PHP, MySQL и другие веб-технологии

PHP.SU Портал

Главная

Помощь

Поиск

Поиск Яндекс Вакансии

Пользователи

Здравствуйте, Гость

( Вход · Регистрация · Правила форума )

Забыли пароль?

Прости меня мама - я быдлокодер

Форумы портала PHP.SU » PHP » Программирование на PHP » Вопросы новичков (Модераторы: OrmaJever, Саныч, Строитель)

Страниц (1): [1]

Описание: в чём мои ошибки

Поиск в теме | Версия для печати

nordghost	Отправлено: 14 Апреля, 2012 - 02:45:11
Посетитель Покинул форум Сообщений всего: 319 Дата рег-ции: Окт. 2011 Помог: 3 раз(а)	Вот пример гостевой книги, хотел бы спросить у опытных людей где я набыдлокодил где есть дыры в безопасности данного скрипта и как оптимизировать код: index.php CODE (htmlphp): скопировать код в буфер обмена <!DOCTYPE html> <html> <head> </head> <body> <form name="answers" action="getinfo.php" method="POST"> <p>Ваше имя: <input type="text" name="uname"/> </p> <p>Ваше e-mail: <input type="text" name="mail"/> </p> <p>Введите сообщение</p> <textarea cols="40" rows="10" name="coment"> </textarea> </p> <input type="submit" value="Оставить отзыв"/> </form> <?php $host='localhost'; $user='root'; $pwd=''; $conn=mysql_connect($host, $user, $pwd) or die (mysql_error()); mysql_select_db('gbook') or die (mysql_error()); $result=mysql_query("SELECT * FROM msgs") or die (mysql_error()); mysql_close($conn); while($rand=mysql_fetch_assoc($result)){ echo "<div style='border:1px solid grey; width:300px; margin:10px;'> <div style='background-color:grey; color:white; text-align:center; width:300px; height:20px;'>$rand[name]</div> <div>$rand[msg] <a href='getinfo.php?del=$rand[id]'>Удалить</a> </div> </div>"; } ?> </body> </html> getinfo.php PHP: скопировать код в буфер обмена <?PHP $host='localhost'; $user='root'; $pwd=''; if($_SERVER['REQUEST_METHOD']=="POST"){ $name=trim(strip_tags($_POST['uname'])); $mail=trim(strip_tags($_POST['mail'])); $text=trim(strip_tags($_POST['coment'])); $conn=mysql_connect($host, $user, $pwd) or die (mysql_error()); mysql_select_db('gbook') or die (mysql_error()); mysql_query("INSERT INTO msgs( name, email, msg ) VALUES ('$name', '$mail', '$text') ") or die (mysql_error()); mysql_close($conn); }else { $id=$_GET['del']; $conn=mysql_connect($host, $user, $pwd) or die(mysql_error()); mysql_select_db('gbook') or die (mysql_error()); mysql_query("DELETE FROM msgs WHERE msgs.id='$id'") or die (mysql_error()); } header("Location: index.php"); ?> ----- <a href="http://netbeans[dot]org/">Моя любимая IDE</a> :3

DelphinPRO	Отправлено: 14 Апреля, 2012 - 03:21:41
Активный участник Покинул форум Сообщений всего: 7187 Дата рег-ции: Февр. 2012 Помог: 353 раз(а)	от xss худо-бедно защитились, а про sql-injection забыли попробуйте перейти по ссылке CODE (text): скопировать код в буфер обмена index.php?del=1' or 1 = 1 or 1 = '1 и вся ваша таблица очистится ----- Чем больше узнаю, тем больше я не знаю.

LIME	Отправлено: 14 Апреля, 2012 - 03:23:51
Активный участник Покинул форум Сообщений всего: 10732 Дата рег-ции: Нояб. 2010 Помог: 322 раз(а)	nordghost пишет: $text=trim(strip_tags($_POST['coment'])); лучше использовать htmlspecialchars() это позволит писать куски кода в тексте nordghost пишет: die (mysql_error()); это используется только при разработке юзеру не надо знать подробностей просто вывод типа "Сори, база сдохла.Приходите послезавтра." nordghost пишет: }else { $id=$_GET['del']; PHP: скопировать код в буфер обмена }elseif (isset($_GET['del'])) { $id=(int)$_GET['del']; надо же предусмотреть ручной вызов от какого-нибудь шалуна nordghost пишет: $rand[name] это вообще непонятно как работает надо {$rand[name]} (Добавление) а вот и наш шалун)))

nordghost	Отправлено: 14 Апреля, 2012 - 14:13:22
Посетитель Покинул форум Сообщений всего: 319 Дата рег-ции: Окт. 2011 Помог: 3 раз(а)	Спасибо огромное всё понял и поисправлял)) теперь буду знать) ----- <a href="http://netbeans[dot]org/">Моя любимая IDE</a> :3

Поиск в теме | Версия для печати

Страниц (1): [1]

Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)

« Вопросы новичков »

Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.