漏洞分析：discuz7.2 faq.php 最新注入漏洞分析 |

发表评论
592 次浏览

A+

显示不全请点击全屏阅读

之前乌云上被提交的漏洞，今天突然被人发出来了，然后就各种中枪。奈何各种考试马上就来了，我也没工夫去写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
						
							}