记一次S-CMS逻辑漏洞审计

S-CMS逻辑漏洞审计报告

漏洞概述在审计S-CMS企业建站系统时，发现其积分转余额功能存在逻辑漏洞。攻击者可利用该漏洞，通过篡改表单数据，实现积分与余额的非法转换，导致账户余额异常增加。此漏洞仅在系统开启积分转余额功能时触发，影响范围限于相关功能模块。

漏洞细节

漏洞位置
文件路径：/member/index.php
代码段：第22行至第70行
漏洞触发条件
系统需开启积分转余额功能（配置界面如图1所示）。
漏洞原理
代码逻辑缺陷：在处理积分转余额请求时，系统未对用户提交的转换金额进行充分验证（如图2）。攻击者可修改表单中的转换金额字段（如amount），绕过前端或后端校验，直接提交恶意数据。
数据库操作漏洞：在执行余额更新前，系统未严格校验用户积分是否足够或转换比例是否合法（如图3）。例如，用户可通过提交负值或超量数值，导致余额非预期增加。
漏洞利用步骤
步骤1：登录系统并进入积分转余额功能页面。
步骤2：拦截请求（如通过Burp Suite），修改表单中的转换金额为任意值（如负数或超大数值）。
步骤3：提交请求后，系统直接更新数据库余额字段，导致账户余额异常增加（如图4）。

漏洞分析

代码缺陷
原始代码中，系统仅简单检查用户是否登录（如图5），未对转换金额的合法性（如正数、积分余额充足性）进行验证。
修改后的代码（如图6）显示，系统在数据库操作前未输出关键校验信息，导致漏洞隐蔽性增强。
影响范围
攻击者可无限增加账户余额，破坏系统经济平衡。
若系统关联支付或提现功能，可能导致资金损失。

修复建议

输入验证
在接收用户输入时，严格校验转换金额是否为正数且符合积分余额范围。
if (!is_numeric($_POST['amount']) || $_POST['amount'] <= 0) { die("无效的转换金额");}if ($_POST['amount'] > $user_integral) { die("积分不足");}
事务处理
使用数据库事务确保积分扣除与余额增加的原子性，避免部分失败导致数据不一致。
日志记录
记录所有转换操作的详细日志，便于追踪异常行为。
代码审计
对类似功能（如积分兑换、余额支付）进行全面检查，避免同类逻辑漏洞。

总结该漏洞源于系统对用户输入的信任过度及校验缺失，属于典型的业务逻辑缺陷。通过修复输入验证、增强事务处理及完善日志机制，可有效防范此类攻击。建议开发者在功能实现中始终遵循“最小权限原则”和“防御性编程”思想。

热门标签