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

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

缺陷编号：wooyun-2011-02960

漏洞标题：ECShop 2.7.2 红包注入漏洞

相关厂商：ShopEx

漏洞作者： tenzy

提交时间：2011-10-09 20:51

修复时间：2011-11-08 20:51

公开时间：2011-11-08 20:51

漏洞类型：SQL注射漏洞

危害等级：高

自评Rank：15

漏洞状态：厂商已经确认

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

Tags标签： 无

4人收藏 收藏
分享漏洞：

漏洞详情

披露状态：

2011-10-09： 细节已通知厂商并且等待厂商处理中
2011-10-10： 厂商已经确认，细节仅向厂商公开
2011-10-20： 细节向核心白帽子及相关领域专家公开
2011-10-30： 细节向普通白帽子公开
2011-11-09： 细节向实习白帽子公开
2011-11-08： 细节向公众公开

简要描述：

flow.php页面看似intval的红包ID，其实可以注入。以下文章仅从代码推测，并未进行测试，不过这太明显了，就不测试了，各位看官测试失败请跟帖骂娘，死亡节奏技术小组、恶灵战队路过。

详细说明：

/flow.php 1168行开始

$bonus = bonus_info(intval($_GET['bonus']));
        if ((!empty($bonus) && $bonus['user_id'] == $_SESSION['user_id']) || $_GET['bonus'] == 0)
        {
            $order['bonus_id'] = $_GET['bonus'];
        }
        else
        {
            $order['bonus_id'] = 0;
            $result['error'] = $_LANG['invalid_bonus'];
        }
        /* 计算订单的费用 */
        $total = order_fee($order, $cart_goods, $consignee);


$bonus = bonus_info(intval($_GET['bonus']));
上面这一句貌似把$_GET['bonus']给变整形了，没错，这句话没问题，继续向下看，

if ((!empty($bonus) && $bonus['user_id'] == $_SESSION['user_id']) || $_GET['bonus'] == 0)
        {
            $order['bonus_id'] = $_GET['bonus'];
        }


上面代码显示：如果通过GET来的红包ID获取到红包信息并且红包属于你，
就执行下面这句

$order['bonus_id'] = $_GET['bonus'];


看到没有，这里$order['bonus_id']就获得了一个没有过滤的红包ID，
然后
下面这句

/* 计算订单的费用 */
        $total = order_fee($order, $cart_goods, $consignee);


未经过滤的红包ID进入了order_fee函数，来看看这个函数的代码(在/includes/lib_order.php中)，

function order_fee($order, $goods, $consignee)
{
//.....省略若干行，来到643行，
 if (!empty($order['bonus_id']))
    {
        $bonus          = bonus_info($order['bonus_id']);
        $total['bonus'] = $bonus['type_money'];
    }
}


未经过滤的红包ID再次进入bonus_info函数，这个函数也是在上面的这个文件里，
来到1408行

function bonus_info($bonus_id, $bonus_sn = '')
{
    $sql = "SELECT t.*, b.* " .
            "FROM " . $GLOBALS['ecs']->table('bonus_type') . " AS t," .
                $GLOBALS['ecs']->table('user_bonus') . " AS b " .
            "WHERE t.type_id = b.bonus_type_id ";
    if ($bonus_id > 0)
    {
        $sql .= "AND b.bonus_id = '$bonus_id'";
    }
    else
    {
        $sql .= "AND b.bonus_sn = '$bonus_sn'";
    }
    return $GLOBALS['db']->getRow($sql);
}


恰好，字符串大于0，因此注入字符串可以顺利进入SQL，好了，恭喜发财，rank值多给点

漏洞证明：

就不证明，自己看代码

修复方案：

在function bonus_info($bonus_id, $bonus_sn = '')函数里把$bonus_id intval一下不就OK了么

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

漏洞回应

厂商回应：

危害等级：中

漏洞Rank：10

确认时间：2011-10-10 09:53

厂商回复：

感谢提交，我们尽快处理

最新状态：

暂无