发布于 2014-08-09 06:12:20 | 246 次阅读 | 评论: 0 | 来源: 网友投递
Discuz!社区论坛系统
Crossday Discuz! Board（简称 Discuz!）是北京康盛新创科技有限责任公司推出的一套通用的社区论坛软件系统。自2001年6月面世以来，Discuz!已拥有13年以上的应用历史和200多万网站用户案例，是全球成熟度最高、覆盖率最大的论坛软件系统之一。
本文主要讲解并分析了discuz7.2版的faq.php文件的一个sql注入漏洞,下文包含注入代码和漏洞形成的原因分析.感兴趣的同学参考学习下,如果你的网站存在此漏洞赶紧打补丁.
注入代码实例:
...
ksort($gids);
$groupids = array();
foreach($gids as $row) {
$groupids[] = $row[0];
}
$query = $db->query("SELECT * FROM {$tablepre}usergroups u LEFT JOIN {$tablepre}admingroups a ON u.groupid=a.admingid WHERE u.groupid IN (".implodeids($groupids).")");
...
}
function implodeids($array) {
if(!empty($array)) {
return "'".implode("','", is_array($array) ? $array : array($array))."'";
} else {
return '';
}
}
首先定义一个数组groupids，然后遍历$gids（这也是个数组，就是$_GET[gids]），将数组中的所有值的第一位取出来放在groupids中。
为什么这个操作就造成了注入？
discuz在全局会对GET数组进行addslashes转义，也就是说会将'转义成'，所以，如果我们的传入的参数是：gids[1]='的话，会被转义成$gids[1]='，而这个赋值语句$groupids[] = $row[0]就相当于取了字符串的第一个字符，也就是，把转义符号取出来了。
再看后面，在将数据放入sql语句前，他用implodeids处理了一遍。我们看到implodeids函数
很简单一个函数，就是将刚才的$groupids数组用','分割开，组成一个类似于'1','2','3','4'的字符串返回。
但是我们的数组刚取出来一个转义符，它会将这里一个正常的'转义掉，比如这样：
'1','','3','4'
有没有看出有点不同，第4个单引号被转义了，也就是说第5个单引号和第3个单引号闭合。