好得很程序员自学网

<tfoot draggable='sEl'></tfoot>

Discuz! X2.5 / X3 / X3.1中CSRF攻击防御可被绕过 - 网站

写插件的时候偶然发现的...可绕过Discuz自带的CSRF攻击

discuz在判断表单时用这个函数:

submitcheck('submit', true)

 

 

其中第二个参数本来应该是是否允许GET请求的选项。

 

在X2.5 / X3 / X3.1中,第二个参数为true时竟然不判断formhash直接return true了……

 

使用批量搜索工具搜索discuz源代码里的submitcheck(*, *)……然后发现官方也用了这种写法,尿了尿了

 

另外表示X2和之前的都没看过,不确定是否有问题。 我本地的X2.5 X3 X3.1这个文件基本都一样……

 

source/class/helper/helper_form.php

 

if($allowget || ($_SERVER['REQUEST_METHOD'] == 'POST' && !empty($_GET['formhash']) && $_GET['formhash'] == formhash() && empty($_SERVER['HTTP_X_FLASH_VERSION']) && (empty($_SERVER['HTTP_REFERER']) || preg_replace("/https?:\/\/([^\:\/]+).*/i", "\\1", $_SERVER['HTTP_REFERER']) == preg_replace("/([^\:]+).*/", "\\1", $_SERVER['HTTP_HOST'])))) {

 

 

 

 

当$allowget = TRUE时,后面整个括号就可以无视了

修复方案:

最简单的办法,修改下这个判断语句的逻辑吧

查看更多关于Discuz! X2.5 / X3 / X3.1中CSRF攻击防御可被绕过 - 网站的详细内容...

  阅读:84次