漏洞详情

披露状态：

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

厂商回复：

感谢提交！

最新状态：

暂无

---

漏洞评价：