Discuz X3.1任务刷积分漏洞分析与修复方案

Discuz X3.1任务刷积分漏洞分析与修复方案
在互联网技术快速发展的今天，论坛系统作为用户互动的重要平台，其安全性一直是开发者关注的重点。本文将详细介绍Discuz X3.1中存在的一项任务刷积分漏洞，并提供相应的修复方法。
该漏洞最早被发现于4月份之前，虽然新版本可能已经对此问题进行了修复，但近期有VIP会员反馈称，仍然遇到了通过QQ互联绑定刷积分的情况。为了帮助更多用户了解这一问题，我们将乌云平台上关于Discuz X3.1任务刷积分漏洞的方法整理如下：
当用户完成任务时（home.php?mod=draw&do=view&id=xx），系统对任务先前的状态缺乏必要的判断。


function draw($id) {
    global $_G;
    if(!($this->task = C::t('common_task')->fetch_by_uid($_G['uid'], $id))) {
        showmessage('task_nonexistence');
    } elseif($this->task['status'] != 0) {
        showmessage('task_not_underway');
    } elseif($this->task['tasklimits'] && $this->task['achievers'] >= $this->task['tasklimits']) {
        return -1;
    }
}

上述代码展示了任务完成逻辑中的主要部分。然而，进一步分析后可以发现，在获取任务奖励的过程中，缺少了对任务是否已领取的关键判断。这导致攻击者无需真正领取任务即可无限次获取奖励。
为验证此漏洞，可按照以下步骤操作：
1. 新建一个任务，例如选择红包类任务。
2. 不要申请任务，直接进入任务详细页面（如：home.php?mod=task&do=view&id=2）。
3. 将地址改为领取任务奖励形式（如：home.php?mod=task&do=draw&id=2）。
4. 刷新该页面，即可不断获得奖励。
针对这一问题，建议采取以下修复措施：
在source\\class\\class_task.php中的draw函数部分增加任务是否已被领取的判断条件：


elseif($this->task['status'] != '0') {
    showmessage('task_not_underway');
}

完成上述修改后，再次尝试利用漏洞时，系统将提示“不是进行中的任务”，从而有效阻止非法刷分行为。
总结来说，对于任何软件系统而言，安全始终是第一位的。定期检查和更新程序代码，及时修补已知漏洞，是确保论坛系统稳定运行不可或缺的一部分。希望本文能为广大Discuz用户带来一定参考价值。

推荐阅读