You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cxf.apache.org by "Willem Jiang (JIRA)" <ji...@apache.org> on 2010/06/07 10:39:59 UTC

[jira] Resolved: (CXF-2837) Add null pointer check in SoapOutInterceptor

     [ https://issues.apache.org/jira/browse/CXF-2837?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Willem Jiang resolved CXF-2837.
-------------------------------

    Fix Version/s: 2.2.10
                       (was: 2.2.9)
       Resolution: Fixed

Applied patch into trunk and cxf 2.2.x branch with thanks to William.

> Add null pointer check in SoapOutInterceptor
> --------------------------------------------
>
>                 Key: CXF-2837
>                 URL: https://issues.apache.org/jira/browse/CXF-2837
>             Project: CXF
>          Issue Type: Improvement
>          Components: Soap Binding
>            Reporter: William Tam
>            Assignee: Willem Jiang
>             Fix For: 2.3, 2.2.10
>
>
> It allows soap headers to be skipped (e.g. by Camel).   
> Index: rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapOutInterceptor.java
> ===================================================================
> --- rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapOutInterceptor.java	(revision 22)
> +++ rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapOutInterceptor.java	(working copy)
> @@ -219,6 +219,11 @@
>                      continue;
>                  }
>                  Object arg = objs.get(part);
> +
> +                if (arg == null) {
> +                    continue;
> +                }
> +
>                  objs.remove(part);
>                  if (!(startedHeader || preexistingHeaders)) {
>                      try {

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.