Помогите в коде !!!

22 Окт 2014
Где же здесь дыра?
Потому что если ввести в адресную строку "><sсript>alеrt()</sсript> то выводит кусок кода в нажатии кнопки, капец полный, эта фигня в дневниках, plugins/notes/index.php?sort=c,t двиг 1.8.0 социал.

<?
include_once '../../sys/inc/start.php';
include_once '../../sys/inc/compress.php';
include_once '../../sys/inc/sess.php';
include_once '../../sys/inc/home.php';
include_once '../../sys/inc/settings.php';
include_once '../../sys/inc/db_connect.php';
include_once '../../sys/inc/ipua.php';
include_once '../../sys/inc/fnc.php';
include_once '../../sys/inc/user.php';
/* Бан пользователя */
if (isset($user) && mysql_result(mysql_query("SELECT COUNT(*) FROM `ban` WHERE `razdel` = 'notes' AND `id_user` = '$user[id]' AND (`time` > '$time' OR `view` = '0' OR `navsegda` = '1')"), 0)!=0)
{
header('Location: /ban.php?'.SID);exit;
}
$set['title']='Дневники';
include_once '../../sys/inc/thead.php';
title();
aut(); // форма авторизации
echo "<div id='comments' class='menus'>";
echo "<div class='webmenu'>";
echo "<a href='index.php' class='activ'>Дневники</a>";
echo "</div>";

echo "<div class='webmenu last'>";
echo "<a href='dir.php'>Категории</a>";
echo "</div>";

echo "<div class='webmenu last'>";
echo "<a href='search.php'>Поиск</a>";
echo "</div>";
echo "</div>";
if (isset($_GET['sort']) && $_GET['sort'] =='t')$order='order by `time` desc';
elseif (isset($_GET['sort']) && $_GET['sort'] =='c') $order='order by `count` desc';
else $order='order by `time` desc';
if(isset($user))
{
echo'<div class="foot">';
echo "<a href="user.php">Мои дневники</a> | ";
echo "<a href="add.php">Создать дневник</a>";
echo '</div>';
}
echo'<div class="foot">';
if (isset($_GET['sort']) && $_GET['sort'] =='t'){
echo"<b>Новые</b> | <a href='?sort=c'>Популярные</a>n";
}elseif (isset($_GET['sort']) && $_GET['sort'] =='c'){
echo"<a href='?sort=t'>Новые</a> | <b>Популярные</b>n";
}else{
echo"<b>Новые</b> | <a href='?sort=c'>Популярные</a>n";
}
echo '</div>';
$k_post=mysql_result(mysql_query("SELECT COUNT(*) FROM `notes` "),0);
$k_page=k_page($k_post,$set['p_str']);
$page=page($k_page);
$start=$set['p_str']*$page-$set['p_str'];
$q=mysql_query("SELECT * FROM `notes` $order LIMIT $start, $set[p_str]");
echo "<table class='post'>n";
if ($k_post==0)
{
echo "  <div class='mess'>n";
echo "Нет записейn";
echo "  </div>n";
}
while ($post = mysql_fetch_assoc($q))
{
/*-----------зебра-----------*/
if ($num==0)
{echo "  <div class='nav1'>n";
$num=1;
}elseif ($num==1)
{echo "  <div class='nav2'>n";
$num=0;}
/*---------------------------*/
echo "<img src='/style/icons/dnev.png' alt='*'> ";
echo "<a href='list.php?id=$post[id]'>" . text($post['name']) . "</a>";
echo " <span style='time'>(".vremja($post['time']).")</span>n";
$k_n= mysql_result(mysql_query("SELECT COUNT(*) FROM `notes` WHERE `id` = $post[id] AND `time` > '$ftime'",$db), 0);
if ($k_n!=0)echo " <img src='/style/icons/new.gif' alt='*'>";
echo "  </div>n";
}
echo "</table>n";
if (isset($_GET['sort'])) $dop="sort=$_GET[sort]&";
else $dop='';
if ($k_page>1)str('?'.$dop.'',$k_page,$page); // Вывод страниц
include_once '../../sys/inc/tfoot.php';
?>

Статистика WMID автора:

203250180600 BL

Претензии | Отзывы

Комментарии:

Hack

26 Окт 2014
Автор темы
PluginS, чувак, я это понял только когда еще начал смотреть код, что незащищенный запрос GET стоит мля...
Как правильно профильтровать чтобы небыло ничего плохого?
______________________
уф

PluginS

25 Окт 2014
А разве не ясно, где?
if (isset($_GET['sort'])) $dop="sort=$_GET[sort]&";
else $dop='';

Saint

25 Окт 2014
пассивная xss , просто безвредная ))
ну можешь для внушения что это поможет профильтровать htmlentities()
______________________
Я есть сущность

Hack

25 Окт 2014
Автор темы
RuDati, не поместится, создай для меня тему в форуме.
______________________
уф

RuDati 25 Окт 2014
И вообще, для начала сделаем так:
[code]
<?php
include_once '../../sys/inc/start.php';
include_once '../../sys/inc/compress.php';
include_once '../../sys/inc/sess.php';
include_once '../../sys/inc/home.php';
include_once '../../sys/inc/settings.php';
include_once '../../sys/inc/db_connect.php';
include_once '../../sys/inc/ipua.php';
include_once '../../sys/inc/fnc.php';
include_once '../../sys/inc/user.php';
error_reporting(1);
//Данная строка укажет на кракозебры кода
/* Бан пользователя */
if (isset($user) && mysql_result(mysql_query("SELECT COUNT(*) FROM `ban` WHERE `razdel` = 'notes' AND `id_user` = '$user[id]' AND (`time` > '$time' OR `view` = '0' OR `navsegda` = '1')"), 0)!=0)
{
header('Location: ../../ban.php?'.SID);
exit;
}
$set['title']='Дневники';
include_once '../../sys/inc/thead.php';
title();
aut(); // форма авторизации
echo "<div id='comments' class='menus'>";
echo "<div class='webmenu'>";
echo "<a href='index.php' class='activ'>Дневники</a>";
echo "</div>";

echo "<div class='webmenu last'>";
echo "<a href='dir.php'>Категории</a>";
echo "</div>";

echo "<div class='webmenu last'>";
echo "<a href='search.php'>Поиск</a>";
echo "</div>";
echo "</div>";
if (isset($_GET['sort']) && $_GET['sort'] =='t')$order='order by `time` desc';
elseif (isset($_GET['sort']) && $_GET['sort'] =='c') $order='order by `count` desc';
else $order='order by `time` desc';
if(isset($user))
{
echo'<div class="foot">';
echo "<a href="user.php">Мои дневники</a> | ";
echo "<a href="add.php">Создать дневник</a>";
echo '</div>';
}
echo'<div class="foot">';
if (isset($_GET['sort']) && $_GET['sort'] =='t'){
echo"<b>Новые</b> | <a href='?sort=c'>Популярные</a>n";
}elseif (isset($_GET['sort']) && $_GET['sort'] =='c'){
echo"<a href='?sort=t'>Новые</a> | <b>Популярные</b>n";
}else{
echo"<b>Новые</b> | <a href=

Hack

25 Окт 2014
Автор темы
RuDati, идем в лс, тут ссылку писать не буду, мне лишние люди на тест сайте не нужны.
______________________
уф