URL: https://www.opennet.me/cgi-bin/openforum/vsluhboard.cgi
Форум: vsluhforumID3
Нить номер: 99431
[ Назад ]

Исходное сообщение
"Критическая уязвимость в системе управления web-контентом Dr..."

Отправлено opennews , 16-Окт-14 00:20 
В выпуске (https://www.drupal.org/drupal-7.32) системе управления web-контентом Drupal 7.32 устранена критическая уязвимость (https://www.drupal.org/SA-CORE-2014-005) (CVE-2014-3704), позволяющая осуществить подстановку SQL-кода. Уязвимости присвоен наивысший уровень опасности (Highly critical (https://drupal.org/security-team/risk-levels)), сигнализирующий на возможность осуществления удалённой атаки, способной привести к получению доступа к системе.


Уязвимость вызвана ошибкой в реализации метода "prepared statement" в интерфейсе абстрактного доступа к БД и может быть эксплуатирована анонимным посетителем. Отправив специально оформленный запрос к сайту, реализованному на базе Drupal, атакующий может выполнить произвольный SQL-запрос на сервере, манипулируя которым можно добиться выполнения PHP-кода или получения доступа к серверу. Прототип эксплоита не публикуется, но в тестовом сценарии фигурирует передача значения "1 ;INSERT INTO {test} SET name = 'test12345678'; -- ".


URL: https://www.drupal.org/drupal-7.32
Новость: http://www.opennet.me/opennews/art.shtml?num=40844


Содержание

Сообщения в этом обсуждении
"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 00:20 
> Drupal

Я ни разу не удивлён. Но думал, что Джумлу вряд ли переплюнут.


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Джон , 16-Окт-14 04:54 
Есть сервер с голой джумлой. Взламываешь? 100 баксов твои. Надо больше говори, не вопрос.

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 05:56 
> Надо больше говори, не вопрос.

Ну это смотря насколько больше :). Если вы 31337 баксов как гугля предложите - у вас появится несколько чуваков, выискивающих баги fulltime, вероятно :)


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено XakRu , 16-Окт-14 10:46 
Если вас еще не взломали - это не значит, что ваша система неуязвима. Возможно вы просто нам не интересны!

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Michael Shigorin , 16-Окт-14 12:20 
> Есть сервер с голой джумлой. Взламываешь? 100 баксов твои.
> Надо больше говори, не вопрос.

А теперь объясняю, как выглядит в жизни.

Некую CMS засунули на хостинг, активно залатывают, сайт наполняют контентом.  Проходит год-два-три.  Изначальные студент/фрилансер/студия вроде как уже и не нужны, всё устоялось, проблем особых нет.  Латание плавно или скачком прекращается, продолжается наполнение.  Проходит ещё какое-то время, за которое поддерживаемая ветка закрывается, переезд на следующую требует как минимум обращения к с/ф/с, подчас нетривиальной мороки (т.е. времени и денег), изредка существенной переработки части функциональности.

Так вот важно не то, сколько именно сейчас в условной "голой жумле", которая нафиг никому не упёрлась сама по себе, эксплуатируемых/известных уязвимостей.  Важно то, сколько их вылезет к тем нескольким годам плюс какому-то времени, и сколько будет заткнуто за эти первые годы.

Я доступно излагаю?


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 13:06 
Прям за меня сказал.

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Vov , 16-Окт-14 14:50 
А имеется ли разница между 100 потенциальными уязвимостями Жумлы или одной (найденной) в CMSВАШЕЙМЕЧТЫ если результат практически один (потеря данных или что-то подобное)

Уязвимости находятся регулярно во всем мало-мальски используемом. Можно поступить подобно страусу и написать свой велосипед с вдвое большим кол-вом уязвимостей, но пребывать в спокойствии, не находя сообщений о них.


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Сергей , 16-Окт-14 15:03 
Разница в том, что 50 из тех 100 могут быть известны только тем, кто ими пользуется.

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено YetAnotherOnanym , 16-Окт-14 15:25 
Если она одна и закрыта, потенциальные злоумышленники переключатся на ПО, в котором их находят пачками.

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Michael Shigorin , 17-Окт-14 14:23 
> А имеется ли разница между 100 потенциальными уязвимостями Жумлы или одной (найденной)
> в CMSВАШЕЙМЕЧТЫ если результат практически один (потеря данных или что-то подобное)

Видите ли, практика показывает, что на условную сотню найденных уязвимостей жумлы приходится условно одна найденная дырка в "CMSНАШЕЙМЕЧТЫ".  И с таким "авторским коллективом" оной жумлы это как было предсказуемо годы тому, так и останется.  Увы.

Про "потенциальные уязвимости" можете рассказывать "потенциальным заказчикам", кинетические уже на этом молоке обожглись и мальчикам-руководам не верят.


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено yutoks , 17-Окт-14 11:05 
тут от мозга админа больше зависит, чем от скриптов.
как рассчитал безопасность системы.

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено all_glory_to_the_hypnotoad , 16-Окт-14 00:23 
> Уязвимость вызвана ошибкой в реалиpации интерфейса абстрактного доступа к БД

вот, значит, как теперь называется склеивание строчек в sql запрос на коленке в пыхе


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Нимо Ан , 16-Окт-14 00:48 
Давно пора запретить передачу SQL от клиента (middleware) к серверу (СУБД) и реализовать унифицированный интерфейс к хранимым процедурам, а внутри уже пусть городят как хотят.

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено vitalif , 16-Окт-14 01:56 
чур тебя

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 05:57 
> Давно пора запретить SQL

//fixed. Bobby Tables meets John The Scripter.


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 09:35 
Мдя, как всё тяжко то у людей...

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Обычный аноним , 16-Окт-14 09:53 
> Мдя, как всё тяжко то у людей...

Так ты марсианин???


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 10:02 
>> Мдя, как всё тяжко то у людей...
> Так ты марсианин???

А ты комменты читаешь по одному, не обращая внимания на иерархию? :)


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Обычный аноним , 16-Окт-14 14:03 

> А ты комменты читаешь по одному, не обращая внимания на иерархию? :)

Промах. Руки трясутся :))


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено manster , 16-Окт-14 00:27 
...сколько еще таких сюрпризов на просторах любителей экономить на prepare-параметрах...
Возможно это от большой любви ко всякого рода ORM...

Практически любой проект, не использующий prepare-параметры, но активно работающий с БД, имеет потенциальную возможность предоставить брешь.


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено uldus , 16-Окт-14 00:35 
> .сколько еще таких сюрпризов на просторах любителей экономить на prepare-параметрах...

Вообще-то эта уязвимость как раз в коде обработки  prepare-параметров :-)

Drupal uses prepared statements in all its SQL queries. To handle IN statements there is an expandArguments function to expand arrays.
...
The function assumes that it is called with an array which has no keys. Example:

    db_query("SELECT * FROM {users} where name IN (:name)", array(':name'=>array('user1','user2')));

  Which results in this SQL Statement

    SELECT * from users where name IN (:name_0, :name_1)

  with the parameters name_0 = user1 and name_1 = user2.

  The Problem occurs, if the array has keys, which are no integers. Example:

    db_query("SELECT * FROM {users} where name IN (:name)", array(':name'=>array('test -- ' => 'user1','test' => 'user2')));

  this results in an exploitable SQL query:

     SELECT * FROM users WHERE name = :name_test -- , :name_test AND status = 1

  with parameters :name_test = user2.

  Since Drupal uses PDO, multi-queries are allowed. So this SQL Injection can
      be used to insert arbitrary data in the database, dump or modify existing data
      or drop the whole database.

  With the possibility to INSERT arbitrary data into the database an
  attacker can execute any PHP code through Drupal features with callbacks.



"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено manster , 16-Окт-14 01:16 
>[оверквотинг удален]
>   with parameters :name_test = user2.
>   Since Drupal uses PDO, multi-queries are allowed. So this SQL
> Injection can
>       be used to insert arbitrary data
> in the database, dump or modify existing data
>       or drop the whole database.
>   With the possibility to INSERT arbitrary data into the database
> an
>   attacker can execute any PHP code through Drupal features with
> callbacks.

Возможно, это предположение только. Надо смотреть код. Сейчас многие т.н. драйвера могут маскироваться под placeholders для prepare, но не отрабатывающие "честный биндинг", а просто выполняют замену значениями через search/replace

В случае, когда выполняется prepare+binding подстановка кода не очень возможна, если только как строковое значение поля, когда тип позволяет.


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено vitalif , 16-Окт-14 01:58 
Да у них просто видимо этот prepare эмулируется их же кодом.

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 09:50 
> Да у них просто видимо этот prepare эмулируется их же кодом.

Там у них "для облегчения" есть функция expandArguments которая перед pdo-ным prepare переписывает тело запроса хитрым preg_replace-ом :), более подробно не изучал :)


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 09:39 
>[оверквотинг удален]
>   with parameters :name_test = user2.
>   Since Drupal uses PDO, multi-queries are allowed. So this SQL
> Injection can
>       be used to insert arbitrary data
> in the database, dump or modify existing data
>       or drop the whole database.
>   With the possibility to INSERT arbitrary data into the database
> an
>   attacker can execute any PHP code through Drupal features with
> callbacks.

А то что при prepare запроса, "prepared" запрос на сервере не меняется php-никам невдомёк конечно, они настолько привыкли к "расширениям" добавляемым всякими ORM и тому подобным...


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 11:03 
prepared statements так не работает, это что-то похожее по api на него. Как уже некоторые заметили, некоторые ущербные биндинги эмулируют софтварно, например почти все к mysql

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено angra , 16-Окт-14 17:20 
Можно еще сказать, что данная ошибка есть следствие дизайна PHP, в котором нет различия между обычными и ассоциативными массивами aka хешами.

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено manster , 16-Окт-14 18:16 
Да есть такое. Типизированные массивы и списки тогда уж. Само по себе числовые индексы или символьные не большое зло - частный случай индексов. Но это плата за универсальность. Решили сэкономить на списках и кортэжах. Но ошибка не совсем с этим связана, "палёный код", впрочем от друпала не удивительно может быть.

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено all_glory_to_the_hypnotoad , 16-Окт-14 20:13 
> Cамо по себе числовые индексы или символьные не большое зло - частный случай индексов.

ни что ни у чего тут не частный случай.

> Но это плата за универсальность.

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


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено demimurych , 16-Окт-14 00:38 
Вы бы в патч посмотрели
или хотя бы новость внимательно прочли, чтобы понять в чем проблема и как она появилась.

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 09:51 
> Вы бы в патч посмотрели
> или хотя бы новость внимательно прочли, чтобы понять в чем проблема и
> как она появилась.

http://www.opennet.me/openforum/vsluhforumID3/99431.html#18


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 00:42 
Интересно whitehouse.gov и london.gov.uk уже пропатчили? Они когда-то громко кричали о переходе на Drupal Ж-)

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено йцу , 16-Окт-14 08:52 
Загуглите "prepared statement in" - удивитесь количеству одинаковых вопросов на stackoverflow (речь не только о PHP). Есть вообще БД API, которое поддерживает из коробки биндинг коллекций к одному плейсхолдеру? А ведь достаточно часто возникающий кейс.

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 09:56 
> Загуглите "prepared statement in" - удивитесь количеству одинаковых вопросов на stackoverflow
> (речь не только о PHP). Есть вообще БД API, которое поддерживает
> из коробки биндинг коллекций к одному плейсхолдеру? А ведь достаточно часто
> возникающий кейс.

Оно не нужно, не дело СУБД разгребать коллекции, и реализация такого алгоритма должна зависеть от ситуации а не "сейчас мы регулярками преобразуем строку запроса..."

p.s. старое утверждение про невозможность выполнения всех 3-х критериев разработки работает :)


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено йцу , 16-Окт-14 13:45 
Ну алгоритм-то зачастую один и тот же - взять список чисел/строк и впихнуть в IN выражение.

Не дело СУБД - ok, но почему в тех же PDO, JDBC и т.п. API не добавить соответствующий метод? В DQL (doctrine) и HQL (hybernate) к примеру такая возможность есть (я понимаю, что это несколько более высокоуровневые штуки, но не вижу причину чтобы не реализовать это уровнем ниже - ну ведь нужная же вещь).


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 15:01 
> Ну алгоритм-то зачастую один и тот же - взять список чисел/строк и
> впихнуть в IN выражение.
> Не дело СУБД - ok, но почему в тех же PDO, JDBC
> и т.п. API не добавить соответствующий метод? В DQL (doctrine) и
> HQL (hybernate) к примеру такая возможность есть (я понимаю, что это
> несколько более высокоуровневые штуки, но не вижу причину чтобы не реализовать
> это уровнем ниже - ну ведь нужная же вещь).

вы же сами ответили - разный уровень.
Если вы пришете в запросе in ( :param ), то :param - это то что выдерается при подготовке запроса ( построении плана и т.д. ), по уму в том же drupal должно бы быть 2 варианта

in ( :param )
и
in ( #суперидентификатонедопустимыйдляsqlзапроса#paramscollection )

тут же в унификации можно зайти очень далеко, в некоторых системах применяют что то вроде "Select &fields from &table &whereparam", но в этом случае все понимают что нельзя просто так что угодно в параметры передавать, такой usecase вы тоже предлагаете в jdbc/pdo встроить?


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Аноним , 16-Окт-14 15:05 
>по уму в том же drupal должно бы быть
> 2 варианта
> In ( :param )
> и
> in ( #суперидентификатонедопустимыйдляsqlзапроса#paramscollection )

хм почему то думал что in ( :param ) это из mysql -кого синтаксиса, по сути в друпале и есть второй вариант, просто с ошибкой в реализации.


"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Zontus , 16-Окт-14 14:51 
PERL DBI

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено Прохожий , 16-Окт-14 17:40 
Perl-Pg и нормальная логика SQL с хранимыми процедурами, триггерами и т.д. Неубиваемая штука. Только все больше обрастает функционалом и обрастает.

"Критическая уязвимость в системе управления web-контентом Dr..."
Отправлено chuk , 17-Окт-14 20:00 
Вот дурачье! На кой вообще было изобретать 7-ю версию, которое ничего общего не имеет с 6-й?
6-я - лучшая из Друпалов!