Jive笔记8--查询的Bug

Bug 重现
(1)创建一个Root Forum,everyOne 可读
(2)创建一个Hidden Category,其下创建一个Forum,命名为Forum(2)
(3)创建第三个Category,其下创建一个Forum(3)均为可读。

随意在各个Forum填写数据,Rebuild Index.

进入search.jsp
选中Forum(3),填入Forum(2)中某个message的内容中某些单词,Search,查询结果出来,呵呵。Forum(2)赫然在返回的结果中。

正确的结果应该是:这里选择了Forum(3),也就是在forum(3)中查询,那么应该是只允许返回Forum(3)下的结果(地球人都知道)。


原因分析
看DBQuery.java:executeQuery()的代码:
 // Forum filter -- we can ignore filtering if we are searching all
            // forums in the system.
            if (forums.length != 0 && factory.getForumCount() != forums.length) {
                String[] forumIDs = new String[forums.length];
                for (int j=0; j<forumIDs.length; i++) {
                    forumIDs[j] = Long.toString(forums[j].getID());
                }
                multiFilter.add(new FieldFilter("forumID", forumIDs));
                filterCount++;
            }


问题就在这个sb的if()判断!!

这里牵涉到ForumFactoryProxy.createQuery()里面的一段代码
public Query createQuery() {
        // Special implementation of this method so that we can determine the
        // actual list of forums that the user has permissions to search over.
        ArrayList forumList = new ArrayList();
        for (Iterator iter = forums(); iter.hasNext(); ) {
            forumList.add(iter.next());
        }
        Forum [] forums = new Forum[forumList.size()];
        for (int i=0; i<forums.length; i++) {
            forums[i] = (Forum)forumList.get(i);
        }
        return createQuery(forums);
    }

从这段代码可以看到,获得了用户可访问的forums,传递给DBQuery。
这样似乎可以限制用户在其可访问的forums中搜索。
所以有了DBQuery::executeQuery()中的哪个if(判断),意思是:如果传入的forums和(DBForumFactory)factory.getForumCount()相同,则认为查询所有Forums,不做判断。否则,是在某几个forum中查询。

上面的逻辑很对,,但是加了Category以后就出问题了

首先,为了兼容factory.getForums(),factory.getForumCount()都是调用了RootForumCategory的方法,也就是说,只返回了root forum.

你发现问题了么?

没错,按照上面的判断,我选择了forum(2),此时系统有1个rootForum,显然if(factory.getForumCount() != forums.length)这样的判断是不成立的。所以“在forum(2)中查询这个条件”这个条件就被忽略了。于是,返回了不应该出现的查询结果。


究其根本的原因有2个,(1)jive2.5后来加入的Category以后同时为了保证兼容,而带来的问题;(2)搜索的权限控制问题。

顺便带一句,jive官方网站的论坛没有rootForum.所以,上面的判断永远成立。于是这个问题就被“隐藏起来”了。

解决办法
自己想!!!

 

评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值