SQL注入和prepared语句，何时会被过度使用？

Question

我已经读了很多关于sql注入的书，并且已经使用mysqli编写的语句一年多了。我越接近我的问题是，为什么这个MySQLI编写的语句允许SQL注入？

现在，我想创建一个函数，根据用户的搜索条件运行查询。我使用这个，所以我可以使用很多不同的标准。

这里有一个简化的例子，你可以围绕我的问题来思考：假设我们有两张桌子，一张有轮胎，一张有轮子。

CREATE TABLE IF NOT EXISTS `wheels` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `brand` varchar(64) COLLATE utf8_unicode_ci NOT NULL,
  `size` int(11) NOT NULL,
  `price` int(11) NOT NULL,
  PRIMARY KEY (`id`),
  KEY `brand` (`brand`),
  KEY `size` (`size`),
  KEY `price` (`price`)
);
CREATE TABLE IF NOT EXISTS `tires` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `brand` varchar(64) COLLATE utf8_unicode_ci NOT NULL,
  `size` int(11) NOT NULL,
  `price` int(11) NOT NULL,
  PRIMARY KEY (`id`),
  KEY `brand` (`brand`),
  KEY `size` (`size`),
  KEY `price` (`price`)
);

现在我们有一个表单让用户进行搜索

<form method='post' ...>
  Looking for : <select name='item'>
    <option value='tires'>Tires</option>
    <option value='wheels'>Wheels</option>
  </select>
  Search by : <select name='type'>
    <option value='size'>Size</option>
    <option value='price'>Maximum price</option>
  </select>
  <input type='text' name='criteria' />
</form>

现在，处理请求的php如下所示：

$item=filter_input(INPUT_POST,'item',FILTER_SANITIZE_STRING);
$type=filter_input(INPUT_POST,'type',FILTER_SANITIZE_STRING);
$criteria=filter_input(INPUT_POST,'criteria',FILTER_SANITIZE_NUMBER_INT);

function build_query($item,$type,$criteria){
  switch($item){
    case 'tires': $table='tires'; break;
    case 'wheels': $table='wheels'; break;
    defaults: /*error handling : bad search criteria*/ break;
  }
  switch($type){
    case 'size': $field='size'; $operator='='; break;
    case 'price': $field='price'; $operator='<='; break;
    defaults: /*error handling : bad search criteria*/ break;
  }
  $value=intval($criteria);

  $sql= ....

  //Do the rest of sqli magic here and return the results.

}

请注意，只有$标准（成为$值）不是“硬编码”的，并从搜索表单发送到DB。所以问题是：如果$table、$field和$operator变量来自我的php内部代码，是否也需要绑定它们？

换句话说：

这是不是太过分了？

$sql="SELECT * FROM ? WHERE ???"; 
$stmt->bind_param('sssi',$table,$field,$operator,$value);

和/或这是否足够？

$sql="SELECT * FROM ".$table." WHERE ".$field.$operator."?";
$stmt->bind_param('i',$value);

如前所述，这是一个简化的示例，因此您可以更好地理解问题。

Answer 1

是的，您将使用PDO走得更远。PDO仅用于参数，不用于表或运算符。使用第二个查询。

http://php.net/manual/en/pdostatement.bindparam.php不表示允许使用表和运算符，我记得不久前测试过，发现它们不允许。

将PHP变量绑定到用于准备语句的SQL语句中相应的命名占位符或问号占位符。与PDOStatement：：bindValue（）不同，该变量被绑定为引用，仅在调用PDOStatement：：execute（）时进行计算。

大多数参数是输入参数，即以只读方式用于构建查询的参数。一些驱动程序支持调用将数据作为输出参数返回的存储过程，一些驱动程序还支持作为输入/输出参数发送数据并更新以接收数据。