phpdisk代码设计缺陷导致SQL注入一枚 – 网站安全

 漏洞存在版本为PHPDisk F-Core系列

测试版本为PHPDisk F-Core v1.1 20140703

SQL注入在发布资源时被触发，参数post_tag为注入点，上代码

/modules/post.inc.php，第124行左右
 

$db->query_unbuffered("insert into {$tpf}posts set ".$db->sql_array($ins)."");
$pid = $db->insert_id();
make_tags($tags,$tag_arr,$pid);//注入点
$db->query_unbuffered("update {$tpf}categories set cate_num=cate_num+1 where cate_id='$cate_id'");
$sysmsg[] = $settings[pd_post_name].'发布成功';

跟进make_tags函数看一下
 

function make_tags($tags,$tag_arr,$file_id){
global $db,$tpf,$timestamp,$pd_uid;
if($tags){
$tags = filter_tag($tags); 
$tags_str = '';
for($i=0;$i<count($tag_arr);$i++){
if($tag_arr[$i]){
$tags_str .= "'".filter_tag($tag_arr[$i])."',";//filter_tag对tag的值做了处理
$rs = $db->fetch_one_array("select count(*) as total from {$tpf}post2tag where tag_name='{$tag_arr[$i]}' and pid='{$file_id}'");
if(!$rs['total']){
$ins = array(
'tag_name' => $tag_arr[$i],
'pid' => $file_id,
);
$db->query_unbuffered("insert into {$tpf}post2tag set ".$db->sql_array($ins).";");
}
unset($rs);
}
}
$tags_str = (substr($tags_str,-1) ==',') ? substr($tags_str,0,-1) : $tags_str;
$db->query_unbuffered("update {$tpf}tags set tag_count=tag_count-1 where tag_name in (select tag_name from {$tpf}post2tag where pid='$file_id')");
//echo "delete from {$tpf}post2tag where pid='$file_id' and tag_name not in ($tags_str)";
$db->query_unbuffered("delete from {$tpf}post2tag where pid='$file_id' and tag_name not in ($tags_str)"//留下反斜杠注入发生

filter_tag函数是重点，跟进这个函数看下
 

function filter_tag($str){
return str_replace(array('"',"'",'/','(',')','*'),'',$str);
}

这个函数对单引号等特殊字符进行过滤，过滤为空~，但是由POST传入的数据是经过addslashes过滤的，在这里再把单引号拿掉的话，就会把反斜杠给剩下，导致注入的发生

测试流程：

本地搭建环境来测试一下

官方测试站点也测一下

http://demo.phpdisk.com/f/
 

没有mysql报错，盲注或者延时都可以

修复方案：

有addslashes的情况下，可以不需要filter_tag了