php - 我的 php 代码是否容易受到 SQL 注入(inject)攻击

Question

我的一个 friend 告诉我，使用这个脚本来验证用户身份对 SQL 漏洞来说是真正的风险

<?php
if(strlen(strstr($_SERVER['HTTP_USER_AGENT'],"-- IPB Vaidation  --")) <= 0 ){ 
die('Login Failed!, Please try again.');
}
$name = strtolower($_GET["name"]);
$password = $_GET["password"];
$digits = $_GET["digits"];
$random_number = 70; 
$sum_total2 = $digits * $random_number;
$con = mysql_connect("127.0.0.1","usernamehere","passhere");
if (!$con)
  {
  die('Could not connect: ' . mysql_error());
  }
mysql_select_db("database here", $con);
$sql = "SELECT * FROM ipbmembers WHERE members_seo_name='{$name}'";
$result = mysql_query($sql) or die(mysql_error());
while($row = mysql_fetch_array($result))
  {
  $trueHash = $row['members_pass_hash'];
  $salt = $row['members_pass_salt'];
  }
$hash = md5(md5($salt) . md5($password));
if($hash == $trueHash)
{
echo "Thank you for logging in";
echo ($sum_total2);
}
else
{
echo "Login Failed!, Please try again.";
echo ($sum_total2);
}  
?>

谁能告诉我它是如何完成的？或者我做错了什么？

谢谢!

Answer 1

是的，您的代码很容易受到攻击。看一下这个声明:

$sql = "SELECT * FROM ipbmembers WHERE members_seo_name='{$name}'";

如果$name的值为'怎么办？删除表 ipbmembers； --？那么您构建的 SQL 语句将是:

SELECT * FROM ipbmembers WHERE members_seo_name=''; DROP TABLE ipbmembers; --'

您将执行 SELECT，然后删除表。

不要从外部变量构建 SQL。使用占位符和准备好的语句。