В выпуске (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
> DrupalЯ ни разу не удивлён. Но думал, что Джумлу вряд ли переплюнут.
Есть сервер с голой джумлой. Взламываешь? 100 баксов твои. Надо больше говори, не вопрос.
> Надо больше говори, не вопрос.Ну это смотря насколько больше :). Если вы 31337 баксов как гугля предложите - у вас появится несколько чуваков, выискивающих баги fulltime, вероятно :)
Если вас еще не взломали - это не значит, что ваша система неуязвима. Возможно вы просто нам не интересны!
> Есть сервер с голой джумлой. Взламываешь? 100 баксов твои.
> Надо больше говори, не вопрос.А теперь объясняю, как выглядит в жизни.
Некую CMS засунули на хостинг, активно залатывают, сайт наполняют контентом. Проходит год-два-три. Изначальные студент/фрилансер/студия вроде как уже и не нужны, всё устоялось, проблем особых нет. Латание плавно или скачком прекращается, продолжается наполнение. Проходит ещё какое-то время, за которое поддерживаемая ветка закрывается, переезд на следующую требует как минимум обращения к с/ф/с, подчас нетривиальной мороки (т.е. времени и денег), изредка существенной переработки части функциональности.
Так вот важно не то, сколько именно сейчас в условной "голой жумле", которая нафиг никому не упёрлась сама по себе, эксплуатируемых/известных уязвимостей. Важно то, сколько их вылезет к тем нескольким годам плюс какому-то времени, и сколько будет заткнуто за эти первые годы.
Я доступно излагаю?
Прям за меня сказал.
А имеется ли разница между 100 потенциальными уязвимостями Жумлы или одной (найденной) в CMSВАШЕЙМЕЧТЫ если результат практически один (потеря данных или что-то подобное)Уязвимости находятся регулярно во всем мало-мальски используемом. Можно поступить подобно страусу и написать свой велосипед с вдвое большим кол-вом уязвимостей, но пребывать в спокойствии, не находя сообщений о них.
Разница в том, что 50 из тех 100 могут быть известны только тем, кто ими пользуется.
Если она одна и закрыта, потенциальные злоумышленники переключатся на ПО, в котором их находят пачками.
> А имеется ли разница между 100 потенциальными уязвимостями Жумлы или одной (найденной)
> в CMSВАШЕЙМЕЧТЫ если результат практически один (потеря данных или что-то подобное)Видите ли, практика показывает, что на условную сотню найденных уязвимостей жумлы приходится условно одна найденная дырка в "CMSНАШЕЙМЕЧТЫ". И с таким "авторским коллективом" оной жумлы это как было предсказуемо годы тому, так и останется. Увы.
Про "потенциальные уязвимости" можете рассказывать "потенциальным заказчикам", кинетические уже на этом молоке обожглись и мальчикам-руководам не верят.
тут от мозга админа больше зависит, чем от скриптов.
как рассчитал безопасность системы.
> Уязвимость вызвана ошибкой в реалиpации интерфейса абстрактного доступа к БДвот, значит, как теперь называется склеивание строчек в sql запрос на коленке в пыхе
Давно пора запретить передачу SQL от клиента (middleware) к серверу (СУБД) и реализовать унифицированный интерфейс к хранимым процедурам, а внутри уже пусть городят как хотят.
чур тебя
> Давно пора запретить SQL//fixed. Bobby Tables meets John The Scripter.
Мдя, как всё тяжко то у людей...
> Мдя, как всё тяжко то у людей...Так ты марсианин???
>> Мдя, как всё тяжко то у людей...
> Так ты марсианин???А ты комменты читаешь по одному, не обращая внимания на иерархию? :)
> А ты комменты читаешь по одному, не обращая внимания на иерархию? :)Промах. Руки трясутся :))
...сколько еще таких сюрпризов на просторах любителей экономить на prepare-параметрах...
Возможно это от большой любви ко всякого рода ORM...Практически любой проект, не использующий prepare-параметры, но активно работающий с БД, имеет потенциальную возможность предоставить брешь.
> .сколько еще таких сюрпризов на просторах любителей экономить на 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.
>[оверквотинг удален]
> 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 подстановка кода не очень возможна, если только как строковое значение поля, когда тип позволяет.
Да у них просто видимо этот prepare эмулируется их же кодом.
> Да у них просто видимо этот prepare эмулируется их же кодом.Там у них "для облегчения" есть функция expandArguments которая перед pdo-ным prepare переписывает тело запроса хитрым preg_replace-ом :), более подробно не изучал :)
>[оверквотинг удален]
> 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 и тому подобным...
prepared statements так не работает, это что-то похожее по api на него. Как уже некоторые заметили, некоторые ущербные биндинги эмулируют софтварно, например почти все к mysql
Можно еще сказать, что данная ошибка есть следствие дизайна PHP, в котором нет различия между обычными и ассоциативными массивами aka хешами.
Да есть такое. Типизированные массивы и списки тогда уж. Само по себе числовые индексы или символьные не большое зло - частный случай индексов. Но это плата за универсальность. Решили сэкономить на списках и кортэжах. Но ошибка не совсем с этим связана, "палёный код", впрочем от друпала не удивительно может быть.
> Cамо по себе числовые индексы или символьные не большое зло - частный случай индексов.ни что ни у чего тут не частный случай.
> Но это плата за универсальность.
это не универсальность, а головная боль от дебиловатых разработчиков пыха.
Вы бы в патч посмотрели
или хотя бы новость внимательно прочли, чтобы понять в чем проблема и как она появилась.
> Вы бы в патч посмотрели
> или хотя бы новость внимательно прочли, чтобы понять в чем проблема и
> как она появилась.
Интересно whitehouse.gov и london.gov.uk уже пропатчили? Они когда-то громко кричали о переходе на Drupal Ж-)
Загуглите "prepared statement in" - удивитесь количеству одинаковых вопросов на stackoverflow (речь не только о PHP). Есть вообще БД API, которое поддерживает из коробки биндинг коллекций к одному плейсхолдеру? А ведь достаточно часто возникающий кейс.
> Загуглите "prepared statement in" - удивитесь количеству одинаковых вопросов на stackoverflow
> (речь не только о PHP). Есть вообще БД API, которое поддерживает
> из коробки биндинг коллекций к одному плейсхолдеру? А ведь достаточно часто
> возникающий кейс.Оно не нужно, не дело СУБД разгребать коллекции, и реализация такого алгоритма должна зависеть от ситуации а не "сейчас мы регулярками преобразуем строку запроса..."
p.s. старое утверждение про невозможность выполнения всех 3-х критериев разработки работает :)
Ну алгоритм-то зачастую один и тот же - взять список чисел/строк и впихнуть в IN выражение.Не дело СУБД - ok, но почему в тех же PDO, JDBC и т.п. API не добавить соответствующий метод? В DQL (doctrine) и HQL (hybernate) к примеру такая возможность есть (я понимаю, что это несколько более высокоуровневые штуки, но не вижу причину чтобы не реализовать это уровнем ниже - ну ведь нужная же вещь).
> Ну алгоритм-то зачастую один и тот же - взять список чисел/строк и
> впихнуть в 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 встроить?
>по уму в том же drupal должно бы быть
> 2 варианта
> In ( :param )
> и
> in ( #суперидентификатонедопустимыйдляsqlзапроса#paramscollection )хм почему то думал что in ( :param ) это из mysql -кого синтаксиса, по сути в друпале и есть второй вариант, просто с ошибкой в реализации.
PERL DBI
Perl-Pg и нормальная логика SQL с хранимыми процедурами, триггерами и т.д. Неубиваемая штука. Только все больше обрастает функционалом и обрастает.
Вот дурачье! На кой вообще было изобретать 7-ю версию, которое ничего общего не имеет с 6-й?
6-я - лучшая из Друпалов!