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

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

缺陷编号:wooyun-2014-049453

漏洞标题:ThinkSNS某功能缺陷可导致重要信息泄漏

相关厂商:ThinkSNS

漏洞作者: 齐迹

提交时间:2014-01-21 18:02

修复时间:2014-04-21 18:02

公开时间:2014-04-21 18:02

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

危害等级:低

自评Rank:3

漏洞状态:厂商已经确认

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

Tags标签:

4人收藏 收藏
分享漏洞:


漏洞详情

披露状态:

2014-01-21: 细节已通知厂商并且等待厂商处理中
2014-01-25: 厂商已经确认,细节仅向厂商公开
2014-01-28: 细节向第三方安全合作伙伴开放
2014-03-21: 细节向核心白帽子及相关领域专家公开
2014-03-31: 细节向普通白帽子公开
2014-04-10: 细节向实习白帽子公开
2014-04-21: 细节向公众公开

简要描述:

ThinkSNS某功能缺陷可导致用户上传的身份证信息泄漏。

详细说明:

问题发生在
/apps/public/Lib/Action/AccountAction.class.php
行483

public function doAuthenticate(){
....
$data['attach_id'] = t($_POST['attach_ids']);


这里附件ID 是通过post方式传递的
意味着主要修改为别人附件ID 岂不是??
附件可是有用户认证的身份证。遍历一遍 呵呵
其实导致这个问题还有很多地方。这里我就不一一列举了!你们自己revrew一下把

漏洞证明:

接着看展示的地方
393行

public function authenticate(){
$auType = model('UserGroup')->where('is_authenticate=1')->findall();
$this->assign('auType', $auType);
$verifyInfo = D('user_verified')->where('uid='.$this->mid)->find();
if($verifyInfo['attach_id']){
$a = explode('|', $verifyInfo['attach_id']);
foreach($a as $key=>$val){
if($val !== "") {
$attachInfo = D('attach')->where("attach_id=$a[$key]")->find();
$verifyInfo['attachment'] .= $attachInfo['name'].'&nbsp;<a href="'.getImageUrl($attachInfo['save_path'].$attachInfo['save_name']).'" target="_blank">下载</a><br />';
}
}
}


接把ID解开 查询出来
看看字段大小!attach_id varchar(255)
一次遍历50个ID还是可以的

修复方案:

这里查询附件 应该判断一下权限 或者附件类型!

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


漏洞回应

厂商回应:

危害等级:低

漏洞Rank:5

确认时间:2014-01-25 08:54

厂商回复:

感谢提交!

最新状态:

暂无