geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rick McGuire (JIRA)" <>
Subject [jira] Commented: (GERONIMO-2510) CORBA/RMI delegate code requires delegate implementation to be on the system classpath
Date Fri, 20 Oct 2006 09:16:35 GMT
    [ ] 
Rick McGuire commented on GERONIMO-2510:

I believe his analysis is (mostly) correct.  The specification for javax.rmi.CORBA.Util.loadClass()
has a fairly involved search order for loading classes, and it appears that this search order
is used even when the class getting loaded is the Util delegate itself.  In any event, Yoko
should not be using Class.forName() to be loading this.  

> CORBA/RMI delegate code requires delegate implementation to be on the system classpath
> --------------------------------------------------------------------------------------
>                 Key: GERONIMO-2510
>                 URL:
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: CORBA
>            Reporter: Dain Sundstrom
> When the java.rmi.Util class is loaded, it uses Class.forName to load the UtilDelegate
specified in a system property.  This is what yoko uses:
>     private static UtilDelegate delegate = null;
>     private static final String defaultDelegate = "org.apache.yoko.rmi.impl.UtilImpl";
>     static {
>         // Initialize delegate
>         String delegateName = System.getProperty("javax.rmi.CORBA.UtilClass", defaultDelegate);
>         try {
>             delegate = (UtilDelegate)Class.forName(delegateName).newInstance();
>         } catch (Exception e) {
>             throw new RuntimeException("Can not create Util delegate: "+delegateName,
>         }
>     }
> IIRC Class.forName(String) uses the class loader of the class containing the code, and
since java.rmi.Util will always be on the system class path our delegate must be on the system
class path :(
> I see a few choices:
> * change yoko to use the thread context class loader... then add a gbean to set the property
and load the Util class. repeat of every delegate class.
> * write a delegate delegate that allows us to swap out the actual delegate implementation
> * split out out delegate class into a jar we can put on the system class path

This message is automatically generated by JIRA.
If you think it was sent incorrectly contact one of the administrators:
For more information on JIRA, see:


View raw message