Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.2
    • Fix Version/s: 1.3.2
    • Component/s: None
    • Labels:
      None

      Description

      when there's a carriage return either before or after the name then you get strange null pointer exceptions in the DeploymentWebservice code as it expects the Xml to be in a particular form and text-nodes in the DOM really make it un-happy.

      This problem seems to be in the following code.

      OMElement zipPart = (OMElement) namePart.getNextOMSibling();

      This code is vulnerable if a carriage return is present after the <name> tag before <package> tag. AFAIK this happens because of the behaviour of Axiom. Axiom creates an OMText node for the carriage return. So when getNextOMSibling() is called instead of <package> element OMText is selected. But since the OMText cannot be cast to OMElement a ClassCastException occurs. This can be avoided by obtaining zipart OMElement with a call to getChild() or any related method which returns an OMElement reference. I can provide a patch for this if necessary.

      reported by Buddhika Chamith and Ciaran.

      see http://markmail.org/thread/pspxrl7qkhpraq4a

        Issue Links

          Activity

          Alexis Midon created issue -
          Alexis Midon made changes -
          Field Original Value New Value
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Alexis Midon made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Alexis Midon made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Alexis Midon made changes -
          Link This issue is duplicated by ODE-594 [ ODE-594 ]
          Alexis Midon made changes -
          Fix Version/s 1.3.2 [ 12313906 ]
          Fix Version/s 1.3.1 [ 12313680 ]
          Alexis Midon made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]

            People

            • Assignee:
              Alexis Midon
              Reporter:
              Alexis Midon
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development