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

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

缺陷编号：wooyun-2014-082805

漏洞标题：PHPMyWind最新版代码执行漏洞

相关厂商：phpmywind.com

漏洞作者： xfkxfk

提交时间：2014-11-11 19:01

修复时间：2014-12-30 14:44

公开时间：2014-12-30 14:44

漏洞类型：命令执行

危害等级：高

自评Rank：15

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

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

Tags标签： 无

4人收藏 收藏
分享漏洞：

漏洞详情

披露状态：

2014-11-11： 细节已通知厂商并且等待厂商处理中
2014-11-16： 厂商主动忽略漏洞，细节向第三方安全合作伙伴开放
2015-01-10： 细节向核心白帽子及相关领域专家公开
2015-01-20： 细节向普通白帽子公开
2015-01-30： 细节向实习白帽子公开
2014-12-30： 细节向公众公开

简要描述：

PHPMyWind最新版代码执行漏洞，可GetShell

详细说明：

PHPMyWind最新版代码执行漏洞，可GetShell
PHPMyWind的整个系统权限分为：

超级管理员
站点管理员
文章发布员
普通用户


此漏洞点在文章发布员权限，有点门槛，但是门槛不高，一个文章发布员还是很好拿下的
文件good_save.php

//添加商品信息
if($action == 'add')
{
	//栏目权限验证
	IsCategoryPriv($classid,'add');
	//初始化参数
	if(!isset($typeid))        $typeid = '-1';
	if(!isset($brandid))       $brandid = '-1';
	if(!isset($attrid))        $attrid = '';
	if(!isset($attrvalue))     $attrvalue = '';
	if(!isset($flag))          $flag   = '';
	if(!isset($picarr))        $picarr = '';
	if(!isset($rempic))        $rempic = '';
	if(!isset($remote))        $remote = '';
	if(!isset($autothumb))     $autothumb = '';
	if(!isset($autodesc))      $autodesc = '';
	if(!isset($autodescsize))  $autodescsize = '';
	if(!isset($autopage))      $autopage = '';
	if(!isset($autopagesize))  $autopagesize = '';
	$attrstr = '';
......
//商品属性
	if(is_array($attrid) && is_array($attrvalue))
	{
		//组成商品属性与值
		$attrstr .= 'array(';
		$attrids = count($attrid);
		for($i=0; $i<$attrids; $i++)
		{
			$attrstr .= '"'.$attrid[$i].'"=>'.'"'.$attrvalue[$i].'"'; 
			if($i < $attrids-1)
			{
				$attrstr .= ',';
			}
		}
		$attrstr .= ');';
	}
......
$sql = "INSERT INTO `$tbname` (classid, parentid, parentstr, typeid, typepid, typepstr, brandid, brandpid, brandpstr, title, colorval, boldval, flag, goodsid, payfreight, weight, attrstr, marketprice, salesprice, housenum, housewarn, warnnum, integral, source, author, linkurl, keywords, description, content, picurl, picarr, hits, orderid, posttime, checkinfo {$fieldname}) VALUES ('$classid', '$parentid', '$parentstr', '$typeid', '$typepid', '$typepstr', '$brandid', '$brandpid', '$brandpstr', '$title', '$colorval', '$boldval', '$flag', '$goodsid', '$payfreight', '$weight', '$attrstr', '$marketprice', '$salesprice', '$housenum', '$housewarn', '$warnnum', '$integral', '$source', '$author', '$linkurl', '$keywords', '$description', '$content', '$picurl', '$picarr', '$hits', '$orderid', '$posttime', '$checkinfo' {$fieldvalue})";
	if($dosql->ExecNoneQuery($sql))
	{
		header("location:$gourl");
		exit();
	}


这里在添加商品时，商品的属性内容进入数据库后是一个数组，以数组形式存储的
如：

array("1"=>"111","2"=>"111");


文件goodsshow.php

<div class="TwoOfTwos"> 
		<!-- 详细区域开始 -->
		<div class="goodsConts">
			<?php
			//检测文档正确性
			$r = $dosql->GetOne("SELECT * FROM `#@__goods` WHERE id=$id");
			if(@$r)
			{
			//增加一次点击量
			$dosql->ExecNoneQuery("UPDATE `#@__goods` SET hits=hits+1 WHERE id=$id");
			$row = $dosql->GetOne("SELECT * FROM `#@__goods` WHERE id=$id");
			?>
			<h1 class="title"><?php echo $row['title']; ?></h1>
......
<!-- 商品信息开始 -->
				<div class="fr">
					<ul class="tb-meta">
						<li> <span>市场价</span><strong class="lt"><?php echo $row['marketprice']; ?></strong>元 </li>
						<li> <span>促销价</span><strong class="price"><?php echo $row['salesprice']; ?></strong>元 </li>
						<li> <span>浏览数</span><?php echo $row['hits']; ?> 次</li>
						<li> <span>配　送</span><?php if($row['payfreight']==0){echo '买家承担运费';}else{echo '商家承担运费';} ?></li>
					</ul>
					<div class="tb-skin">
						<p class="tb-note-title"><span>请选择您要的商品信息</span><a href="shoppingcart.php" class="end">结算购物车</a></p>
						<form name="gform" id="gform" method="post">
							<dl class="tb-prop">
							<?php
							//将商品属性id与值组成数组
							$rowattr = String2Array($row['attrstr']);
							$row2 = $dosql->Execute('SELECT * FROM `#@__goodsattr` WHERE `goodsid`='.$row['typeid']." AND `checkinfo`=true");
							if($dosql->GetTotalRow() > 0)
							{
								$i = 0;
								while($row2 = $dosql->GetArray())
								{
							?>


从这里看出，这里在显示商品信息时，看这里

//将商品属性id与值组成数组
$rowattr = String2Array($row['attrstr']);


跟进String2Array函数，文件common.func.php

/*字符串转数组*/
if(!function_exists('String2Array'))
{
	function String2Array($data)
	{
		if($data == '') return array();
		@eval("\$array = $data;");
		return $array;
	}
}


这里强制将字符串转成数组，使用eval函数
整个过程由于商品属性信息进入数据库时，没有过滤，在现实商品信息时，也没有处理
导致在通过eval时，导致代码执行漏洞

漏洞证明：

文章发布员可以添加商品：

1.png


在文章属性处添加商品属性：
如：

111|222{${phpinfo()}}


然后在商品展示页面：

2.png

修复方案：

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

漏洞回应

厂商回应：

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

忽略时间：2014-12-30 14:44

厂商回复：

最新状态：

暂无