Недостаточная фильтрация входящих данных

Проблема: Пользователю которому разрешена загрузка файлов на сервер (не картинок), может выйти за пределы разрешенной папки загрузки, а если он имеет администраторский аккаунт на сайте, то и повредить данные скрипта.

Ошибка в версии: Все версии

Степень опасности: Средняя (Высокая при наличии администраторского аккаунта на сайте)

Как исправить в полной новости...
Для исправления откройте файл engine/inc/files.php и найдите:
$serverfile = trim( htmlspecialchars( strip_tags( $_POST['serverfile'] ) ) );


замените на:

if ($member_id['user_group'] == 1) $serverfile = trim( htmlspecialchars( strip_tags( $_POST['serverfile'] ) ) ); else $serverfile = '';

        if ( $serverfile != '' ) {

            $serverfile = str_replace( "\\", "/", $serverfile );
            $serverfile = str_replace( "..", "", $serverfile );
            $serverfile = str_replace( "/", "", $serverfile );
            $serverfile_arr = explode( ".", $serverfile );
            $type = totranslit( end( $serverfile_arr ) );
            $curr_key = key( $serverfile_arr );
            unset( $serverfile_arr[$curr_key] );

            if ( in_array( strtolower( $type ), $allowed_files ) )
                $serverfile = totranslit( implode( ".", $serverfile_arr ) ) . "." . $type;
            else $serverfile = '';

        }

        if( $serverfile == ".htaccess") die("Hacking attempt!");


Откройте файл engine/classes/thumb.class.php и найдите:

$this->img['des'] = imagecreatetruecolor( $this->img['lebar_thumb'], $this->img['tinggi_thumb'] );


и добавьте выше:
        if ($this->img['lebar_thumb'] < 1 ) $this->img['lebar_thumb'] = 1;
        if ($this->img['tinggi_thumb'] < 1 ) $this->img['tinggi_thumb'] = 1;

Другие новости по теме:

Категория: Баги ----- Просмотров: 3339 ----- Комментариев: 36

Вернуться

  • Группа: User
  • ICQ: --
  • Регистрация: 10.10.2009
  • Комментариев: 25
  • Публикаций: 4
^
спасибо, залатал
  • skorge

  • 8 июня 2010 09:32
  • Группа: User
  • ICQ: 233206672
  • Регистрация: 7.04.2009
  • Комментариев: 16
  • Публикаций: 0
^
спасибо.
  • Группа: r00t Admin
  • ICQ: --
  • Регистрация: 11.12.2007
  • Комментариев: 411
  • Публикаций: 389
^
Не прошло и дня. Уязвимость номер №2 за этот месяц. Ещё есть около 2 - 3, а то и больше, а это только капля в море.

--------------------
  • Группа: User
  • ICQ: 485348926
  • Регистрация: 17.06.2009
  • Комментариев: 20
  • Публикаций: 0
^
Спасибо

--------------------
  • pav

  • 8 июня 2010 12:25
  • Группа: User
  • ICQ: --
  • Регистрация: 19.01.2009
  • Комментариев: 253
  • Публикаций: 9
^
спалили сцуко wink
  • Группа: r00t Admin
  • ICQ: --
  • Регистрация: 11.12.2007
  • Комментариев: 411
  • Публикаций: 389
^
Цитата: BlackSmoke
спалили сцуко

wink

--------------------
  • Группа: User
  • ICQ: --
  • Регистрация: 9.02.2009
  • Комментариев: 64
  • Публикаций: 20
^
ниче страшного! дырки будут всегда!) wink
и всегда будет kzpromo ef

--------------------
  • myzikk

  • 8 июня 2010 13:44
  • Группа: User
  • ICQ: --
  • Регистрация: 24.12.2009
  • Комментариев: 42
  • Публикаций: 3
^
nugumanov,
Который всегда их обнаружит и грамотно использует kzpromo nh
  • Группа: Гости
  • ICQ: --
  • Регистрация: --
  • Комментариев: 0
  • Публикаций: 0
^
Я не понимаю, за что вы говорите админу спасибо? nj Что он спалил? Фикс этой дыры (как и прошлой) выложил Целсофт на оф. сайте вчера вечером, а тут эта новость появилась сегодня.

Лично я пока вижу, что админ может только в комментах к каждой пофиксиной ошибке скрипта писать, что он знает ещё штуки три, а то и больше дырок. Ну так сайт посвящён ДЛЕ, так? Ну так возьми и скажи что это за дыры, чтоб Целсофт их залатал, а то строишь из себя, nj неизвестно кого...

Хочешь реально что-то сделать полезного? Ну так спали хотябы парочку дыр Целсофту, а то только ветер гоняешь и понторезку свою зря мучаешь ng
  • Группа: User
  • ICQ: --
  • Регистрация: 9.02.2009
  • Комментариев: 64
  • Публикаций: 20
^
да пошел нахyй целсофт, палить ему что-то
я не успокоюсь пока не сломаю целсофт wink

кто со мною, те уже давно залатались! bg
нет предела совершенству! wassat

--------------------
  • Группа: r00t Admin
  • ICQ: --
  • Регистрация: 11.12.2007
  • Комментариев: 411
  • Публикаций: 389
^
Цитата: Kino-man
Ну так возьми и скажи что это за дыры, чтоб Целсофт их залатал

В том, то и дело, ходят слухи, что существуют ещё около 2 - 3, может и больше уязвимостей, о которых я не знаю вообще. al Все морозяться, bn специально не палят уязвимости. А вот данная новость это вчерашний взлом mastermind.pp.ua/datalife-engine, тобишь удаление файла dbconfig.php

Цитата: nugumanov
кто со мною, те уже давно залатались!

в каком месте? bm

Статью которую я скопипастил с какого-то форума "Обзор уязвимостей DLE (DataLife Engine)", можно было найти месяц назад в интернете, а то и больше. Ведь именно со статьи всё началось, и складывается впечатление, что Celsoft-у всё равно, есть в движке уязвимости или нет. А почему бы не контролировать разработчику все хакерские форумы, всё то, что на них пишут?! Почему бы сразу свой php код не проверять, не исправлять. Мы ведь здесь не собрались, только обгаживать DLE и его разработчика, каждый занимается вебмастерством, в свою очередь клиенты последних покупают только лицензионный DLE, а не nulled. Скажем мы ж не звери, чтобы везде сайты пихать на nulled, а почему бы и нет, чтобы была лицензия?! То же хорошо, зато ты сможешь лично потрясти разработчика, за какие-либо баги и уязвимости. В данном случае, посетители подобных сайтов, как mastermind.pp.ua/datalife-engine находят баги, уязвимости, и используют их на своё усмотрение.
Я бы на месте Celsoft предложил ввести очень достойную плату за выявление всяких "дырок".

--------------------
  • Группа: User
  • ICQ: --
  • Регистрация: 9.02.2009
  • Комментариев: 64
  • Публикаций: 20
^
в каком месте?

в engine/inc/files.php wink

--------------------
  • Группа: Гости
  • ICQ: --
  • Регистрация: --
  • Комментариев: 0
  • Публикаций: 0
^
nugumanov,
Его же сегодня обновили.
  • Группа: User
  • ICQ: 288569677
  • Регистрация: 21.01.2008
  • Комментариев: 703
  • Публикаций: 245
^
Kino-man,
Что?
А ты чё не ищеш уязвимостей?
А отекуда Целкин узнал о уязвимости?
Сам нашол?
А хуя, он сцука все моды и хаки прикручивает.
Начать можно с чего угодно, но вы и не кумекаете, Правила сайта я лично покупал за 1 у.е. потом Целкин призрел и себе прикрутил.
Лицуху Тегов от Коляна я тож пакупал за 40 у.е. пожизненую и шо?
А Целкин себе типа прикрутил, до того же кривые, как оно и сам, так что не{censored} тут {censored} шпана и указывать нам, что мы должны, кому и за что.
Ненравится, пиздуйте к Целкину, лижите ему жоППу за подареную типа лицуху, тестите, здавайте ему и долаживайте о том, что тут или на таких же как этот Ресурсе сказано, тока не парте нам тут мозг и не огорчайте и не до этого ЗЛЫХ Админов, потому как можно нарватся реально и услушать о себе того, чего вы так боитеся признать и такими помрёте.
nq

--------------------
  • neoks

  • 8 июня 2010 19:09
  • Группа: User
  • ICQ: --
  • Регистрация: 20.07.2009
  • Комментариев: 556
  • Публикаций: 121
^
Kino-man,
Все палится людьми с нуледа и античата, и после этого целкин копирует эту инфу и делает патч даже не указав атора.
К примеру последние три дыки были пропалены на нуледе и античате а потом уже через месяц как эти дырки начали разносить на форумах без хайда, подключился и целкин сделав заклатку скопировав всю инфу о закрытие да и все.

P.S: Что сделал целка за все время существования DLE кроме того что скомуниздил пару модулей и переделал CMS под себя ?
1) Распиарил и дал популярность DLE он ? (нет)
2) Выпустил куча модулей и хаков для DLE он ? (нет)
3) Сделал тысячи шаблонов, оригиналы рипы и т.д он ? (нет)

За шесть лет целкин сделал только 15 модулей в DLE но не более.
А теперь еше и права качает {censored} (вот убрать все такие сайты про DLE и что останется для DLE ? 15 модулей за 6 лет ? ну да очень много)

--------------------
  • Группа: User
  • ICQ: --
  • Регистрация: 22.05.2010
  • Комментариев: 12
  • Публикаций: 0
^
Массово перестали загружатся фотки, по одной загружает, а при массовом добавльть не хочет server error. Как исправить, или что может в админке настроить. Версия 8.5
  • Voven

  • 8 июня 2010 21:10
  • Группа: User
  • ICQ: --
  • Регистрация: 19.07.2008
  • Комментариев: 13
  • Публикаций: 1
^
Господа!
Так может есть смысл закончить ругать Целсофт и забацать свой двиг а-ля дле.ин.уа, открытый исходный код типа как в джумле и т.д. Зачем делать модули для Целсофта

--------------------
  • Группа: User
  • ICQ: --
  • Регистрация: 9.02.2009
  • Комментариев: 64
  • Публикаций: 20
^
для целсофта никто модули не делает, модули делают для народа
а целсофт только ворует и присваивает чужое авторство fellow

--------------------
  • Группа: r00t Admin
  • ICQ: --
  • Регистрация: 11.12.2007
  • Комментариев: 411
  • Публикаций: 389
^
Модуль для целсофта - "прямые руки и трезвый ум" wink
Разве в джумле и друпале нету дырок? Есть, но движки не так популярны среди вареза, музла и всякой порнухи. Я уже не говорю за вордпрес, который нагружает сервер как больной, да и смотря какая ОС. Понятное дело, что DLE, меньше всех грузит сервер, и "хорошо индексируеться". Это называеться поставили разработчика перед фактом, что есть то-то и то-то, и нужно каким-то образом исправить. С одной стороны затишье перед бурей. Хотя с другой стороны, если разраб у себя на сайте все дырки закрыл, значит он о них и так знает, либо использует движок старой версии. Но это опять же не факт, я больше считаю, что дырки закрыл. Ну а что ещё можно думать?! Ни кто никого не собираеться упрекать. И я считаю, что о уязвимостях он знает. Если они есть, можно слить базу, шаблон, какая-то выгода в этом есть. Для нас хорошо и для разраба хорошо. Хотя это только мысли.
Скажем, ещё давно шла волна варезников, даже удивительно было, у всех шаблоны в три колонки, как положено, хостеру то пофиг. А нагрузки на сервер ноль, зашибись. Вот с этого всё и началось, вся раскрутка и известность.

--------------------
  • NoSay

  • 8 июня 2010 22:23
  • Группа: Модератор
  • ICQ: --
  • Регистрация: 12.11.2009
  • Комментариев: 81
  • Публикаций: 10
^
Благодарю.

--------------------
  • neoks

  • 8 июня 2010 22:43
  • Группа: User
  • ICQ: --
  • Регистрация: 20.07.2009
  • Комментариев: 556
  • Публикаций: 121
^
Baggio,
Цитата: Baggio
Хотя с другой стороны, если разраб у себя на сайте все дырки закрыл, значит он о них и так знает, либо использует движок старой версии.

Я только об этом подумал wink
Все дырки даже если только нашли не пашут на оф сайте у целки, зато у остальных владельцев лицензии и нуледа работают без проблем. angry


Voven,
Уже так и сделали (завтра будет релиз SL-CMS 2 wink )

--------------------
  • Группа: User
  • ICQ: --
  • Регистрация: 9.02.2009
  • Комментариев: 64
  • Публикаций: 20
^
ждем новых вкусняшечек)))

--------------------
  • Voven

  • 8 июня 2010 22:59
  • Группа: User
  • ICQ: --
  • Регистрация: 19.07.2008
  • Комментариев: 13
  • Публикаций: 1
^
Цитата: neoks
Уже так и сделали (завтра будет релиз SL-CMS 2 )


Да в принципе ничего смешного, я не предлагал джумлу сделать. Давайте будем тестить SL-CMS nh

--------------------
  • Группа: r00t Admin
  • ICQ: --
  • Регистрация: 11.12.2007
  • Комментариев: 411
  • Публикаций: 389
^
Цитата: nugumanov
ждем новых вкусняшечек)))

В новом релизе 9.0 будет в два раза больше дырок. wink Хотя в какой-то степени благодарю Celsoft за то, что не закрыл дырки lol посливают больше шаблонов. bi

--------------------
  • Lion__

  • 9 июня 2010 02:32
  • Группа: Гости
  • ICQ: --
  • Регистрация: --
  • Комментариев: 0
  • Публикаций: 0
^
на днях и я вылажу

Соглашусь с neoks, целка никогда не указывала кто нашёл баг, всё тихонько ждала и исправляла, на офф форуме подымал я эту тему
  • SaD

  • 9 июня 2010 08:42
  • Группа: User
  • ICQ: --
  • Регистрация: 20.02.2009
  • Комментариев: 303
  • Публикаций: 15
^
Соглашусь с neoks, целка никогда не указывала кто нашёл баг, всё тихонько ждала и исправляла, на офф форуме подымал я эту тему

1. Найти автора уязвимости порой может быть проблематично
2. Если бы автор уязвимости захотел стать популярным, то сам отправил бы новость о уязвимости Целсофту.

С другой стороны мне не ясно, почему не назначит вознаграждение за найденные уязвимости, как сказал Baggio

Тому что уязвимости не работают на оф.сайте, тоже есть простое объяснение. Я не думаю что Целсофт использует у себя обычную версию ДЛЕ и обновляет её запуская update.php wink . Скорее всего у него просто стоит полностью модифицированная DLE, где от самой DLE осталось очень мало. nh

--------------------
  • Группа: r00t Admin
  • ICQ: --
  • Регистрация: 11.12.2007
  • Комментариев: 411
  • Публикаций: 389
^
Цитата: SaD
Скорее всего у него просто стоит полностью модифицированная DLE, где от самой DLE осталось очень мало.

О, кстати тоже вариант. wink

--------------------
  • Lion__

  • 9 июня 2010 12:29
  • Группа: Гости
  • ICQ: --
  • Регистрация: --
  • Комментариев: 0
  • Публикаций: 0
^
SaD
Здесь ты не прав, писал целке об 1 баге, в ответ - вы не клиент dle. В релизе 8.5 было исправлен баг, поднял тему на форуме, меня просто начали игнорить

Baggio
Проверь новости)))
  • Группа: r00t Admin
  • ICQ: --
  • Регистрация: 11.12.2007
  • Комментариев: 411
  • Публикаций: 389
^
Цитата: Lion__
SaD
Здесь ты не прав, писал целке об 1 баге, в ответ - вы не клиент dle. В релизе 8.5 было исправлен баг, поднял тему на форуме, меня просто начали игнорить

Пушо целка {censored}а. am Ему не по{censored}, клиент или не клиент.

--------------------
  • Lion__

  • 9 июня 2010 13:48
  • Группа: Гости
  • ICQ: --
  • Регистрация: --
  • Комментариев: 0
  • Публикаций: 0
^
Ему не по{censored}, клиент или не клиент

Согласен

Baggio
не по теме, но ты задолбал вылаживать на ексе дле шоп wink
  • Группа: r00t Admin
  • ICQ: --
  • Регистрация: 11.12.2007
  • Комментариев: 411
  • Публикаций: 389
^
Цитата: Lion__
Baggio
не по теме, но ты задолбал вылаживать на ексе дле шоп wink

То старая версия. wink старую можно nh

--------------------
  • Lion__

  • 9 июня 2010 14:46
  • Группа: Гости
  • ICQ: --
  • Регистрация: --
  • Комментариев: 0
  • Публикаций: 0
^
старую можно

Да со старой всё равно, просто потом пишут мне в аську и просят помощи как сделать то, как сделать то. Ужас!
  • Группа: r00t Admin
  • ICQ: --
  • Регистрация: 11.12.2007
  • Комментариев: 411
  • Публикаций: 389
^
Так давай 3.3 залью lol вообще писать в аську перестанут lol (шутка конечно)

--------------------
  • Lion__

  • 9 июня 2010 15:07
  • Группа: Гости
  • ICQ: --
  • Регистрация: --
  • Комментариев: 0
  • Публикаций: 0
^
Baggio
Неее)
  • Группа: Гости
  • ICQ: --
  • Регистрация: --
  • Комментариев: 0
  • Публикаций: 0
^
nugumanov,
целкософт УГ! когда поломаеш, скажеш ))
  • Группа: Гости
  • ICQ: --
  • Регистрация: --
  • Комментариев: 0
  • Публикаций: 0
^
Цитата: Datalife
целкософт УГ! когда поломаеш, скажеш ))

Когда поломает -- все узнают=))