ThinkSNS某功能缺陷可导致重要信息泄漏

漏洞详情

披露状态:

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

厂商回复:

感谢提交!

最新状态:

暂无


漏洞评价: