mirror of https://github.com/OpenIdentityPlatform/OpenDJ.git

Jean-Noel Rouvignac
06.57.2014 a594eae911ce9693d627af0bf144c879ba97b034
refs
author Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Thursday, November 6, 2014 16:57 +0100
committer Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Thursday, November 6, 2014 16:57 +0100
commita594eae911ce9693d627af0bf144c879ba97b034
tree b986f8a364e60640c5b96ce299557a6000deeca9 tree | zip | gz
parent e54a2046c8f4a385e42d0ce899d1761a2753d401 view | diff
OPENDJ-1623 NullPointerException while doing psearch (with changesonly=false) on cn=changelog

Problem introduced by r11087.

ChangelogBackend.java:
In initializePersistentSearch(), always create the ChangeNumberRange object.
Problem is that with changesOnly=false, this object will be created twice: one from ChangelogBackend.registerPersistentSearch() and another from ChangelogBackend.search().
I really think we should have a unique Backend.preSearch() method which is always called regardless of whether this is a persistent search or not.
This would allow to factorize search initialization in a single place, before it can be used to return entries by the initial and persistent phases.
1 files modified
17 ■■■■■ changed files
opendj3-server-dev/src/server/org/opends/server/backends/ChangelogBackend.java 17 ●●●●● diff | view | raw | blame | history