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

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

缺陷编号：wooyun-2013-046845

漏洞标题：PHPSHE SQL注入2（全版通杀）

相关厂商：phpshe.com

漏洞作者： xfkxfk

提交时间：2013-12-24 10:24

修复时间：2014-03-21 10:26

公开时间：2014-03-21 10:26

漏洞类型：SQL注射漏洞

危害等级：高

自评Rank：20

漏洞状态：漏洞已经通知厂商但是厂商忽略漏洞

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

Tags标签： 无

4人收藏 收藏
分享漏洞：

漏洞详情

披露状态：

2013-12-24： 细节已通知厂商并且等待厂商处理中
2013-12-29： 厂商主动忽略漏洞，细节向第三方安全合作伙伴开放
2014-02-22： 细节向核心白帽子及相关领域专家公开
2014-03-04： 细节向普通白帽子公开
2014-03-14： 细节向实习白帽子公开
2014-03-21： 细节向公众公开

简要描述：

PHPSHE SQL注入，全版本通杀。

详细说明：

在/module/index/product.php文件
在商品咨询，和商品评价时未对参数请求头部信息过滤，导致sql注入。
看看商品评价的代码：

//#####################@ 商品评价 @#####################//
	case 'commentadd':
		if (isset($_p_pesubmit)) {
			$info['product_id'] = intval($_g_id);
			$info['comment_text'] = pe_texthtml(pe_dbhold($_p_comment_text));
			$info['comment_atime'] = time();
			$info['user_id'] = $_s_user_id;
			$info['user_name'] = $_s_user_name;
			$info['user_ip'] = pe_ip();
			if ($db->pe_insert('comment', $info)) {
				product_num("commentnum", $info['product_id']);
				$result = true;
				$info['comment_atime'] = pe_date($info['comment_atime']);
				$info['comment_text'] = htmlspecialchars($_p_comment_text);


有一个info[‘user_ip’]的提交内容，这个是从pe_ip函数获取：/include/function/global.func.php：

//获取ip
function pe_ip()
{
    if (isset($_SERVER)){
        if (isset($_SERVER["HTTP_X_FORWARDED_FOR"])){
            $realip = $_SERVER["HTTP_X_FORWARDED_FOR"];
        } else if (isset($_SERVER["HTTP_CLIENT_IP"])) {
            $realip = $_SERVER["HTTP_CLIENT_IP"];
        } else {
            $realip = $_SERVER["REMOTE_ADDR"];
        }
    } else {
        if (getenv("HTTP_X_FORWARDED_FOR")){
            $realip = getenv("HTTP_X_FORWARDED_FOR");
        } else if (getenv("HTTP_CLIENT_IP")) {
            $realip = getenv("HTTP_CLIENT_IP");
        } else {
            $realip = getenv("REMOTE_ADDR");
        }
    }
    return $realip;
}


这里取到的IP没有过滤，直接带人pe_insert语句。

public function pe_insert($table, $set)
	{
		//处理设置语句
		$sqlset = $this->_doset($set);
		return $this->sql_insert("insert into `".dbpre."{$table}` {$sqlset}");
	}
//处理设置语句
	protected function _doset($set)
	{
		if (is_array($set)) {
			foreach ($set as $k => $v) {
				$set_arr[] = "`{$k}` = '{$v}'";
			}
			$sqlset = 'set '.implode($set_arr, ' , ');
		}
		else {
			$sqlset = "set {$set}";
		}
		return $sqlset;
	}
public function sql_insert($sql)
	{
		$this->query($sql);
		if ($insert_id = mysql_insert_id()) {
			return $insert_id;
		}
		else {
			$result = mysql_affected_rows();
			return $result > 0 ? $result : 0;
		}
	}


最后导致insert型sql注入。

漏洞证明：

我们来评价商品，然后抓包修改头部，添加Client-ip：

4.png


可以看到我们添加的内容，执行成功了，返回的result为true。
看看sql语句的执行情况，是否正常执行了：

5.png

修复方案：

过滤。

版权声明：转载请注明来源 xfkxfk@乌云

漏洞回应

厂商回应：

危害等级：无影响厂商忽略

忽略时间：2014-03-21 10:26

厂商回复：

最新状态：

2014-05-19：感谢@xfkxfk 提供代码审计，sql注入漏洞已修复！

漏洞评价：

评论