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

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

缺陷编号：wooyun-2014-075777

漏洞标题：TinyShop修复不当再次注入（全版本）

相关厂商：tinyrise.com

漏洞作者： xiaoL

提交时间：2014-09-17 15:05

修复时间：2014-12-16 15:06

公开时间：2014-12-16 15:06

漏洞类型：SQL注射漏洞

危害等级：中

自评Rank：10

漏洞状态：厂商已经确认

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

Tags标签： 无

4人收藏 收藏
分享漏洞：

漏洞详情

披露状态：

2014-09-17： 细节已通知厂商并且等待厂商处理中
2014-09-19： 厂商已经确认，细节仅向厂商公开
2014-09-22： 细节向第三方安全合作伙伴开放
2014-11-13： 细节向核心白帽子及相关领域专家公开
2014-11-23： 细节向普通白帽子公开
2014-12-03： 细节向实习白帽子公开
2014-12-16： 细节向公众公开

简要描述：

在修复http://www.wooyun.org/bugs/wooyun-2014-068153漏洞时候，处理不当，治标不治本。

详细说明：

public static function getIP()
	{
		if (isset($_SERVER["HTTP_X_FORWARDED_FOR"]))$ip = $_SERVER["HTTP_X_FORWARDED_FOR"];
		elseif (isset($_SERVER["HTTP_CLIENT_IP"])) $ip = $_SERVER["HTTP_CLIENT_IP"];
		elseif (isset($_SERVER["REMOTE_ADDR"])) $ip = $_SERVER["REMOTE_ADDR"];
		elseif (getenv("HTTP_X_FORWARDED_FOR")) $ip = getenv("HTTP_X_FORWARDED_FOR");
		elseif (getenv("HTTP_CLIENT_IP")) $ip = getenv("HTTP_CLIENT_IP");
		elseif (getenv("REMOTE_ADDR")) $ip = getenv("REMOTE_ADDR");
		else $ip = "Unknown";
		return $ip;
	}


之前漏洞中对getip并未做修改，而是对/controller/controller_class.php文件中添加了ip判断，但是使用getip的还有另外一个点（仅剩下此处了）
在protected\classes\Log.php中

public static function op($manager_id,$action,$content)
	{
		$logs = array('manager_id'=>$manager_id,'action'=>$action,'content'=>$content,'ip'=>Chips::getIP(),'url'=>Url::requestUri(),'time'=>date('Y-m-d H:i:s'));
		$model = new Model('log_operation');
		$model->data($logs)->insert();
	}


因此所有使用log::op操作的函数都存在注入。

漏洞证明：

4444.png


555.png

修复方案：

重写getIP函数
在return之前直接检查ip是否规范。
请自查一下log::op函数调用位置。

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

漏洞回应

厂商回应：

危害等级：低

漏洞Rank：4

确认时间：2014-09-19 00:14

厂商回复：

非常感谢您为TinyShop信息安全做的贡献,
我们会尽快一并修复,非常感谢您的支持。

最新状态：

暂无