You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicemix.apache.org by ff...@apache.org on 2009/06/02 11:37:02 UTC

svn commit: r780983 - /servicemix/components/bindings/servicemix-cxf-bc/trunk/src/main/java/org/apache/servicemix/cxfbc/interceptors/JbiInWsdl1Interceptor.java

Author: ffang
Date: Tue Jun  2 09:37:02 2009
New Revision: 780983

URL: http://svn.apache.org/viewvc?rev=780983&view=rev
Log:
[SMXCOMP-555]cxf bc provider should handle soap fault correctly with empty detail

Modified:
    servicemix/components/bindings/servicemix-cxf-bc/trunk/src/main/java/org/apache/servicemix/cxfbc/interceptors/JbiInWsdl1Interceptor.java

Modified: servicemix/components/bindings/servicemix-cxf-bc/trunk/src/main/java/org/apache/servicemix/cxfbc/interceptors/JbiInWsdl1Interceptor.java
URL: http://svn.apache.org/viewvc/servicemix/components/bindings/servicemix-cxf-bc/trunk/src/main/java/org/apache/servicemix/cxfbc/interceptors/JbiInWsdl1Interceptor.java?rev=780983&r1=780982&r2=780983&view=diff
==============================================================================
--- servicemix/components/bindings/servicemix-cxf-bc/trunk/src/main/java/org/apache/servicemix/cxfbc/interceptors/JbiInWsdl1Interceptor.java (original)
+++ servicemix/components/bindings/servicemix-cxf-bc/trunk/src/main/java/org/apache/servicemix/cxfbc/interceptors/JbiInWsdl1Interceptor.java Tue Jun  2 09:37:02 2009
@@ -232,7 +232,8 @@
             nodeList = soapFault.getElementsByTagName("faultstring");
             message.put("faultstring", nodeList.item(0).getFirstChild().getTextContent());
             nodeList = soapFault.getElementsByTagName("detail");
-            if (nodeList != null && nodeList.getLength() > 0) {
+            if (nodeList != null && nodeList.getLength() > 0
+                && nodeList.item(0).getFirstChild() != null) {
                 jbiFaultDetail = doc.importNode(nodeList.item(0).getFirstChild(), true);
             } else {
                 message.put("hasdetail", false);
@@ -249,7 +250,8 @@
             nodeList = soapFault.getElementsByTagName("soap:Reason");
             message.put("faultstring", nodeList.item(0).getFirstChild().getTextContent());
             nodeList = soapFault.getElementsByTagName("soap:Detail");
-            if (nodeList != null && nodeList.getLength() > 0) {
+            if (nodeList != null && nodeList.getLength() > 0
+                && nodeList.item(0).getFirstChild() != null) {
                 jbiFaultDetail = doc.importNode(nodeList.item(0).getFirstChild(), true);
             } else {
                 message.put("hasdetail", false);