You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by dk...@apache.org on 2014/10/28 19:30:18 UTC

[4/5] git commit: [CXF-6030] Add an NPE guard and remove "null" headers

[CXF-6030] Add an NPE guard and remove "null" headers


Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/63d56507
Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/63d56507
Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/63d56507

Branch: refs/heads/2.7.x-fixes
Commit: 63d56507b0541cfa049a8dbaf91f2f66157c88ac
Parents: 9e171ed
Author: Daniel Kulp <dk...@apache.org>
Authored: Tue Oct 28 14:11:39 2014 -0400
Committer: Daniel Kulp <dk...@apache.org>
Committed: Tue Oct 28 14:30:05 2014 -0400

----------------------------------------------------------------------
 .../binding/soap/interceptor/SoapHeaderOutFilterInterceptor.java  | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf/blob/63d56507/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapHeaderOutFilterInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapHeaderOutFilterInterceptor.java b/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapHeaderOutFilterInterceptor.java
index 0a5a4e7..48e47a6 100644
--- a/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapHeaderOutFilterInterceptor.java
+++ b/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapHeaderOutFilterInterceptor.java
@@ -34,13 +34,12 @@ public class SoapHeaderOutFilterInterceptor extends AbstractSoapInterceptor {
     }
 
     public void handleMessage(SoapMessage message) throws Fault {
-        // TODO Auto-generated method stub
         Iterator<Header> iter =  message.getHeaders().iterator();
         
         while (iter.hasNext()) {
             Header hdr  = iter.next();
             //Only remove inbound marked headers..
-            if (hdr.getDirection() == Header.Direction.DIRECTION_IN) {
+            if (hdr == null || hdr.getDirection() == Header.Direction.DIRECTION_IN) {
                 iter.remove(); 
             }
         }