为什么我不能将表名传递给准备好的 PDO 语句?
$stmt = $dbh->prepare('SELECT * FROM :table WHERE 1');
if ($stmt->execute(array(':table' => 'users'))) {
var_dump($stmt->fetchAll());
}
是否有另一种安全的方法可以将表名插入到 SQL 查询中?有了safe,我的意思是我不想做
$sql = "SELECT * FROM $table WHERE 1"
表名和列名不能被 PDO 中的参数替换。
在那种情况下,您只需要手动过滤和清理数据。一种方法是将速记参数传递给将动态执行查询的函数,然后使用
switch()
语句创建用于表名或列名的有效值白名单。这样,用户输入就不会直接进入查询。例如:
function buildQuery( $get_var )
{
switch($get_var)
{
case 1:
$tbl = 'users';
break;
}
$sql = "SELECT * FROM $tbl";
}
通过不保留默认情况或使用返回错误消息的默认情况,您可以确保只使用您想要使用的值。
要了解为什么绑定表(或列)名称不起作用,您必须了解预准备语句中的占位符是如何工作的:它们不是简单地替换为(适当转义的)字符串,然后执行生成的 SQL。相反,要求“准备”一条语句的 DBMS 提出了一个完整的查询计划,说明它将如何执行该查询,包括它将使用哪些表和索引,无论您如何填写占位符,它们都是相同的。
无论您用
SELECT name FROM my_table WHERE id = :value
替换什么,:value
的计划都是相同的,但是看似相似的SELECT name FROM :table WHERE id = :value
无法计划,因为DBMS不知道您实际上要从哪个表中选择。
这不是像 PDO 这样的抽象库可以或应该解决的问题,因为它会破坏准备语句的 2 个关键目的:1) 允许数据库提前决定查询将如何运行,并使用同一个计划多次; 2)通过将查询逻辑与变量输入分开来防止安全问题。
我看到这是一篇旧帖子,但我发现它很有用,并认为我会分享一个类似于@kzqai 建议的解决方案:
我有一个接收两个参数的函数,例如...
function getTableInfo($inTableName, $inColumnName) {
....
}
我在里面检查我设置的数组,以确保只有带有“祝福”表的表和列是可访问的:
$allowed_tables_array = array('tblTheTable');
$allowed_columns_array['tblTheTable'] = array('the_col_to_check');
然后运行 PDO 之前的 PHP 检查看起来像...
if(in_array($inTableName, $allowed_tables_array) && in_array($inColumnName,$allowed_columns_array[$inTableName]))
{
$sql = "SELECT $inColumnName AS columnInfo
FROM $inTableName";
$stmt = $pdo->prepare($sql);
$stmt->execute();
$result = $stmt->fetchAll(PDO::FETCH_ASSOC);
}
使用前者本质上并不比后者更安全,您需要清理输入,无论它是参数数组的一部分还是简单变量。所以我认为将后一种形式与
$table
一起使用没有任何问题,前提是您在使用之前确保$table
的内容是安全的(字母数字加下划线?)。
(迟到的答案,请参阅我的旁注)。
尝试创建“数据库”时适用相同的规则。
您不能使用准备好的语句来绑定数据库。
即:
CREATE DATABASE IF NOT EXISTS :database
不会工作。请改用安全列表。
旁注: 我添加了这个答案(作为社区维基),因为它经常用于结束问题,有些人在尝试绑定 database 而不是表和/或列时发布了与此类似的问题。
我有点想知道你是否可以像这样简单地提供自己的自定义消毒功能:
$value = preg_replace('/[^a-zA-Z_]*/', '', $value);
我还没有真正考虑过,但似乎删除字符和下划线以外的任何内容都可能有效。
至于这个帖子中的主要问题,其他帖子说得很清楚为什么我们不能在准备语句时将值绑定到列名,所以这里是一个解决方案:
class myPdo{
private $user = 'dbuser';
private $pass = 'dbpass';
private $host = 'dbhost';
private $db = 'dbname';
private $pdo;
private $dbInfo;
public function __construct($type){
$this->pdo = new PDO('mysql:host='.$this->host.';dbname='.$this->db.';charset=utf8',$this->user,$this->pass);
if(isset($type)){
//when class is called upon, it stores column names and column types from the table of you choice in $this->dbInfo;
$stmt = "select distinct column_name,column_type from information_schema.columns where table_name='sometable';";
$stmt = $this->pdo->prepare($stmt);//not really necessary since this stmt doesn't contain any dynamic values;
$stmt->execute();
$this->dbInfo = $stmt->fetchAll(PDO::FETCH_ASSOC);
}
}
public function pdo_param($col){
$param_type = PDO::PARAM_STR;
foreach($this->dbInfo as $k => $arr){
if($arr['column_name'] == $col){
if(strstr($arr['column_type'],'int')){
$param_type = PDO::PARAM_INT;
break;
}
}
}//for testing purposes i only used INT and VARCHAR column types. Adjust to your needs...
return $param_type;
}
public function columnIsAllowed($col){
$colisAllowed = false;
foreach($this->dbInfo as $k => $arr){
if($arr['column_name'] === $col){
$colisAllowed = true;
break;
}
}
return $colisAllowed;
}
public function q($data){
//$data is received by post as a JSON object and looks like this
//{"data":{"column_a":"value","column_b":"value","column_c":"value"},"get":"column_x"}
$data = json_decode($data,TRUE);
$continue = true;
foreach($data['data'] as $column_name => $value){
if(!$this->columnIsAllowed($column_name)){
$continue = false;
//means that someone possibly messed with the post and tried to get data from a column that does not exist in the current table, or the column name is a sql injection string and so on...
break;
}
}
//since $data['get'] is also a column, check if its allowed as well
if(isset($data['get']) && !$this->columnIsAllowed($data['get'])){
$continue = false;
}
if(!$continue){
exit('possible injection attempt');
}
//continue with the rest of the func, as you normally would
$stmt = "SELECT DISTINCT ".$data['get']." from sometable WHERE ";
foreach($data['data'] as $k => $v){
$stmt .= $k.' LIKE :'.$k.'_val AND ';
}
$stmt = substr($stmt,0,-5)." order by ".$data['get'];
//$stmt should look like this
//SELECT DISTINCT column_x from sometable WHERE column_a LIKE :column_a_val AND column_b LIKE :column_b_val AND column_c LIKE :column_c_val order by column_x
$stmt = $this->pdo->prepare($stmt);
//obviously now i have to bindValue()
foreach($data['data'] as $k => $v){
$stmt->bindValue(':'.$k.'_val','%'.$v.'%',$this->pdo_param($k));
//setting PDO::PARAM... type based on column_type from $this->dbInfo
}
$stmt->execute();
return $stmt->fetchAll(PDO::FETCH_ASSOC);//or whatever
}
}
$pdo = new myPdo('anything');//anything so that isset() evaluates to TRUE.
var_dump($pdo->q($some_json_object_as_described_above));
以上只是一个例子,不用说,copy->paste是不行的。根据您的需要进行调整。 现在这可能无法提供 100% 的安全性,但它允许在列名作为动态字符串“进入”时对列名进行一些控制,并且可以在用户端更改。此外,无需使用您的表列名称和类型构建一些数组,因为它们是从 information_schema 中提取的。
手动保护您的代码免受 DDL 和 DML 攻击。
喜欢:
function protect($a){//sql protect
$a=trim(preg_replace("#insert |delete |update | --|drop |replace |alter |modify |create |select #sui"," ", $a));
return $a;
}
简短的回答是不,你不能在带有 PDO 的准备好的执行语句中使用动态表名、字段名等,因为它会向它们添加引号,这会破坏查询。但是如果你能清理它们,那么你就可以安全地将它们放在查询本身中,就像使用 MySQLi 一样。
正确的方法是使用 mysqli 的 mysqli_real_escape_string() 函数,因为 mysql_real_escape_string 被仓促地从 PHP 中删除,没有考虑它如何影响动态结构应用程序。
$unsanitized_table_name = "users' OR '1'='1"; //SQL Injection attempt
$sanitized_table_name = sanitize_input($unsanitized_table_name);
$stmt = $dbh->prepare("SELECT * FROM {$unsanitized_table_name} WHERE 1"); //<--- REALLY bad idea
$stmt = $dbh->prepare("SELECT * FROM {$sanitized_table_name} WHERE 1"); //<--- Not ideal but hey, at least you're safe.
//PDO Cant sanitize everything so we limp along with mysqli instead
function sanitize_input($string)
{
$mysqli = new mysqli("localhost","UsahName","Passerrrd");
$string = $mysqli->real_escape_string($string);
return $string;
}