Discuz x3.1任务刷积分漏洞说明和修复方法

discuz教程强烈建议 2024-02-07 14:05 299 0

　　这个是4月份之前的漏洞的，不知道新版本有没有修复这个问题，但最新一个VIP会员反馈遇到刷积分的问题，可能是QQ互联绑定刷积分导致的，所以这里就把乌云上面的discuz x3.1任务刷积分漏洞的方法拿过来跟大家分享下。, ,在完成任务时（home.php?mod=draw&do=view&id=xx），任务先前的状态缺少判断,完成任务的链接形如：home.php?mod=draw&do=view&id=xx,这个地址最终在 source\class\class_task.php 中被处理, ,约第370行：,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;, , }, ,……,复制代码, ,之后就是获得任务奖励了,总觉得上面这一段少了些什么判断？我们对比下其他代码, ,约第473行：,function giveup($id) {, , global $_G;, , if($_GET[‘formhash’] !=FORMHASH) {, , showmessage(‘undefined_action’);, , } elseif(!($this->task=C::t(‘common_task’)->fetch_by_uid($_G[‘uid’], $id))) {, , showmessage(‘task_nonexistence’);, , } elseif($this->task[‘status’] !=‘0’) {, , showmessage(‘task_not_underway’);, , },复制代码, ,这一段是放弃任务的判断，我们看到如果 $this->task[‘status’] !=‘0’,就是说任务没有开始的时候，是不能放弃任务的。, ,但是，在上面那段获取任务奖励的代码中，并没有判断任务是否开始，造成了无需领取任务，就可以无限次数获取奖励。, ,此漏洞还可以用于强行获取由于用户组不符，没有权限领取的任务的奖励。, ,漏洞详细利用，请见漏洞证明。, ,漏洞证明：,1、新建一个任务，就选择红包类任务吧, ,2、此时千万不要申请任务，而是进入任务详细页面（完成之后就不能刷了）,home.php?mod=task&do=view&id=2,这样就能看到任务详情了，任务的奖励是威望+1。,我们把地址改为领取任务奖励,home.php?mod=task&do=draw&id=2,打开这个地址，获得了威望+1。, ,不断刷新这个页面，即可不断获得奖励。, ,修复方案：, ,在 source\class\class_task.php 中的 draw 函数部分，加入任务是否领取的判断,即加上,……, ,elseif($this->task[‘status’] !=‘0’) {, , showmessage(‘task_not_underway’);, ,},复制代码, ,这样，再次使用漏洞时，就会提示：不是进行中的任务,这个是4月份之前的漏洞的，不知道新版本有没有修复这个问题，但最新一个VIP会员反馈遇到刷积分的问题，可能是QQ互联绑定刷积分导致的，所以这里就把乌云上面的discuz x3.1任务刷积分漏洞的方法拿过来跟大家分享下。在完成任务时（home.php?mod=draw&do=view&id=xx），任务先前的状态缺少判断完成任务的链接形如：home.php?mod=draw&do=view&id=xx 这个地址最终在 source\class\class_task.php 中被处理约第370行： 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; } …… 复制代码之后就是获得任务奖励了总觉得上面这一段少了些什么判断？我们对比下其他代码约第473行： function giveup($id) { global $_G; if($_GET[‘formhash’] !=FORMHASH) { showmessage(‘undefined_action’); } elseif(!($this->task=C::t(‘common_task’)->fetch_by_uid($_G[‘uid’], $id))) { showmessage(‘task_nonexistence’); } elseif($this->task[‘status’] !=‘0’) { showmessage(‘task_not_underway’); } 复制代码这一段是放弃任务的判断，我们看到如果 $this->task[‘status’] !=‘0’,就是说任务没有开始的时候，是不能放弃任务的。但是，在上面那段获取任务奖励的代码中，并没有判断任务是否开始，造成了无需领取任务，就可以无限次数获取奖励。此漏洞还可以用于强行获取由于用户组不符，没有权限领取的任务的奖励。漏洞详细利用，请见漏洞证明。漏洞证明： 1、新建一个任务，就选择红包类任务吧 2、此时千万不要申请任务，而是进入任务详细页面（完成之后就不能刷了） home.php?mod=task&do=view&id=2 这样就能看到任务详情了，任务的奖励是威望+1。我们把地址改为领取任务奖励 home.php?mod=task&do=draw&id=2 打开这个地址，获得了威望+1。不断刷新这个页面，即可不断获得奖励。修复方案：在 source\class\class_task.php 中的 draw 函数部分，加入任务是否领取的判断即加上 …… elseif($this->task[‘status’] !=‘0’) { showmessage(‘task_not_underway’); } 复制代码这样，再次使用漏洞时，就会提示：不是进行中的任务

　　标签：discuz修复漏洞积分

　　Discuz教程：X3.2统计代码添加无效出错的解决方法

　　Discuz X3.2/X2.5缩略图添加水印方法

时间：(2024-02-07 14:05:15)
本站资源均来自互联网或会员发布，如果不小心侵犯了您的权益请与我们联系。我们将立即删除！谢谢！