[jboss-cvs] JBossAS SVN: r88191 - projects/jboss-cl/trunk/classloader/src/main/java/org/jboss/classloader/spi/base.

[jboss-cvs-commits at lists.jboss.org]

Author: alesj
Date: 2009-05-05 02:04:48 -0400 (Tue, 05 May 2009)
New Revision: 88191

Modified:
   projects/jboss-cl/trunk/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoaderDomain.java
Log:
Move failIfBlacklisted  flag to the check method.

Modified: projects/jboss-cl/trunk/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoaderDomain.java
===================================================================
--- projects/jboss-cl/trunk/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoaderDomain.java	2009-05-05 06:02:04 UTC (rev 88190)
+++ projects/jboss-cl/trunk/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoaderDomain.java	2009-05-05 06:04:48 UTC (rev 88191)
@@ -240,7 +240,7 @@
       
       String path = ClassLoaderUtils.classNameToPath(name);
 
-      checkClassBlackList(classLoader, name, path, allExports);
+      checkClassBlackList(classLoader, name, path, allExports, true);
       
       boolean findInParent = (isUseLoadClassForParent() == false);
       
@@ -1465,13 +1465,16 @@
     * @param name the name
     * @param path the path of the class resource
     * @param allExports whether to look at all exports
+    * @param failIfBlackListed <code>true</code> if a blacklisted class should
+    *                          result in ClassNotFoundException; <code>false</code>
+    *                          if a <code>null</code> return value is acceptable
     * @throws ClassNotFoundException when the class is blacklisted
     */
-   void checkClassBlackList(BaseClassLoader classLoader, String name, String path, boolean allExports) throws ClassNotFoundException
+   void checkClassBlackList(BaseClassLoader classLoader, String name, String path, boolean allExports, boolean failIfBlackListed) throws ClassNotFoundException
    {
       if (allExports)
       {
-         if (globalClassBlackList.containsKey(path))
+         if (failIfBlackListed && globalClassBlackList.containsKey(path))
          {
             if (log.isTraceEnabled())
                log.trace("Found " + name + " in global blacklist");
@@ -1503,10 +1506,8 @@
       if (result != null)
          return result;
       
-      if (failIfBlackListed)
-      {
-         checkClassBlackList(classLoader, name, path, allExports);
-      }
+      checkClassBlackList(classLoader, name, path, allExports, failIfBlackListed);
+      
       return null;
    }