Тема: Уязвимость всего Социала смотрите (Движок DCMS-Social/Баги/обновления движка)

Энмеркар (24 Jul 2018 в 17:35)
Решайте это вопрос с движком а то все релизы дырявые полностью:) Это лишь один пример с кругами ещё куча чего есть в нём:D

DiZMoD (24 Jul 2018 в 17:42)
Вауууу)))) лол ржу

Энмеркар (24 Jul 2018 в 17:57)
DiZMoD, Так я тоже ржу молодец он конечно только тупа срёт:D

Мр. Форум (24 Jul 2018 в 18:55)
[red]Тему закрыл Создатель biket[/red]

Мр. Форум (24 Jul 2018 в 18:55)
[red]Тему открыл Создатель biket[/red]

Parviz (24 Jul 2018 в 20:50)
--Цитата--
Энмеркар (24 Jul 2018 в 17:35):
Решайте это вопрос с движком а то все релизы дырявые полностью:) Это лишь один пример с кругами ещё куча чего есть в нём:D
----------
Энмеркар, где уязвимость то?

Энмеркар (24 Jul 2018 в 21:09)
Parviz, В bbcode.php а именно в этом коде 
	// if ($set[\'bb_size\'])$bbcode[\'/[size=([0-9]+)](.+)[/size]/isU\']=\'<span style=\"font-size:$1px;\">$2</span>\'; решение простое убрать его к чёрту я его убрал и всё стало нормально:)

SimptomFD (25 Jul 2018 в 01:18)
Я чёт тоже не допер в каком месте это уязвимость.
Размер шрифта обычный.
А во избежание подобных случаев можно просто прописать условия на проверку с минимальным и максимальным размером.

kantry (25 Jul 2018 в 03:37)
--Цитата--
Энмеркар (24 Jul 2018 в 21:09):
Parviz, В bbcode.php а именно в этом коде 
	// if ($set[\'bb_size\'])$bbcode[\'/[size=([0-9]+)](.+)[/size]/isU\']=\'<span style=\"font-size:$1px;\">$2</span>\'; решение простое убрать его к чёрту я его убрал и всё стало нормально:)
----------
Энмеркар, не надо там ничего убирать, квантификатор замени, и все.
/[size=([0-9][b]{1,2}[/b])](.+)[/size]/

Энмеркар (25 Jul 2018 в 10:26)
--Цитата--
kantry (25 Jul 2018 в 03:37):
Энмеркар, не надо там ничего убирать, квантификатор замени, и все.
/[size=([0-9][b]{1,2}[/b])](.+)[/size]/
----------
kantry, Здорова,, а смысл какой!?.дум. проверь сам свою теорию!:) Я за неё не чего хорошего и плохого скажу,, моё дело было это показать что баг есть во всех движках и релизах это ещё с 64 движка идёт,, на спейсе уже давно от этого тега отказались и сайтом там всё нормик и без него:)

SimptomFD (25 Jul 2018 в 17:09)
Вот вам решение данной проблемы:
Открываем файл /sys/fnc/bbcode.php и ищем строку:
<?
if ($set[\'bb_size\'])$bbcode[\'/[size=([0-9]+)](.+)[/size]/isU\']=\'<span style=\"font-size:$1px;\">$2</span>\';
?>
Удаляем ее.
Спускаем чуть ниже по коду и ищем код:
<?
if (count($bbcode))$msg= preg_replace(array_keys($bbcode), array_values($bbcode), $msg);
?>
После этого кода прописываем код:
<?
if ($conf[\'bb_size\'])
{
    $msg = preg_replace_callback(\'/[size=([0-9]+)](.+)[/size]/isU\', \'bb_size\', $msg);
}
?>
Внизу файла, после закрытия функции BBcode() пишем код:
<?
function bb_size($text)
{
    if ($text[1] < 10 || $text[1] > 20)
    {
        $text[1] = 14;
    }
    return \'<span style=\"font-size: \'.$text[1].\'px;\">\'.$text[2].\'</span>\';
}
?>
Где 10 -минимальный размер, 20 - максимальный размер, 14 - размер по умолчанию.
Само условие редактируем по своему усмотрению.
Профит!

kantry (26 Jul 2018 в 06:58)
Энмеркар, Привет! Чего мне проверять то свои теории?.ржу. и при чем тут спейс?
Я специально написал простой регексп, что бы понятно было как такие вещи решаются. Вот еще понятнее, только ты уж сам \"теорию\" проверяй))
$bbcode[\'/[size=([5-9]|(?# от 5 до 9 или)[1-2][d]|(?# любая цифра или)3[0-2](?# не более 2-х))](.+)[/size]/isU\'] = \'<span style=\"font-size:$1px;\">$2</span>\';
Можешь просто скопировать к себе в бб-коды, и проверить.
		[i]Добавлено:[/i]
		SimptomFD, колбек конечно хорошо, меня даже заинтересовало, но.. не профит))

SimptomFD (26 Jul 2018 в 08:32)
kantry, Никто и не говорит, что мое решение единственное) Есть можешь предложить, предлагай, пользователи смогут выбирать что им по душе. В том и смысл поддержки.
В твоём варианте расброс размеров большой.
Я не любитель данных мероприятий, но да, можно строчно его впаять)

Сибирский (26 Jul 2018 в 09:44)
А уязвимость то ту в чем? Это типичный баг

kantry (26 Jul 2018 в 10:37)
Сибирский, да это даже и не баг, например с допотопного телефона никто и не заметит что текст огромный))) Нагрузки ни на сервер, ни на браузер никакой, но то что это неприятно - факт, когда то часто баловался уменьшая шрифт до 1px))
		[i]Добавлено:[/i]
		[b]SimptomFD[/b], я там от 5 до 32-х написал регулярку,эту переписал для движка с которым иногда вожусь. 
Для остальных попробую пояснить.. | - это логическое [b]или[/b]
([green][6-9][/green]|[green][1-6][d][/green]|[green]7[0-2][/green])
[green][6-9][/green] от 6 до 9
или
[green][1-6][d][/green] от 10 до 69
или
[green]7[0-2][/green] - числа от 70 до 72
Все, это стандартный диапазон размера шрифтов.
П.С. Санитарию за юзерами делать надо на входе, а не на выходе! То есть, проверять надо было что он написал до того как это в базу попало.

Энмеркар (26 Jul 2018 в 16:59)
kantry, Здорова,, я не спорю просто за себя говорю что я этим не пользуюсь и знаю что люди на сайтах этим не пользуются,, так что мне оно ненужно и я просто закоментил его,, и у меня другой в этом смысл ещё,, коль появился такой человек который этим пользуется выловить его на этом сразу с его стилем это делать,, а не проморгать его на сайте:)

Parviz (31 Jul 2018 в 22:18)
[size=100]test[/size]

SimptomFD (1 Aug 2018 в 19:26)
Parviz, Не страдай фигнёй.

Parviz (4 Aug 2018 в 21:22)
[size=10]test2[/size]

eaglex (6 Aug 2018 в 18:35)
[size=9]тест (удалил (было 999999999999 и был бардак)[/size]

Мр. Форум (11 Aug 2018 в 17:47)
[red]Тему закрыл Модератор Saint[/red]

Источник: http://dcms-social.ru/forum/13/3/62501/