You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by di...@apache.org on 2008/06/13 16:08:56 UTC

svn commit: r667548 - /webservices/axis2/trunk/java/modules/jaxws/src/org/apache/axis2/jaxws/message/databinding/JAXBUtils.java

Author: dims
Date: Fri Jun 13 07:08:56 2008
New Revision: 667548

URL: http://svn.apache.org/viewvc?rev=667548&view=rev
Log:
adding java2 security checks when creating marshaller/unmarshaller/introspector

Modified:
    webservices/axis2/trunk/java/modules/jaxws/src/org/apache/axis2/jaxws/message/databinding/JAXBUtils.java

Modified: webservices/axis2/trunk/java/modules/jaxws/src/org/apache/axis2/jaxws/message/databinding/JAXBUtils.java
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/jaxws/src/org/apache/axis2/jaxws/message/databinding/JAXBUtils.java?rev=667548&r1=667547&r2=667548&view=diff
==============================================================================
--- webservices/axis2/trunk/java/modules/jaxws/src/org/apache/axis2/jaxws/message/databinding/JAXBUtils.java (original)
+++ webservices/axis2/trunk/java/modules/jaxws/src/org/apache/axis2/jaxws/message/databinding/JAXBUtils.java Fri Jun 13 07:08:56 2008
@@ -585,20 +585,14 @@
             if (log.isDebugEnabled()) {
                 log.debug("JAXBIntrospector created [no pooling]");
             }
-            i = (JAXBIntrospector) AccessController.doPrivileged(
-                    new PrivilegedAction() {
-                        public Object run() {
-                            return context.createJAXBIntrospector();
-                        }
-                    }
-            );
+            i = internalCreateIntrospector(context);
         } else {
             i = ipool.get(context);
             if (i == null) {
                 if (log.isDebugEnabled()) {
                     log.debug("JAXBIntrospector created [not in pool]");
                 }
-                i = context.createJAXBIntrospector();
+                i = internalCreateIntrospector(context);
             } else {
                 if (log.isDebugEnabled()) {
                     log.debug("JAXBIntrospector obtained [from  pool]");
@@ -608,6 +602,18 @@
         return i;
     }
 
+    private static JAXBIntrospector internalCreateIntrospector(final JAXBContext context) {
+        JAXBIntrospector i;
+        i = (JAXBIntrospector) AccessController.doPrivileged(
+                new PrivilegedAction() {
+                    public Object run() {
+                        return context.createJAXBIntrospector();
+                    }
+                }
+        );
+        return i;
+    }
+
     /**
      * Release JAXBIntrospector Do not call this method if an exception occurred while using the
      * JAXBIntrospector. We object my be in an invalid state.