Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-1144

wf: NPE waiting to happen in JCRExpressionStore$StoreIterator.next() logging.

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 3.0 RC4
    • None
    • None
    • None

      Looking through the StoreIterator code I noticed the logging statement potentially causes a NPE. It is reasonable to expect that the call to fetchNext() above the logging statement will return null from time to time. If logging is enabled, then this will produce a NPE.

      The logging statement should look something like:
      log.debug("next() is "+ ((this.next != null) ? this.next.getId().toString() : "null"));

        Acceptance criteria

              jmettraux John Mettraux
              chris_miner Chris Miner
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Bug DoR
                  Task DoD