2018-12-06 Discuz!-BUG反馈 退出网站清除痕迹报错！

故障现象

程序版本：Discuz! X3.1
故障场景：登录站点后，点击退出并执行清除浏览器痕迹操作，页面抛出系统错误：

Discuz! System Error
您当前的访问请求当中含有非法字符，已经被系统拒绝

PHP Debug 堆栈信息

[Line: 0026]member.php(discuz_application->init)
[Line: 0071]source/class/discuz/discuz_application.php(discuz_application->_init_misc)
[Line: 0552]source/class/discuz/discuz_application.php(discuz_application->_xss_check)
[Line: 0355]source/class/discuz/discuz_application.php(system_error)
[Line: 0017]source/function/function_core.php(discuz_error::system_error)
[Line: 0024]source/class/discuz/discuz_error.php(discuz_error::debug_backtrace)

站点已记录详细错误日志，访问被拦截。

浏览器复现环境：IE6‑IE10、火狐、TT、360 浏览器、Chrome、傲游

问题根因

source/class/discuz/discuz_application.php 中 _xss_check() XSS 安全检测函数逻辑变更。新版检测逻辑会双重 urldecode 解析REQUEST_URI，浏览器清除痕迹产生的特殊编码字符被误判为 XSS 恶意字符，触发request_tainting请求污染拦截。

修复方案

文件路径：source/class/discuz/discuz_application.php，定位大约 350 行私有方法 _xss_check()

新版报错代码（出问题版本）

private function _xss_check() {
    static $check = array('"', '>', '<', '\'', '(', ')', 'CONTENT-TRANSFER-ENCODING');
    if(isset($_GET['formhash']) && $_GET['formhash'] !== formhash()) {
        system_error('request_tainting');
    }

    if($_SERVER['REQUEST_METHOD'] == 'GET' ) {
        $temp = $_SERVER['REQUEST_URI'];
    } elseif(empty ($_GET['formhash'])) {
        $temp = $_SERVER['REQUEST_URI'].file_get_contents('php://input');
    } else {
        $temp = '';
    }

    if(!empty($temp)) {
        $temp = strtoupper(urldecode(urldecode($temp)));
        foreach ($check as $str) {
            if(strpos($temp, $str) !== false) {
                system_error('request_tainting');
            }
        }
    }
    return true;
}

回退修复代码（旧版可用逻辑）

将上述函数替换为早期版本逻辑即可解决误拦截：

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

说明：该方式为临时兼容修复，舍弃括号、单引号、大于号的 URI 检测规则，解决浏览器清除痕迹带来的正常请求误拦截，升级程序时注意覆盖还原问题。