应该使用哪种PDO绑定方法来提高安全性?

时间:2015-01-21 22:28:46

标签: php mysql security pdo sql-injection

我知道有两种方法可以在PHP中使用PDO来更新MySQL数据库记录。请有人解释我应该使用哪一个以获得更好的安全性和差异,我有点困惑。

方法一:

$user = "root";
$pass = "";
$dbh = new PDO('mysql:host=somehost;dbname=somedb', $user, $pass);
$sql = "UPDATE coupons SET 
coupon_code = :coupon_code, 
valid_from = :valid_from, 
valid_to = :valid_to,  
discount_percentage = :discount_percentage,  
discount_amount = :discount_amount,  
calculationType = :calculationType,  
limit = :limit  
WHERE coupon_code = :coupon";
$stmt = $dbh->prepare($sql);                                  
$stmt->bindParam(':coupon_code', $_POST['coupon_code'], PDO::PARAM_STR);       
$stmt->bindParam(':valid_from', $_POST['$valid_from'], PDO::PARAM_STR);    
$stmt->bindParam(':valid_to', $_POST['valid_to'], PDO::PARAM_STR);
$stmt->bindParam(':discount_percentage', $_POST['discount_percentage'], PDO::PARAM_STR); 
$stmt->bindParam(':discount_amount', $_POST['discount_amount'], PDO::PARAM_STR);   
$stmt->bindParam(':calculationType', $_POST['calculationType'], PDO::PARAM_STR);   
$stmt->bindParam(':limit', $_POST['limit'], PDO::PARAM_STR);   
$stmt->bindParam(':coupon', $_POST['coupon_code'], PDO::PARAM_STR);   
$stmt->execute();

方法二:

$dbtype="somedbtype";
$dbhost="somehost";
$dbname="somedb";
$dbuser="someuser";
$dbpass= "somepass";
$conn = new PDO("mysql:host=$dbhost;dbname=$dbname",$dbuser,$dbpass);
$title = 'PHP Pattern';
$author = 'Imanda';
$id = 3;
$sql = "UPDATE books 
SET title=?, author=?
WHERE id=?";
$q = $conn->prepare($sql);
$q->execute(array($title,$author,$id));

从我所看到的,方法二不绑定数据,而是将其作为数组类型直接插入查询中。这是否会使脚本更容易受到SQL注入或其他安全风险的影响?

2 个答案:

答案 0 :(得分:4)

两者之间的唯一区别是,如果您将数组传递给execute函数而不是自己调用bindParam,它会自动将所有参数视为PDO::PARAM_STR,而在调用时bindParam你自己可以将它们绑定为整数等等。

来自docs

  

input_parameters

     

具有与正在执行的SQL语句中的绑定参数一样多的元素的值数组。所有值都被视为PDO :: PARAM_STR。

您还可以从示例中看到,在将数组传递到:limit函数时,您可以使用命名参数(例如execute)。你不必只是放?。在这种情况下,你给数组一个键:

$sth->execute(array(':calories' => $calories, ':colour' => $colour));

答案 1 :(得分:1)

这主要是一个偏好问题。两者都可以防止注射。

虽然我认为使用bind()强制数据类型要容易得多,但使用execute(array())将使用字符串。