当前位置:WooYun >> 漏洞信息

漏洞概要 关注数(21) 关注此漏洞

缺陷编号: WooYun-2013-27535

漏洞标题: shopex注册用户任意定义预存款余额或积分

相关厂商: ShopEx

漏洞作者: Code_Sec

提交时间: 2013-07-03 01:23

公开时间: 2013-10-01 01:24

漏洞类型: 设计缺陷/逻辑错误

危害等级: 高

自评Rank: 10

漏洞状态: 厂商已经确认

漏洞来源: http://www.wooyun.org,如有疑问或需要帮助请联系 help@wooyun.org

Tags标签: php源码审核

1人收藏 收藏
分享漏洞:


漏洞详情

披露状态:

2013-07-03: 细节已通知厂商并且等待厂商处理中
2013-07-03: 厂商已经确认,细节仅向厂商公开
2013-07-06: 细节向第三方安全合作伙伴开放(绿盟科技唐朝安全巡航无声信息
2013-08-27: 细节向核心白帽子及相关领域专家公开
2013-09-06: 细节向普通白帽子公开
2013-09-16: 细节向实习白帽子公开
2013-10-01: 细节向公众公开

简要描述:

shopex在注册用户时存在验证不严,导致注册时可以随意定义预存款余额或积分等字段的内容

详细说明:

在文件\core\shop\controller\ctl.passport.php

code 区域
if( !$info = $account->create($_POST,$message) ) {
$this->splash('failed','back',$message,'','',$_POST['from_minipassport']);
}



用户注册的变量都保存在$_POST中,继续看根目录文件index.php

code 区域
filterData($_POST);

require CORE_DIR.'/include_v5/shopCore.php';
return new shopCore();

//过滤字段
function filterData(&$data) {
static $black_list = array(
'order_num','advance','advance_freeze','point_freeze','point_history','member_lv_id',
'point','score_rate','state','role_type','advance_total','advance_consume',
'experience','login_count',
);
foreach($black_list as $v) {
unset($data[$v]);
}
}



显然已经意识到这些重要的字段是不允许通过客户端去赋值的,但是没注意大小写的问题。

shopex.jpg



漏洞证明:

shopex-111.jpg



注册后用户的预存款余额已经变为99999

修复方案:

注意大小写

版权声明:转载请注明来源 Code_Sec@乌云


漏洞回应

厂商回应:

危害等级:高

漏洞Rank:10

确认时间:2013-07-03 12:46

厂商回复:

该问题以前已经发现并处理,相关字段已经做过过滤
非常感谢您为shopex信息安全做的贡献
非常感谢

最新状态:

2013-07-03:确实过滤不严 谢谢@Code_Sec

2013-07-30:该问题已经处理 补丁包: 485safe20130703.zip


漏洞评价:

对本漏洞信息进行评价,以更好的反馈信息的价值,包括信息客观性,内容是否完整以及是否具备学习价值

漏洞评价(共0人评价):
登陆后才能进行评分

评价

  1. 2013-07-03 01:32 | noah ( 普通白帽子 | Rank:384 漏洞数:40 )
    0

    好努力啊。。。

  2. 2013-07-03 02:06 | 乐乐、 ( 普通白帽子 | Rank:868 漏洞数:189 )
    0

    现在还在日站的都是神级人物,扛不住了,睡觉 = =

  3. 2013-07-03 08:50 | 齐迹 ( 普通白帽子 | Rank:804 漏洞数:104 | 重庆地区招聘安全工程师。sec.zbj.com欢迎...)
    0

    高手总是在夜深人静的时候的时候LOL

  4. 2013-07-03 08:52 | Aepl│恋爱 ( 实习白帽子 | Rank:45 漏洞数:15 | Forzen恋爱-不要做你的Guest 只想做的你adm...)
    0

    这个是通用型的?

  5. 2013-07-03 13:28 | xsser 认证白帽子 ( 普通白帽子 | Rank:297 漏洞数:22 | 当我又回首一切,这个世界会好吗?)
    0

    @Aepl│恋爱 2000大洋吧 估计

  6. 2013-07-03 15:51 | dyun ( 普通白帽子 | Rank:102 漏洞数:15 | [code][/code])
    0

    dz~~发财了

  7. 2013-07-03 18:30 | Aepl│恋爱 ( 实习白帽子 | Rank:45 漏洞数:15 | Forzen恋爱-不要做你的Guest 只想做的你adm...)
    0

    如果事通用型的 那就好玩了。。

  8. 2013-07-23 12:55 | 齐迹 ( 普通白帽子 | Rank:804 漏洞数:104 | 重庆地区招聘安全工程师。sec.zbj.com欢迎...)
    0

    学习了。。所以还是白名单靠谱!

登录后才能发表评论,请先 登录