Проверка на безопасность

[rus]

Кто-нибудь может проверить на безопасность и сказать что нужно (или не нужно) дополнить-защитить и как.

http://webbeginner.ru/admin/index.php

логин:demo

пароль:test

[Pavel_html]

сек...

итак.

тут нужно сделать не radio кнопки а флажки.

тут почемуто отображается форма, хотя ид не валиден.

заватра еще добавлю

Edited September 13, 2008 by Pavel_html

[Dimitry Wolotko]

http://webbeginner.ru/admin/edit_lesson.php?id=8

[rus]

http://webbeginner.ru/admin/edit_lesson.php?id=8

Я так понимаю htmlspecialshars нужно применить? или что?

[Dimitry Wolotko]

Ага.

[rus]

Ага.

Ну и думаю что стрипслэшес тоже, да?

А есть какие-либо еще замечания?

сек...

итак.

тут нужно сделать не radio кнопки а флажки.

тут почемуто отображается форма, хотя ид не валиден.

заватра еще добавлю

На счет флажков, - а какая разница-то?

И на счет формы, она и должна отображаться.

В каком смысле id не валиден, поподробнее?

[rus]

Дим, а сейчас попробуй...

[Dimitry Wolotko]

по поводу удаления.

А если удалить надо 189 записей? Вот тут то флажки и пригождаются.

Вбей ID, к примеру, 34658273564278 - почему выводится форма, а не ошибка?

Да, проблема с ">lll решена.

Повесь алерт на удаление, мало ли случайно что-нибудь нажмёшь.

Дата имеет непонятный формат. Что вбивать - то? гг-мм-дд? Или гг-дд-мм?

[rus]

по поводу удаления.

А если удалить надо 189 записей? Вот тут то флажки и пригождаются.

Вбей ID, к примеру, 34658273564278 - почему выводится форма, а не ошибка?

Да, проблема с ">lll решена.

Повесь алерт на удаление, мало ли случайно что-нибудь нажмёшь.

Дата имеет непонятный формат. Что вбивать - то? гг-мм-дд? Или гг-дд-мм?

Да, по поводу флажков разъяснил, спасибо за идею, теперь понятно.

По поводу даты, сделаю - это не проблема.

По поводу алерта иожно подробнее?

А вот по поводу id - это как?

<? php
if (!isset($id)) {$id = id;}
?>

Так?

Edited September 13, 2008 by rus

[Dimitry Wolotko]

При нажатии на кнопку "удалить из БД" - выскакивает алерт - "вы точно хотите удалить 189 записей?!"

По поводу Id - проверяй, есть ли он в БД, если нет, то выводи сообщение с ошибкой.

[rus]

По поводу Id - проверяй, есть ли он в БД, если нет, то выводи сообщение с ошибкой.

<? php
if (!isset($id)) {$id = id;}
echo "Такого id не существует!";
?>

Так?

[Pavel_html]

$r = mysql_query("SELECT * FROM таблица");
$num = mysql_num_rows($r);
if(isset($_GET['id'])){
 $id = ($_GET['id']<=$num) ? true : false;
 if(!$id){
  echo "Неправильный айдишник!";
 }else{
  $id=(int)$_GET['id'];
 }
}

Edited September 15, 2008 by Pavel_html

[Victor Ananiev]

Да кстати а на счет даты, почему ее через timestamp низя серверно вставлять?

[rus]

$r = mysql_query("SELECT * FROM таблица");
$num = mysql_num_rows($r);
if(isset($_GET['id'])){
 $id = (int)($_GET['id']<=$num) ? $_GET['id'] : false;
 if(!$id){
  echo "Неправильный айдишник!";
 }
}

Спасибо попробую.

Да кстати а на счет даты, почему ее через timestamp низя серверно вставлять?

Это будет дорабатываться потом.

Главное проверка на уязвимость базы.

Блин, кто-то столько записей мне наплодил!!!

Edited September 14, 2008 by rus

[Victor Ananiev]

Это не я)

[Dimitry Wolotko]

Наверняка это Паша.

[Victor Ananiev]

Гыы

[Pavel_html]

да это. сначало F5 потом enter и так сорок раз

[Dimitry Wolotko]

да это. сначало F5 потом enter и так сорок раз

Лузер - а автоматизировать слабо?)

[rus]

что-то нифига не получается удалить выбранные чекбоксы, получается удалить либо все, либо один, а как удалить именно выбранные?

Делаю так:

$result = mysql_query ("DELETE FROM table WHERE id='$id'");

to Pavel_html:

Паш, спасибо с айпи получилось, но вот странно одно, из базы все удалил, а когда новую инфу вносишь - у нее id равен не 1, а последнему числу внесенной инфы, тогда в любом случае показывает что ид неправильный.

Почему внесенной инфе не присваивается значение ид = 1?

Edited September 15, 2008 by rus

[Pavel_html]

сам задаю себе такой вопрос)) будем лапика или диму ждать)))

[rus]

...лапика...

[rus]

Есть какие-нибудь предложения (мнения) по какой причине это может быть и как это можно исправить?

Тема еще не закрыта...

Edited September 16, 2008 by rus