Axiom
  1. Axiom
  2. AXIOM-63

OMXMLStreamReaderValidator incorrectly reports mismatched END_ELEMENT events

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.2.9
    • Fix Version/s: 1.2.12
    • Component/s: API
    • Labels:
      None

      Description

      The code in OMXMLStreamReaderValidator implicitly makes the assumption that the parser state (event) may only change during calls to next(). This is incorrect, because calls to getElementText() and nextTag() also consume events from the parser. This sometimes causes OMXMLStreamReaderValidator to report mismatched END_ELEMENT events although the sequence of events produced by the parent XMLStreamReader is correct. An example of this can be seen in AXIS2-4854.

        Activity

        Andreas Veithen created issue -
        Andreas Veithen made changes -
        Field Original Value New Value
        Project WS-Commons [ 12310250 ] Axiom [ 12311190 ]
        Key WSCOMMONS-563 AXIOM-63
        Affects Version/s Axiom 1.2.9 [ 12313561 ]
        Component/s AXIOM [ 12310703 ]
        Fix Version/s Axiom 1.2.11 [ 12315420 ]
        Jeff Turner made changes -
        Affects Version/s Axiom 1.2.9 [ 12315532 ]
        Fix Version/s Axiom 1.2.11 [ 12315534 ]
        Andreas Veithen made changes -
        Component/s API [ 12313878 ]
        Andreas Veithen made changes -
        Fix Version/s 1.2.12 [ 12316132 ]
        Fix Version/s 1.2.11 [ 12315534 ]
        Andreas Veithen made changes -
        Assignee Andreas Veithen [ veithen ]
        Andreas Veithen made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Hudson added a comment -

        Integrated in ws-axiom-trunk #348 (See https://hudson.apache.org/hudson/job/ws-axiom-trunk/348/)
        AXIOM-63: Fixed a couple of issues in OMXMLStreamReaderValidator that caused it to incorrectly report mismatched END_ELEMENT events when nextTag or getElementText is used. Also refactored the code so that (a) it can be used with plain XMLStreamReader implementations and (b) that it uses XMLStreamReaderWrapper as base class.

        Show
        Hudson added a comment - Integrated in ws-axiom-trunk #348 (See https://hudson.apache.org/hudson/job/ws-axiom-trunk/348/ ) AXIOM-63 : Fixed a couple of issues in OMXMLStreamReaderValidator that caused it to incorrectly report mismatched END_ELEMENT events when nextTag or getElementText is used. Also refactored the code so that (a) it can be used with plain XMLStreamReader implementations and (b) that it uses XMLStreamReaderWrapper as base class.
        Andreas Veithen made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Benson Margulies added a comment -

        Andreas, please drop me an email!

        Show
        Benson Margulies added a comment - Andreas, please drop me an email!
        Andreas Veithen made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Andreas Veithen
            Reporter:
            Andreas Veithen
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development