You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ode.apache.org by mr...@apache.org on 2009/07/02 19:39:01 UTC

svn commit: r790694 - /ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/SoapExternalService.java

Author: mriou
Date: Thu Jul  2 17:39:01 2009
New Revision: 790694

URL: http://svn.apache.org/viewvc?rev=790694&view=rev
Log:
When creating a service client and setting the service, Axis2 leaks uncleaned anonymous clients. Results in a memory leak.

Modified:
    ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/SoapExternalService.java

Modified: ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/SoapExternalService.java
URL: http://svn.apache.org/viewvc/ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/SoapExternalService.java?rev=790694&r1=790693&r2=790694&view=diff
==============================================================================
--- ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/SoapExternalService.java (original)
+++ ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/SoapExternalService.java Thu Jul  2 17:39:01 2009
@@ -71,6 +71,7 @@
 import java.util.Map;
 import java.util.concurrent.Callable;
 import java.util.concurrent.ExecutorService;
+import java.lang.reflect.Field;
 
 /**
  * Acts as a service not provided by ODE. Used mainly for invocation as a way to maintain the WSDL description of used
@@ -277,8 +278,7 @@
                 }
                 if (__log.isDebugEnabled())
                     __log.debug("Create a new ServiceClient for " + anonymousService.getName());
-                client = new ServiceClient(_configContext, null);
-                client.setAxisService(anonymousService);
+                client = new ServiceClient(_configContext, anonymousService);
             }
             _cachedClients.set(client);
         }