写插件的时候偶然发现的...可绕过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攻击防御可被绕过 - 网站的详细内容...