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.所以,上面的判断永远成立。于是这个问题就被“隐藏起来”了。
解决办法
自己想!!!