ThinkPHP官网上曾有一段公告指出，在ThinkPHP 3.1.3及之前的版本存在一个sql注入漏洞,漏洞存在于ThinkPHP/Lib/Core/Model.class.PHP 文件 根据官方文档对"防止sql注入"的方法解释(参考http://doc.thinkPHP.cn/manual/sql_injection.html) 使用查询条件预处理可以防止sql注入,没错,当使用如下代码时可以起到效果:
where("id=%d and username='%s' and xx='%f'",array($id,$username,$xx))->select();
或者
where("id=%d and username='%s' and xx='%f'",$id,$xx)->select();
但是,当你使用如下代码时,却没有"防止sql注入"的效果(但是官方文档却说可以防止sql注入):
query('select * from user where id=%d and status=%s',$status);
或者
query('select * from user where id=%d and status=%s',$status));
原因分析:
ThinkPHP/Lib/Core/Model.class.PHP 文件里的parsesql函数没有实现sql过滤. 其原函数为:
_parSEOptions();
$sql = $this->db->parsesql($sql,$options);
}elseif(is_array($parse)){ // sql预处理
$sql = vsprintf($sql,$parse);
}else{
$sql = strtr($sql,array('__TABLE__'=>$this->getTableName(),'__PREFIX__'=>C('DB_PREFIX')));
}
$this->db->setModel($this->name);
return $sql;
}
验证漏洞(举例): 请求地址:
action代码:
query('select * from peipeidui where name="%s"',$_GET['id']);
dump($m);exit;
或者：
query('select * from peipeidui where name="%s"',array($_GET['id']));
dump($m);exit;
结果:
表peipeidui所有数据被列出,sql注入语句起效.
解决方法:
db,'escapeString'),$parse);//此行为新增代码
$sql = vsprintf($sql,'__PREFIX__'=>C('DB_PREFIX')));
}
$this->db->setModel($this->name);
return $sql;
}
总结:
