FieldRef::parse does not completely initialize the FieldRef

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Fixed
    • Priority: Major - P3
    • 3.6.5, 3.7.4
    • Affects Version/s: None
    • Component/s: None
    • None
    • Fully Compatible
    • ALL
    • v3.6
    • Query 2018-04-09
    • 91
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      There are two problems:

      1) FieldRef::parse() is completely responsible for initializing a newly constructed FieldRef when using the FieldRef(StringData) constructor. Right now, it's not initializing _cachedSize, resulting in undefined behavior, including test cases that fail on some platforms but not others.

      2) When calling parse() on an existing FieldRef, not all of its internals get reset. In particular, parsing the empty string puts the FieldRef in an inconsistent state.

              Assignee:
              Justin Seyster
              Reporter:
              Justin Seyster
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: