Petals CDK

Potential NPE detected in "cdk-core" part ...

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 5.1.1
  • Fix Version/s: 5.1.2
  • Component/s: runtime
  • Security Level: Public
  • Description:
    Hide

    According to JBI specification (version 1.0; page 186/226) the "getContent()"
    methode of the "NormalizeMessage" class may return "null" value.
    But in the "SourceUtilImpl" class of package "ow2.petals.component.framework.util"
    this situation is not checked.
    In practice, A usecase based on bc-soap binding component that receives a SOAP
    message with an empthy body part (allowed according WS-I Basic profile 2.0 and
    other related specifications) will throw a NPE due to a NormalizeMessage with
    empty content.

    temp fix on branch version : see commit n° 16147=>16148, in tdejean's petals sandbox
    folder

    Show
    According to JBI specification (version 1.0; page 186/226) the "getContent()" methode of the "NormalizeMessage" class may return "null" value. But in the "SourceUtilImpl" class of package "ow2.petals.component.framework.util" this situation is not checked. In practice, A usecase based on bc-soap binding component that receives a SOAP message with an empthy body part (allowed according WS-I Basic profile 2.0 and other related specifications) will throw a NPE due to a NormalizeMessage with empty content. temp fix on branch version : see commit n° 16147=>16148, in tdejean's petals sandbox folder
  • Environment:
    all (?) . use soapui to generate soap message and send it to an internal petals service engine

Activity

Hide
Mathieu Carrolle added a comment - Thu, 4 Nov 2010 - 12:05:27 +0100 - edited

The fix have not to be done in the createDocument(...) method.
It have to be done in method getInMessageAsDocument() and getoutMessageAsDocument().

Show
Mathieu Carrolle added a comment - Thu, 4 Nov 2010 - 12:05:27 +0100 - edited The fix have not to be done in the createDocument(...) method. It have to be done in method getInMessageAsDocument() and getoutMessageAsDocument().
Mathieu Carrolle made changes - Thu, 4 Nov 2010 - 12:05:27 +0100
Field Original Value New Value
Status New [ 10000 ] Open [ 10002 ]
Priority Blocker [ 1 ]
Mathieu Carrolle made changes - Thu, 4 Nov 2010 - 14:18:40 +0100
Status Open [ 10002 ] In Progress [ 10003 ]
Mathieu Carrolle made changes - Thu, 4 Nov 2010 - 14:25:14 +0100
Status In Progress [ 10003 ] Open [ 10002 ]
Mathieu Carrolle made changes - Thu, 4 Nov 2010 - 14:25:18 +0100
Status Open [ 10002 ] In Progress [ 10003 ]
Mathieu Carrolle made changes - Thu, 4 Nov 2010 - 14:26:23 +0100
Status In Progress [ 10003 ] Resolved [ 10004 ]
Fix Version/s 5.1.2 [ 10180 ]
Resolution Fixed [ 1 ]
Transition Status Change Time Execution Times Last Executer Last Execution Date
New New Open Open
8d 22h 12m
1
Mathieu Carrolle
Thu, 4 Nov 2010 - 12:05:27 +0100
Open Open In Progress In Progress
2h 13m
1
Mathieu Carrolle
Thu, 4 Nov 2010 - 14:18:40 +0100
In Progress In Progress Open Open
6m 34s
1
Mathieu Carrolle
Thu, 4 Nov 2010 - 14:25:14 +0100
Open Open In Progress In Progress
4s
1
Mathieu Carrolle
Thu, 4 Nov 2010 - 14:25:18 +0100
In Progress In Progress Resolved Resolved
1m 5s
1
Mathieu Carrolle
Thu, 4 Nov 2010 - 14:26:23 +0100

People

Dates

  • Created:
    Tue, 26 Oct 2010 - 14:52:43 +0200
    Updated:
    Thu, 4 Nov 2010 - 14:26:23 +0100
    Resolved:
    Thu, 4 Nov 2010 - 14:26:23 +0100