[SQL Inj][DLE] Патч безопасности

12
SEOCondition
На сайте с 03.10.2011
Offline
118
3976

Собственно, на офф.сайте информации пока нет, однако на ачате уже давно идёт обсуждение баги с неопределенностью переменной $_TIME в некоторых случаях.

Что можно сделать? Перевести юзера в группу 1 т.е сделать администратором => SQL Inj.

#1 Как лечить?

Открыть файлик sitelogin.php (находится в папке engine/modules/)

Найти:

if( $is_logged ) { // проверка авторизации

Сразу после этого дописать

if(!$_TIME) $_TIME = time() + ($config['date_adjust'] * 60); // определение переменной $_TIME

Пофиксили? Читаем дальше :)

#2 В каких версиях SQL Inj?

Посмотрел только по клиентским сайтам, а это от 7х и до 9.7 - есть во всех версиях из этого диапазона.

[⚡] PBN под ключ, обслуживание, консультации — обращайтесь в личку
WebAlt
На сайте с 02.12.2007
Offline
266
#1

Слышу звон, не зная где он. ТС, то что вы скопипастили хрен знает откуда не устраняет проблему, официальный патч вышел два месяца назад и здесь уже выкладывался, причём уязвимость актуальна только при включенном register_globals:

http://dle-news.ru/bags/v97/1538-patchi-bezopasnosti-dlya-versiy-97-i-nizhe.html

SEOCondition
На сайте с 03.10.2011
Offline
118
#2
WebAlt:
Слышу звон, не зная где он. ТС, то что вы скопипастили хрен знает откуда не устраняет проблему, официальный патч вышел два месяца назад и здесь уже выкладывался, причём уязвимость актуальна только при включенном register_globals:
http://dle-news.ru/bags/v97/1538-patchi-bezopasnosti-dlya-versiy-97-i-nizhe.html

если бы было так - не выкладывал бы.

Звонов никаких не слышу - я в норме :)

Условия

register_globals on

magic_quotes_gpc off

если уж на то пошло :)

Во всяком случае, патчить Вас никто не заставляет.

 $db->query( "UPDATE " . USERPREFIX . "_users set hash='" . $hash . "', lastdate='{$_TIME}', logged_ip='" . $_IP . "' WHERE user_id='{$member_id['user_id']}'" ); 

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

P.S: спасибо за ссылку, но я её по рассылке получил давненько уже.

siv1987
На сайте с 02.04.2009
Offline
427
#3
SEOCondition:
если бы было так - не выкладывал бы.

А что "не так" в патче, и "так" у вас?

SEOCondition:
P.S: спасибо за ссылку, но я её по рассылке получил давненько уже.

Ну раз получили, зачем копипастить бояны?

se_noob
На сайте с 10.10.2006
Offline
110
#4

на офф сайте это было в ноябре

$_TIME = time () + ($config['date_adjust'] * 60);

и никаких условий не нужно.

H
На сайте с 17.10.2012
Offline
46
#5
magic_quotes_gpc off

Хрен с ним с globals, но отключить gps это как ключи под ковриком оставить.

se_noob
На сайте с 10.10.2006
Offline
110
#6
hume:
Хрен с ним с globals, но отключить gps это как ключи под ковриком оставить.

имхо, все зависит от того, как обрабатываются переменные внутри скрипта

izbushka
На сайте с 08.06.2007
Offline
110
#7
hume:
Хрен с ним с globals, но отключить gps это как ключи под ковриком оставить.

Ничего что

Данная возможность была помечена УСТАРЕВШЕЙ начиная с версии PHP 5.3.0 и была УДАЛЕНА в версии PHP 5.4.0.
H
На сайте с 17.10.2012
Offline
46
#8
Данная возможность была помечена УСТАРЕВШЕЙ начиная с версии PHP 5.3.0 и была УДАЛЕНА в версии PHP 5.4.0.

Убрать то убрали, вот только смысл в этом какой? Теперь тоже самое только уже ручками придется делать. Имхо, тупизм.

Начиная с версии 5.3 много чего стали убирать, не сказать что это повод для гордости, в итоге большинство сидит на проверенном 5.2

siv1987
На сайте с 02.04.2009
Offline
427
#9
hume:
Убрать то убрали, вот только смысл в этом какой?

Смысл в том, что это нужно делать в скриптах, и в тех случаях когда это нужно, а не на уровне php.

maestro.vic
На сайте с 01.11.2010
Offline
65
#10

SEOCondition,

Попросил celsoft прокомментировать ваше первое сообщение. Привожу цитату того что мне ответили:

Патч для этого выпускался два месяца назад http://dle-news.ru/bags/v96/1538-patchi-bezopasnosti-dlya-versiy-97-i-nizhe.html

Если вы для версии 9.7 раннее не ставили патчи, то для нее вам необходимо поставить патч http://dle-news.ru/bags/v97/1549-patchi-bezopasnosti-dlya-versii-97.html

Данный комментарий находится на этой странице, внизу.

Значит все таки ошибки были, раз через час после моей просьбы на официальном сайте появилась новость http://dle-news.ru/bags/v97/1549-patchi-bezopasnosti-dlya-versii-97.html с патчем.

Спасибо за создание данной темы!

12

Авторизуйтесь или зарегистрируйтесь, чтобы оставить комментарий