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

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

缺陷编号：wooyun-2014-073727

漏洞标题：TinyShop 越权操作两处

相关厂商：tinyrise.com

漏洞作者： zxx

提交时间：2014-08-26 12:32

修复时间：2014-11-24 12:34

公开时间：2014-11-24 12:34

漏洞类型：设计缺陷/逻辑错误

危害等级：高

自评Rank：15

漏洞状态：厂商已经确认

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

Tags标签： 无

4人收藏 收藏
分享漏洞：

漏洞详情

披露状态：

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

简要描述：

sql语句中的限定条件不足

详细说明：

问题一：
任意收货地址删除，/protected/controllers/ucenter.php中

public function address_del()
    {
        $id = Filter::int(Req::args("id"));
        $this->model->table("address")->where("id=$id")->delete();//where条件中没限定user_id
        $this->redirect("address");
    }


问题二：
任意订单信息查询，在/protected/controllers/simple.php中

public function address_other(){
        Session::set("order_status",Req::args());
        $this->layout = '';
        $id = Filter::int(Req::args("id"));
        if($id){
            $model = new Model("address");
            $data = $model->where("id = $id")->find();//同样没有对user_id进行限制
            $this->redirect("address_other",false,$data);
        }
        else $this->redirect();
    }

漏洞证明：

问题一：
poc：
http://localhost/index.php?con=ucenter&act=address_del&id=3 后面id随便改，想删谁删谁
问题二：
poc：
http://localhost/index.php?con=simple&act=address_other&id=1 后面id对应地址号，可以遍历得到用户信息

QQ截图20140824210618.png


这里的这条记录是系统带的，user_id=1，说明可以随便看

修复方案：

加入user_id限定一下

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

漏洞回应

厂商回应：

危害等级：中

漏洞Rank：6

确认时间：2014-08-28 16:20

厂商回复：

非常感谢您为TinyShop信息安全做的贡献，此问题已经360库带反馈，不过仍然感谢你的支持。

最新状态：

暂无