避免在注册表单php中进行SQL注入


Avoid SQL injection in registration form php

我在本地主机上有一个简单的注册表单(仍在测试),我想知道它是否会受到SQL注入的攻击?

代码:

$name = mysql_real_escape_string($_POST['name']);
$password = mysql_real_escape_string($_POST['password']);
$password = md5($password);
$email = mysql_real_escape_string($_POST['email']);
$refId = $_GET['refid'];
$ip = $_SERVER['REMOTE_ADDR'];

$add = mysql_query("INSERT INTO `users` (`name`, `password`, `email`, `refId`, `ip`)
VALUES('$name','$password','$email','$refId', '$ip')") or die(mysql_error());

这安全吗?或者有人可以使用SQL注入(我想知道如何使用)?如何避免注射?

避免注入的最佳方法是使用Prepared Statements
对于准备好的语句,我更喜欢使用PDO来处理我所有的DB内容。以下是我为检索一些基本登录信息而编写的一些PDO示例代码:

$sql=new PDO("mysql:host=127.0.0.1;dbname=name","user","password");
        $user=$_POST[user];
        $query="select Salt,Passwd from User
                where Name=:user";
        $stmt=$sql->prepare($query);
        $stmt->bindParam(':user',$user);
        $stmt->execute();
        $dr=$stmt->fetch();        
        $sql=null; 
        $password=$_POST[pass];
        $salt=$dr['Salt'];

etc

有关PDO的更多信息,请阅读本页。如果你想知道这里的每一行代码都在做什么,请阅读我在另一篇文章中给出的答案。

它确实很脆弱;你还没有逃脱$_GET['refid']。以这个URL为例:

yourpage.php?refid='%2C'')%3B%20DROP%20TABLE%20users%3B--

避免SQL注入很容易。使用事先准备好的报表和PDO。例如:

$query = $dbh->prepare("INSERT INTO `users`(`name`, `password`, `email`, `refId`, `ip`)
VALUES(:name, :password, :email, :refId, :ip)");
$query->bindValue(':name', $_POST['name']);
$query->bindValue(':password', md5($_POST['password'])); # Do not use MD5 for password hashing, and especially not without salt.
$query->bindValue(':email', $_POST['email']);
$query->bindValue(':refid', $_GET['refid']);
$query->bindValue(':ip', $_SERVER['REMOTE_ADDR']);
$query->execute();

您没有mysql_real_escape_string $refId,即使它是从$_GET中获取的。这基本上让你所有其他的预防注射措施都变得毫无意义。在切线上,如果在md5ing之前对密码进行转义,则会更改哈希。