Форум АНТИЧАТ

Форум АНТИЧАТ (https://forum.antichat.xyz/index.php)
-   Проверка на уязвимости (https://forum.antichat.xyz/forumdisplay.php?f=110)
-   -   Скрипт добовления новостей работает с БД (https://forum.antichat.xyz/showthread.php?t=62065)

Doom123 17.02.2008 23:09

Проверьте плиз самописный сайт новостей
 
Большая просьба проверить сайтег который я написал там через админку идёт добовление + редактирование + удаление новостей а на главной отображаю эти новости кому не лень посмотрите плз (диз это так..)

Проверьте плиз если можно както оптимизировать код
или если есть какието баги


http://zakazi.ho.ua - сам сайтег
http://zakazi.ho.ua/admin.php - админка

пароль админа : test

исходники:
http://zakazi.ho.ua/Zakaz.zip

GreenBear 17.02.2008 23:57

непонятно зачем ты каждый раз пытаешься таблицу создать

Doom123 18.02.2008 00:14

ну в книге по пхп было написано что лучше делать так т.к это удобней... ну еси скипт не большой тоесть еси таблиц не много

GreenBear 18.02.2008 00:19

сожги книгу и прокляни ее автора

Doom123 18.02.2008 00:37

адин из них разработчик денвера второй известный в россии прогроммист вроде норм авторы=)

П.с если таблиц несколько сказали как это всё сделать 2мя запросами.. так что если ты о запросах то всё ок)

Isis 18.02.2008 01:41

1) Типа SQL
Цитата:

http://zakazi.ho.ua/index.php?s=-1
2) Обход бана:
a)Делитим куки ban
b) Ставим куки count со значение 6

Цитата:

PHP код:

    if (isset($_GET['s']) && !empty($_GET['s']) && is_numeric($_GET['s'])) 
    {
    
$pg=htmlspecialchars($_GET['s']);
    } 


Зачем цифры в htmlspecialchars?

Doom123 18.02.2008 01:47

Ну бан это так.. я придумать больше нечё не мог а с sql я промазал .. =)

Кстате с той sql Можно было чтото зделать?

там идёт умножение на число и если бы ты передал

в параметре s union+select или какие либо слова

вышло бы union+select * 2 что было бы равно 0 не так ли?

а если index.php?s=-1+union+select+bla+bla

это всё умножилось бы на 2 скжем и вышло бы -2 =)

xXvladXx 18.02.2008 13:22

if (isset($_GET['s']))
{
if(is_numeric($_GET['s']) and $_GET['s']>0){
$pg=$_GET['s'];
}else{
$pg=0;
}
}
можно сделать так...

freddi 18.02.2008 13:45

не фильтруется ни один параметр админки если мы не авторизировались.
step, name1, cont, redact ...

Doom123 18.02.2008 15:14

Када ты не авторизован ты ничего не сделаеш

freddi 18.02.2008 15:47

Doom123, всмысле ничего не сделаю? если у тебя при входе все фильтруется, а когда ты еще не вошел нет, то уже явно чтото с кодом не так.

desTiny 19.02.2008 19:59

Странный код в mysql_connect.php: код по созданию таблицы не выполнитс НИКОГДА, так как в случае её отсутствия мы выходим ещё в строке
PHP код:

mysql_select_db($db)
    or die(
"Не могу выбрать базу данных: ".mysql_error()); 


desTiny 19.02.2008 20:02

И автроизация через куки, и баны через них - шедевр!! Авторизация через .htaccess - гораздо лучше!

GreenBear 19.02.2008 21:47

авторизацию по кукам вполне нормальное явление. с антибрутом согласен...
в общем рекомендую сделать все по посту #4
и учить по нормальному мануалу - php.net/docs

desTiny 19.02.2008 22:14

#14:
Сейчас цель - показать слабые места конкретного сайта, а не обучить ТС-а красивому php=)

Но авторизация через куки в случае с админкой, у которой администрация возможна только со всеми правами, имхо, - бессмысленная трата времени и потенциальная слабость в связи с
а) непрофессиональной их обработкой
б) их можно тупо спереть несколькими способами=)

Doom123 20.02.2008 01:48

GreenBear - хватит уговаривать меня сжечь книгу=) нормально всё с книгой и для новичка вполне подходит а по докам эта не учёба а просто описание... в книги тебе говорят что стоит применять, где и когда
desTiny - Ну извените.. в принципе для этого и существует практика чтоб в будущем знать что и как делать... куки и всю эту дребидень использовал т.к практиковал только что выученый материал=)

и кстате о чём тут вообще речь?
mysql_select_db() - Функция выбора Базы Данных
А База Данных и таблица в этой базе это 2 разные вещи

desTiny 20.02.2008 21:30

"mysql_select_db() - Функция выбора Базы Данных"
а, сорри, совсем крыша едет.. Тогда вообще бред в каждом вызове пытаться создать таблицу... Действительно, поставь книжку на полку и не трогай её...


Время: 01:29