我的PDO INSERT INTO是否足够安全？

Question

我希望您对我的代码有意见。 是否足够安全以防止任何注射。 多谢您的回覆。

class Product extends DB {

  public function __construct() {
    $db = $this->DB();

    if(isset($_POST['productName'])) {
        foreach ($_POST as $key => $value) {
            if (ini_get('magic_quotes_gpc'))
                $_POST[$key] = stripslashes($_POST[$key]);
                $_POST[$key] = htmlspecialchars(strip_tags($_POST[$key]));
        }
        $this->AddProduct();
    }
  }

  public function AddProduct() {
    $sSQL = "INSERT INTO ".PREFIX."product (productName, productPrice) 
                VALUES (:productName,:productPrice)";
    $query = $this->db->prepare($sSQL);
    $query->execute(array(
        ":productName" => $_POST['productName'],
        ":productPrice" => $_POST['productPrice']
    )); 
  }
}

Answer 1

使用查询参数足以使其免受SQL注入漏洞的攻击。

调用htmlspecialchars和strip_tags的代码与SQL注入无关。 可能需要使用它来防止跨站点脚本漏洞，但这是一个单独的问题。 在将数据插入数据库时​​，我不建议执行这些步骤。 当您输出到HTML时，只需过滤XSS漏洞即可。 否则，您将得到字面& 序列存储在数据库中，还为时过早。 您不必每次都使用数据在HTML中显示。 仅在输出时对其进行编码，而不在输入时进行编码。

我从来没有为可能的magic_quotes_gpc进行补偿。 在部署应用程序时进行测试，然后中止部署。 在2014年对任何 PHP实例设置magic_quotes_gpc都是无效的。