[Админам]

Уязвимость позволяет получить имя администратора+hash пароля+salt

Версии: <= 2.5.2

Используют "дырку" через:

_http://site.com/infernoshout.php?do=options&area=commands

Уязвимый код в infernoshout.php & inferno_settings.php

$commands = unserialize($this->settings['s_commands']);

if ($this->vbulletin->db->affected_rows() < 1 && !$this->vbulletin->db->query_first("select * from " . TABLE_PREFIX . "infernoshoutusers where s_user='{$this->vbulletin->userinfo['userid']}'"))
{
$this->vbulletin->db->query("
insert into " . TABLE_PREFIX . "infernoshoutusers
(s_user, s_commands)
values
({$this->vbulletin->userinfo['userid']}, '" . serialize($commands) . "')
");
}

Фикс

Originally Posted by kerk

serialize($commands) заменить на $this->vbulletin->db->escape_string(serialize($commands))
и еще
искать похожие вхождения и фильтровать и их так же

а есть исправление? спасибо.

serialize($commands) заменить на $this->vbulletin->db->escape_string(serialize($commands))

и еще
искать похожие вхождения и фильтровать и их так же

Originally Posted by kerk

искать похожие вхождения и фильтровать и их так же

искать именно serialize($commands) или именно какие то похожие?

блин....
посмотрел другие файлы этого хака
да он ВЕСЬ дырявый, мамадарагая...
практически в каждом запросе есть дырки, не фильтрованные входящие данные

Originally Posted by kerk

блин....
посмотрел другие файлы этого хака
да он ВЕСЬ дырявый, мамадарагая...
практически в каждом запросе есть дырки, не фильтрованные входящие данные

могли бы поправить/отфильтровать?

rambo добавил 08-17-2012 в 06:58 PM
кстати в infernoshout.php не нашел ничего такого как serialize($commands) только в inferno_settings.php есть.

Originally Posted by rambo

только в inferno_settings.php есть.

именно там и нужно заменять

Originally Posted by kerk

именно там и нужно заменять

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

В современном обществе принято фильтровать глобальные объекты - POST/GET.
Разве булка сама этого не делает?

Originally Posted by mraxe

булка сама этого не делает?

булка делает
но не все хакописатели