发现了一个重大问题,Discuz!x3.2跟Ucenter1.6.0的BUG

最近搭建Discuz!x3.2和Ucenter1.6.0的时候，发现经常报通知失败，跟踪发现Discuz!x2.5是不存在该问题的，原因是 source/class/discuz/discuz_application.php
该文件有一个校验函数叫 _xss_check

下面是源码
Discuz!x2.5

1. private function _xss_check() {
   
2.                 $temp = strtoupper(urldecode(urldecode($_SERVER['REQUEST_URI'])));
   
3.                 if(strpos($temp, '<') !== false || strpos($temp, '"') !== false || strpos($temp, 'CONTENT-TRANSFER-ENCODING') !== false) {
   
4.                         system_error('request_tainting');
   
5.                 }
   
6.                 return true;
   
7.         }

复制代码

Discuz!x3.2

1. private function _xss_check() {
   
2. 
   
3.                 static $check = array('"', '>', '<', '\'', '(', ')', 'CONTENT-TRANSFER-ENCODING');
   
4. 
   
5.                 if(isset($_GET['formhash']) && $_GET['formhash'] !== formhash()) {
   
6.                         system_error('request_tainting');
   
7.                 }
   
8. 
   
9.                 if($_SERVER['REQUEST_METHOD'] == 'GET' ) {
   
10.                         $temp = $_SERVER['REQUEST_URI'];
    
11.                 } elseif(empty ($_GET['formhash'])) {
    
12.                         $temp = $_SERVER['REQUEST_URI'].file_get_contents('php://input');
    
13.                 } else {
    
14.                         $temp = '';
    
15.                 }
    
16. 
    
17.                 if(!empty($temp)) {
    
18.                         $temp = strtoupper(urldecode(urldecode($temp)));
    
19.                         foreach ($check as $str) {
    
20.                                 if(strpos($temp, $str) !== false) {
    
21.                                         system_error('request_tainting');
    
22.                                 }
    
23.                         }
    
24.                 }
    
25. 
    
26.                 return true;
    
27.         }

复制代码

对比发现，新版代码的问题在 $temp = $_SERVER['REQUEST_URI'].file_get_contents('php://input'); ，如果是Ucenter下发的通知
$temp拿到的数据是 get的数据和post的数据
因为ucenter 发的post数据是xml格式, 所以下面的检测是肯定会走到 system_error('request_tainting');的

1. foreach ($check as $str) {
   
2. if(strpos($temp, $str) !== false) {
   
3. system_error('request_tainting');
   
4. }

复制代码

因为$check包含了 '<'  '>' 这些符号

1. static $check = array('"', '>', '<', '\'', '(', ')', 'CONTENT-TRANSFER-ENCODING');

复制代码

是跨站检测脚本哦

crx349 发表于 2014-11-14 09:54
是跨站检测脚本哦

关键是他检测post数据 不可以有 < >符号，而ucenter发送的通知是xml 肯定有 < >符号的，所以ucenter一直都通知失败 懂吗

那问题来了，咋解决?

wengongling 发表于 2014-11-14 10:12
那问题来了，咋解决?

我的办法是 既然这个函数是用来检测跨网站的 那忽略 post的数据也是没问题的

$temp = $_SERVER['REQUEST_URI'].file_get_contents('php://input');
改为
$temp = $_SERVER['REQUEST_URI'];