Xiuno BBS 审计之问题13:管理员发帖 doctype=0 时存储型 XSS

问题:管理员发帖 doctype=0 时存储型 XSS(绕过 htmlspecialchars)

此文章为XIUNOX版本重构审计时发现问题,XIUNOX版本已优化修复此问题。分享出来方便后续想基于xiuno bbs4.0.4版本制作维护版本或插件模板等需求的开发者和站长参考。

现象

Xiuno BBS 4.0.4 在帖子内容入库格式化时,对 doctype=0(HTML 类型)的处理存在特权分支:当发帖人 gid==1(管理员组)时,原文 message 未经 htmlspecialchars 也未经 xn_html_safe 白名单过滤,直接赋值给 message_fmt 入库。前台模板 post_list.inc.htmthread.htm 又以 <?php echo $first['message_fmt'];?> 原样输出,未做二次转义。

攻击者只要拿到管理员账号(或通过弱口令、CSRF 劫持后台),即可在帖子中植入任意 <script>,对所有浏览该帖的访客(含其他管理员、版主、普通会员)执行存储型 XSS,进而窃取 bbs_token/bbs_admin_token、伪造操作、挂马、水坑攻击。

源码证据

证据 1:入库格式化逻辑------gid==1doctype==0 时跳过转义 文件:xiunobbs_4.0.4/model/post.func.php 行 349-361

php 复制代码
// 写入时格式化
function post_message_fmt(&$arr, $gid) {
	// hook post_message_fmt_start.php
	// 超长内容截取
	$arr['message'] = xn_substr($arr['message'], 0, 2028000);
	// 格式转换: 类型,0: html, 1: txt; 2: markdown; 3: ubb
	$arr['message_fmt'] = htmlspecialchars($arr['message']);
	// 入库的时候进行转换,编辑的时候,自行调取 message, 或者 message_fmt
	$arr['doctype'] == 0 && $arr['message_fmt'] = ($gid == 1 ? $arr['message'] : xn_html_safe($arr['message']));
	$arr['doctype'] == 1 && $arr['message_fmt'] = xn_txt_to_html($arr['message']);
}

第 360 行:$arr['doctype'] == 0 && $arr['message_fmt'] = ($gid == 1 ? $arr['message'] : xn_html_safe($arr['message']));

  • gid==1(管理员)→ message_fmt = message(原始 HTML,无任何过滤)
  • gid!=1message_fmt = xn_html_safe(message)(白名单过滤)

证据 2:前台模板原样输出 message_fmt,无二次转义 文件:xiunobbs_4.0.4/view/htm/post_list.inc.htm 行 64

php 复制代码
<?php echo $_post['message_fmt'];?>

文件:xiunobbs_4.0.4/view/htm/thread.htm 行 61

php 复制代码
<?php echo $first['message_fmt'];?>

两处均为裸 echo,未调用 htmlspecialchars,直接将入库的 message_fmt 输出到 HTML 文档流。

风险等级与结论

风险等级:高危

结论:

  1. 管理员发 HTML 帖(doctype=0)时内容不经任何转义/过滤直入库,前台裸输出,构成存储型 XSS。
  2. 虽然利用前提是管理员权限,但管理员账号常成为攻击目标(弱口令 md5、CSRF 无 token、Cookie 无 Secure),一旦被攻陷即可借此 XSS 横向打击全站用户,形成"管理员失陷→全站水坑"的攻击链。
  3. xn_html_safe 白名单本应作为兜底,但对 gid==1 完全豁免,属于典型的"信任管理员"假设缺陷------管理员账号被盗后该假设即失效。
  4. 修复建议:取消 gid==1 豁免分支,所有 doctype=0 内容统一经 xn_html_safe 白名单过滤;或在模板输出端对 message_fmt 强制二次转义;同时引入 CSP 头降低 XSS 危害。
相关推荐
小兔崽子去哪了10 分钟前
深度学习 2 / CNN 神经网络
后端·python
rannn_11124 分钟前
【Java面试八股】Java基础篇|数据类型、面向对象、关键字、反射、注解、异常、新特性、序列化、设计模式、IO
java·后端·面试
用户83562907805133 分钟前
Python 实现 Word 文档域的插入与管理
后端·python
MacroZheng34 分钟前
阿里又开源了一个神级 Skill 项目!
java·人工智能·后端
Ysx1 小时前
Dify Custom Tool 设计模式:一行提示词接入一张新报表
人工智能·后端
小卿噢1 小时前
malloc 成功 ≠ 你有内存:一次把 OOM 从头测到尾
linux·后端
JoyT1 小时前
Spring AI 2.0 进阶入门:Badcase、Eval 与大模型应用效果优化
后端
wei_shuo1 小时前
KES 事务处理深度实践:隔离级别选择、MVCC机制应用与并发冲突解决
后端
旺仔不是程序员1 小时前
复合索引最左前缀原则:PostgreSQL 的 WHERE 为什么必须命中第一列
数据库·后端·sql
旺仔不是程序员1 小时前
字段类型不一致:PostgreSQL 报错与索引失效的第一元凶
数据库·后端·sql