You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by ri...@apache.org on 2009/06/12 16:08:24 UTC

svn commit: r784127 - /felix/trunk/framework/src/main/java/org/apache/felix/framework/ModuleImpl.java

Author: rickhall
Date: Fri Jun 12 14:08:24 2009
New Revision: 784127

URL: http://svn.apache.org/viewvc?rev=784127&view=rev
Log:
Use Class.forName() for class load delegation since it handles
array types. (FELIX-1233)

Modified:
    felix/trunk/framework/src/main/java/org/apache/felix/framework/ModuleImpl.java

Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/ModuleImpl.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/ModuleImpl.java?rev=784127&r1=784126&r2=784127&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/ModuleImpl.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/ModuleImpl.java Fri Jun 12 14:08:24 2009
@@ -492,7 +492,10 @@
 
     public Class getClassByDelegation(String name) throws ClassNotFoundException
     {
-        return getClassLoader().loadClass(name);
+        // We do not call getClassLoader().loadClass() because this does not
+        // correctly handle array types, which is necessary in cases like
+        // deserialization using a wrapper class loader.
+        return Class.forName(name, false, getClassLoader());
     }
 
     public URL getResourceByDelegation(String name)