[cp-patches] FYI: Load LoginModules through context class loader

[Mark Wielaard]

Hi,

This was one of the patches needed to get JBoss starting up. We were not
finding any LoginModules since we were trying to find them through the
bootstrap class loader.

2005-10-19  Mark Wielaard  <address@hidden>

        * javax/security/auth/login/LoginContext.java (lookupModule):
        Load LoginModule through Thread context class loader.

We should do an audit of all places where Class.forName() is called to
see whether they really need to have a ClassLoader argument.

Committed,

Mark

diff -u -r1.3 LoginContext.java
--- javax/security/auth/login/LoginContext.java 2 Jul 2005 20:32:46 -0000      
1.3
+++ javax/security/auth/login/LoginContext.java 19 Oct 2005 16:06:30 -0000
@@ -220,7 +220,9 @@
     Exception cause = null;
     try
       {
-        module = (LoginModule) Class.forName 
(entry.getLoginModuleName()).newInstance();
+       ClassLoader cl = Thread.currentThread().getContextClassLoader();
+       Class c = Class.forName(entry.getLoginModuleName(), true, cl);
+       module = (LoginModule) c.newInstance();
       }
     catch (ClassNotFoundException cnfe)
       {

signature.asc
Description: This is a digitally signed message part