Форум Сайтостроение Веб-строительство

Опасен ли данный скрипт?

12
M4
На сайте с 06.01.2013
Offline
8
manual4
1286

Есть сайт на DLE 9.7, есть скрипт который выводит количество новостей напротив категории, но сомневаюсь в его безопасности.

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

Вот как это все выглядит :

Установка:
1) Открыть файл index.php в корне сайта,найти:
$tpl->set ( '{AJAX}', $ajax );
 
 
 
2)Выше этого кода вставить ниже код:
/*********Главное меню**********/ $sql_result = $db->query("select ct.id, (select count(*) from ". PREFIX ."_post ps where (ps.category REGEXP concat( '^(' ,ct.id, ',)' ) or ps.category REGEXP concat ( '(,' ,ct.id, ',)' ) or ps.category REGEXP concat( '(,' ,ct.id, ')$' ) or ps.category=ct.id) AND (DATE_FORMAT(date, '%Y%m%d') =CURDATE()) AND approve = '1') as new_, (select count(*) from ". PREFIX ."_post ps where (ps.category REGEXP concat( '^(' ,ct.id, ',)' ) or ps.category REGEXP concat ( '(,' ,ct.id, ',)' ) or ps.category REGEXP concat( '(,' ,ct.id, ')$' ) or ps.category=ct.id) AND approve = '1') as allnews_ from ". PREFIX ."_category ct" ); $count_news_of_category = ''; while($row = $db->get_row($sql_result)) { if( $row['new_'] == '0') {$news_now = '';} else {$news_now = '+'.$row['new_'].' | ';}; $t = '[ '.$news_now.$row['allnews_'].' ]'; $tpl->set('{count_categ_'.$row['id'].'}', $t); $count_news_of_category .= $row['id'].'#'.$t.'|'; } /************************************/
 
 
В само меню вставляем (обычно находится в вашем шаблоне/main.tpl):
{count_categ_3}
Ш
На сайте с 29.12.2012
Offline
45
Шварц
#1

В SQL-запрсе не может быть дыры-) Дыра может быть в организации доступа к БД - непарамеризованные запросы и т.п. В Вашем случае - обычный SELECT-запрс к БД, ничего в нем опасного нет. Плохо, что запросы в движке, видимо, непараметризованы - но на опасность/безопасность приведенного конкретного запроса это никак не влияет.

M4
На сайте с 06.01.2013
Offline
8
manual4
#2

Скажите пожалуйста - большую ли нагрузку дает такого рода скрипт? На сайте 40 категорий и к каждой он дает значение

И если возможно подскажите к кому или куда обратится чтоб проверить движок на дырки и "заклеить" их

Garin33
На сайте с 31.08.2009
Offline
169
Garin33
#3
manual4:
И если возможно подскажите к кому или куда обратится чтоб проверить движок на дырки и "заклеить" их

В первую очередь купить лицензию (сейчас наверняка нулленая версия) и поставить все рекомендуемые обновления. Все сторонние хаки, плагины, которые вы качаете не с официального сайта потенциально могут быть уязвимы.

Качественный аудит кода будет стоить дорого (от сотен $), сомневаюсь, что вы готовы столько платить.

Как выйти из этого положения - использовать cms с открытым кодом, WP, Drupal и т.д., где нет таких проблем, и 99% необходимых плагинов, модулей качаются с сайтов CMS.

Потому что Drupal - это круто.
S
На сайте с 06.08.2008
Offline
130
spiderbuber
#4

ТС, проверка на уязвимости будет стоить от 200$ (минимум). Нулл лучше юзать от мид-тима с официального сайта и все левые модули удали.

дани мапов
На сайте с 06.09.2012
Offline
204
дани мапов
#5
manual4:
Скажите пожалуйста - большую ли нагрузку дает такого рода скрипт? На сайте 40 категорий и к каждой он дает значение

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

Написание и доработка скриптов (PHP, MySQL, JavaScript, jQuery) (/ru/forum/811046)
WebAlt
На сайте с 02.12.2007
Offline
251
WebAlt
#6
spiderbuber:
Нулл лучше юзать от мид-тима с официального сайта...

Совет, ну просто охрененный.

M4
На сайте с 06.01.2013
Offline
8
manual4
#7

То есть вы считаете что нулед от mid - team действительно хорош?

дани мапов
На сайте с 06.09.2012
Offline
204
дани мапов
#8

нулед от mid - team - это наименьшее из зол :-)

Plazik
На сайте с 29.07.2008
Offline
161
Plazik
#9
manual4:
То есть вы считаете что нулед от mid - team действительно хорош?

Прочитайте например про "чистые" нуледы с другого сайта http://safesearch.ya.ru/replies.xml?item_no=322

И вам уже выше дали совет, не используйте код (сам движок и дополнения), скаченный непонятно с каких сайтов.

Используйте код только с официального сайта.

Trafikgon
На сайте с 24.10.2008
Offline
174
Trafikgon
#10

Купите лицензию и радуйтесь.

. (#)
12

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