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

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

缺陷编号：wooyun-2013-033072

漏洞标题：JEEBBS v3.0 一处逻辑漏洞和一处定向XSS

相关厂商：JEECMS

漏洞作者： 园长

提交时间：2013-07-31 23:10

修复时间：2013-10-29 23:11

公开时间：2013-10-29 23:11

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

危害等级：中

自评Rank：6

漏洞状态：厂商已经确认

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

Tags标签： 无

4人收藏 收藏
分享漏洞：

漏洞详情

披露状态：

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

简要描述：

两个两个一起发吧，凑一对。Jeebbs 一处逻辑漏洞和一处定向XSS。轻易的获取对方cookie...明文密码...

详细说明：

Jeebbs基本上都没有判断referer，没有token，可造成csrf。
1、不科学的积分漏洞
积分兑换方法如下：

@RequestMapping(value = "/member/creditExchange.jspx")
	public void creditExchange(Integer creditIn, Integer creditOut,
			Integer creditOutType, Integer miniBalance, String password,
			HttpServletRequest request, HttpServletResponse response) {
}


可以看到这里直接用了SpringMvc注入参数，而这些参数恰恰是控制程序逻辑的关键。比如构建如下URL，通过GET或者POST方式都能恶意修改用户的积分：
http://localhost/jeebbs/member/creditExchange.jspx?creditIn=26&creditOut=-27600&creditOutType=1&miniBalance=-10000000&password=wooyun
因为他的逻辑是这么写的：

if(user.getPoint()-creditOut>miniBalance){
    balance=true;
}else{
    flag=1;
}


从User对象里面取出积分的数值，而积分兑换威望具体需要多少是在确定兑换关系后由ajax去后台计算出来的，提交的时候也没有验证计算的结果有没有被客户端改过。其中的creditOut和miniBalance都是我们可控的。所以这个等式不管在什么情况下我们都可以让它成立。
Cookie不加密我的明文密码又出来晒太阳了。

1.png


2.png


2、打招呼XSS

3.png


逻辑有做判断：1、用户名为空。2、不允许发送消息给自己。3、用户名不存在。
在控制层并没有做过滤：

4.png


在调用com.jeecms.bbs.manager.impl. BbsMessageMngImpl.java的sendMsg方法的时候依旧没有过滤。到最终的BbsMessageDaoImpl 的save方法还是没有过滤就直接储存了;
一般性的做法，关系到用户交互的地方最好做referer和xss过滤检测，控制层负责收集数据的同时最好处理下用户的请求，就算controller不处理起码在service层做下处理吧。

5.png


漏洞证明：

2.png


5.png


修复方案：

修改下逻辑验证，不要信任客户端传递的参数。

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

漏洞回应

厂商回应：

危害等级：低

漏洞Rank：5

确认时间：2013-08-01 15:44

厂商回复：

感谢对jeecms系列软件提供漏洞信息，我们会在本周内修复

最新状态：

暂无

漏洞评价：

评论

  1. 2013-08-01 07:41 | Blueheart ( 路人 | Rank:0 漏洞数:2 | 路人甲乙丙丁)

    园长你又调皮了，捅人家菊花一次就算了，还一日三次！

  2. 2013-08-01 09:54 | 园长 ( 普通白帽子 | Rank:134 漏洞数:14 | 你在身边就是缘，缘分写在数据库里面。)

    @Blueheart 没办法工作压力太大，捅捅菊花解解闷。