За последние 24 часа нас посетили 19496 программистов и 1601 робот. Сейчас ищут 1033 программиста ...

Есть ошибка или норм?

Тема в разделе "MySQL", создана пользователем KOMn1992, 11 янв 2015.

  1. KOMn1992

    KOMn1992 Новичок

    С нами с:
    8 янв 2015
    Сообщения:
    18
    Симпатии:
    0
    Код (Text):
    1.      elseif (isset($_GET[rights])) {
    2.         $page.="<br/><b>$_GET[rights]</b>";
    3.         $count= mysql_result(mysql_query("SELECT COUNT(id) FROM users WHERE rights='".mysql_real_escape_string($_GET[rights])."'"),0,0);
    4.         if (!isset($_GET['str'])) {$num_page=1;} else {$num_page=$_GET['str'];}
    5.             $num_page=intval($num_page);
    6.             $temp=($num_page-1)*10;
    7.             $sql=mysql_query("SELECT id, login, char_name, level FROM users WHERE rights='$_GET[rights]' LIMIT ".$temp.", 10");
    8.             $page.="<br/>[id] - [login] - [char_name] - [level]<br/><br/>";
    9.             while($users = mysql_fetch_array($sql))
    10.              {
    11.                 $page.="[".$users[id]."] - [".$users[login]."] - [".$users[char_name]."] - [".$users[level]."]  ".
    12.                  "<a href='./?do=admin&amp;mod=players&amp;unset=".$users[id]."'>[СНЯТЬ]</a><a href='./?do=admin&amp;mod=players&amp;redact=".$users[id]."'>[ИЗМ]</a><a href='./?do=admin&amp;mod=players&amp;del=".$users[id]."'>[X]</a><br/>";
    13.              }
    14.              $page.=nav_page(intval(ceil($count/10)), $num_page, "./?do=admin&amp;mod=players&amp;rights=$_GET[rights]&amp;str=");
    15.  
    16.  
    17.      }
    18.      
    Есть сомнения так как ругается поиск уязвимостей на вот это
    $count= mysql_result(mysql_query("SELECT COUNT(id) FROM users WHERE rights='".mysql_real_escape_string($_GET[rights])."'"),0,0); if (!isset($_GET['str'])) {$num_page=1;} else {$num_page=$_GET['str'];}

    То что красным помечено то и кажет как уязвимость:( Если что не так помогите исправить:(
     
  2. igordata

    igordata Суперстар
    Команда форума Модератор

    С нами с:
    18 мар 2010
    Сообщения:
    32.408
    Симпатии:
    1.768
    а тут забыл?
     
  3. KOMn1992

    KOMn1992 Новичок

    С нами с:
    8 янв 2015
    Сообщения:
    18
    Симпатии:
    0
    Что забыл???
     
  4. osvical

    osvical Новичок

    С нами с:
    9 янв 2015
    Сообщения:
    67
    Симпатии:
    0
    думаю, речь о том, что любой пользователь через адресную строку может сунуть ?rights=... и написать что угодно. тем самым изменить твой sql запрос и сделать с БД что угодно =)

    Добавлено спустя 4 минуты 8 секунд:
    кстати, на счет функции mysql_query и ей подобным