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

Форум АНТИЧАТ (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

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


Время: 21:17