You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by eg...@apache.org on 2009/01/15 12:09:34 UTC

svn commit: r734666 - /cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/saaj/SAAJInInterceptor.java

Author: eglynn
Date: Thu Jan 15 03:09:33 2009
New Revision: 734666

URL: http://svn.apache.org/viewvc?rev=734666&view=rev
Log:
Fix for NPE in SAAJInInterceptor when FaultDetail is not populated

Modified:
    cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/saaj/SAAJInInterceptor.java

Modified: cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/saaj/SAAJInInterceptor.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/saaj/SAAJInInterceptor.java?rev=734666&r1=734665&r2=734666&view=diff
==============================================================================
--- cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/saaj/SAAJInInterceptor.java (original)
+++ cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/saaj/SAAJInInterceptor.java Thu Jan 15 03:09:33 2009
@@ -127,7 +127,8 @@
                 if (fault.getRole() != null) {
                     soapFault.setFaultActor(fault.getRole());
                 }
-                if (fault.getDetail() != null) {
+                if (fault.getDetail() != null
+                    && fault.getDetail().getFirstChild() != null) {
                     soapFault.addDetail().appendChild(
                         soapMessage.getSOAPPart().importNode(
                             fault.getDetail().getFirstChild(), true));