Lucene - Core
  1. Lucene - Core
  2. LUCENE-1457

There are a few binary search implmentations in lucene that suffer from a now well known overflow bug

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.4.1, 2.9
    • Component/s: None
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      http://googleresearch.blogspot.com/2006/06/extra-extra-read-all-about-it-nearly.html

      The places I see it are:

      MultiSearcher.subSearcher(int)
      TermInfosReader.getIndexOffset(Term)
      MultiSegmentReader.readerIndex(int, int[], int)
      MergeDocIDRemapper.remap(int)

      I havn't taken much time to consider how likely any of these are to overflow. The values being averaged would have to be very large. That would rule out possible problems for at least a couple of these, but how about something like the MergeDocIDRemapper? Is there a document number that could be reached that has a chance of triggering this bug? If not we can close this and have a record of looking into it.

        Activity

        Hide
        Michael McCandless added a comment -

        Committed revision 723149.

        I fixed these cases, plus one more in FieldCache. Thanks Mark!

        Show
        Michael McCandless added a comment - Committed revision 723149. I fixed these cases, plus one more in FieldCache. Thanks Mark!
        Hide
        Michael McCandless added a comment -

        Reopening for backport to 2.4.1.

        Show
        Michael McCandless added a comment - Reopening for backport to 2.4.1.
        Hide
        Michael McCandless added a comment -

        Committed revision 745798 on 2.4 branch.

        Show
        Michael McCandless added a comment - Committed revision 745798 on 2.4 branch.

          People

          • Assignee:
            Unassigned
            Reporter:
            Mark Miller
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development