ECShop 2.7.2 红包注入漏洞 | WooYun

quasiceo 2014-05-07

展开全文

披露状态：

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);

}