Acode编辑器中的HTML注入问题分析与修复方案

问题背景

Acode是一款基于Cordova框架开发的移动端代码编辑器应用。在最新版本中发现了一个严重的安全问题，攻击者可通过精心构造的文件名触发HTML注入，进而实现跨站脚本攻击(XSS)。由于Cordova应用具有完整的本地API访问权限，该问题可能导致设备文件系统被任意读写、数据泄露等高危后果。

问题原理分析

问题根源位于alert.js对话框组件的实现方式。该组件直接使用innerHTML属性渲染提示文本，而未对用户输入进行任何处理。当攻击者能够控制提示文本内容时，即可注入任意HTML标签和JavaScript代码。

关键问题点：

1. alert.js中直接使用innerHTML渲染文本内容
2. checkFiles.js组件将用户可控的文件名直接拼接进提示信息
3. 通过Android Intent机制可控制文件名参数

攻击链示例：

1. 恶意应用通过Intent发送包含HTML标签的文件名
2. Acode打开文件后，攻击者主动撤销文件访问权限
3. 触发文件检查逻辑时，系统生成包含恶意文件名的提示信息
4. 恶意HTML通过innerHTML解析执行任意JavaScript

问题影响评估

该问题属于高危级别，具有以下特征：

• 攻击者可完全控制提示框内容渲染
• 在Cordova环境下可调用所有设备API
• 无需用户交互即可触发(通过Intent自动执行)
• 影响最新发布的1.10.5版本

典型攻击场景包括：

• 获取编辑器内打开的文件内容
• 修改设备上的项目文件
• 通过SFTP/FTP插件连接攻击者服务器
• 安装恶意插件或执行系统命令

修复方案建议

方案一：使用textContent替代innerHTML

最彻底的解决方案是避免使用innerHTML，改用textContent属性。这会完全禁用HTML解析，从根本上杜绝XSS可能性。

优势：

• 实现简单，一行代码修改
• 完全消除HTML注入风险
• 无需引入额外依赖

劣势：

• 会丢失所有HTML格式渲染能力

方案二：引入DOMPurify过滤

如需保留HTML显示能力，可采用专业的DOM净化库对输入进行过滤。

优势：

• 保留安全的HTML标签(如链接、加粗等)
• 专业的安全过滤规则
• 对现有功能影响较小

劣势：

• 增加应用体积
• 需要维护额外依赖
• 过滤规则配置不当可能遗留风险

最佳实践建议

1. 对所有用户输入渲染点进行检查
2. 建立安全的字符串渲染规范：
   • 默认使用textContent
   • 必须使用HTML时采用DOMPurify
3. 实现自动化安全检测：
   • 静态代码分析检测innerHTML使用
   • 动态XSS测试用例
4. 对Cordova插件进行权限最小化配置

总结

Acode编辑器中的这个案例典型展示了移动混合应用中DOM操作的潜在风险。开发者需要特别注意：在具有系统级权限的Webview环境中，任何客户端问题都可能造成严重后果。通过采用安全的字符串渲染策略和严格的输入验证，可以有效预防此类安全问题。