ОБСУЖДЕНИЕ

Ищу уязвимые на этом коде

5 сообщений1 просмотровОткрыта

#12016-02-06 15:10:17

Ищу уязвым этом код. Кто нибудь может объяснить? Антивир ругает:

CODE

// Удаление буквы
if(isset($_GET['delete'])) {

$_GET['delete'] = abs(intval($_GET['delete']));

    if(isset($_SESSION['reply'][$_GET['delete']])) {

    if($_SESSION['reply'][$_GET['delete']] != '_') {

$_SESSION['uid'][] = $_SESSION['reply'][$_GET['delete']];
$_SESSION['reply'][$_GET['delete']] = '_';

header('location: ?');
exit;
}
}
}

#22016-02-06 15:18:38

CODE


$_SESSION['uid'][] = $_SESSION['reply'][$_GET['delete']];

Может я нуб, но что за 2 скобке и там пустота?

#32016-02-06 15:45:29

да ничего он не уязвим ) Все окей)

#42016-02-06 16:05:39

Это автозаполнение массива двойной размерности.
Т.е $array['uid'][] = 'one';
$array['uid'][] = 'two';

То будут доступны через
$array['uid'][0] и $array['uid'][1]

#52016-02-06 17:02:06

Ну и раз ты выложил, то лучше сделать так

$delete = abs(intval($_GET['delete']));

И далее уже вместо $_GET['delete'] писать $delete

Выглядит лучше и не выбирает каждый раз delete из массива