PHP Pagination SQL Security

Question

Следующий код уязвим для SQL, и мне нужна помощь в его защите; Я не мастер с PHP или MySQL.

Я искал вокруг и видел некоторые очень полезные вещи, чтобы сделать с MySQL реального побега строки и т.д., но при реализации он просто вызывает белую страницу

if(!isset($_GET['pg'])) { $page = 1; } else { $page = $_GET['pg']; } 
    $select = (($page * 20) - 20); 
$mysql_result = mysql_query("SELECT `id` FROM `{$lgsl_config['db']['prefix']}{$lgsl_config['db']['table']}` WHERE ".implode(" AND ", $mysql_where)." ORDER BY id ASC LIMIT $select, 20") or die(mysql_error()); 

<? 
$servers = mysql_result(mysql_query("SELECT COUNT(*) as Num FROM data_servers"),0); 
$pages = ceil($servers/20); 

if(!isset($_GET['pg'])) { $page = 1; } else { $page = $_GET['pg']; } 

if($page > 1) 
{ 
$prev = ($page - 1); 
echo "<a href='?pg=$prev'>[Inapoi]</a> "; 
} 

for($show = 1; $show <= $pages; $show++) 
{ 
if(($page) == $show) 
{ 
echo $show; 
} 
else 
{ 
echo " <a href='?pg=$show'>[$show]</a> "; 
} 
} 

if($page < $pages) 
{ 
$next = ($page + 1); 
echo " <a href='?pg=$next'>[Next]</a> "; 
} 
?> 

Доказательство того, что код является Infact уязвимым на конце ваш URL просто добавить ? рд = MySQL

и вы должны получить ошибку синтаксиса MySQL

Answer 1

Поскольку вы не используете $page напрямую, но вы математическая операция на нем, если она не является числовым значением, она будет оценена как 0, если она не является числом (например, если это строка).

Я предложил бы в любом случае, чтобы добавить некоторые элементы управления, просто чтобы быть уверенным:

if(!isset($_GET['pg']) || !is_numeric($_GET['pg'])) { $page = 1; } else { $page = (int) $_GET['pg']; } 

Таким образом, вы уверены, что $page держит целое.

В этом случае вам не нужна функция эвакуации, но если вы используете непосредственно любую другую переменную $_GET или $_POST, тогда вы должны ее рассмотреть.

Answer 2

Я бы проверить эти ссылки, внимательно их прочитать и понять, что они говорят г, чтобы обеспечить код и сервер

http://phptutorial.info/?mysql-real-escape-string
http://www.sitepoint.com/php-security-blunders/
http://roshanbh.com.np/2008/05/tighten-php-security-functions.html

Они помогли мне, когда я узнавал аспекта безопасности PHP кодирования