显示不全请点击全屏阅读
之前乌云上被提交的漏洞，今天突然被人发出来了，然后就各种中枪。奈何各种考试马上就来了，我也没工夫去写exp或脚本什么的，趁点休息时间把代码看看就好。实话说我很讨厌这种情况，一大堆人会的不会的拿着别人发的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行开始）：
| 01 | } elseif($action == ‘grouppermission’) { |
| 02 |
| 03 | … |
| 04 | … |
| 05 | ksort($gids); |
| 06 | $groupids = array(); |
| 07 | foreach($gids as $row) { |
| 08 | $groupids[] = $row[0]; |
| 09 | } |
| 10 |
| 11 | $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函数：
| 1 | function implodeids($array) { |
| 2 | if(!empty($array)) { |
| 3 | return “‘”.implode(“‘,'”, is_array($array) ? $array : array($array)).”‘”; |
| 4 | } else { |
| 5 | return ”; |
| 6 | } |
| 7 | } |
很简单一个函数，就是将刚才的$groupids数组用’,’分割开，组成一个类似于’1′,’2′,’3′,’4’的字符串返回。
但是我们的数组刚取出来一个转义符，它会将这里一个正常的’转义掉，比如这样：
‘1’,’\’,’3′,’4′
有没有看出有点不同，第4个单引号被转义了，也就是说第5个单引号和第3个单引号闭合。这样3这个位置就等于逃逸出了单引号，也就是产生的注入。我们把报错语句放在3这个位置，就能报错：
EXP我就不给了，省的更多人刷洞。复习去了。
摘自：http://www.leavesongs.com/discuz72-sql-injection.html
Tags：
代码审计, 漏洞分析,如果您喜欢我的博客，欢迎点击图片定订阅到邮箱 也可以点击链接【订阅到鲜果】 
如果我的想法或工具帮助到了你，也可微信扫下方二维码打赏本人一杯咖啡
联系客服
