Hi,

I am attaching a patch to fix a minor issue in Kit.java. If rhino
classes are loaded by the bootloader, ShellLine ends up calling
Kit::classOrNull() with a null loader. Attached patch makes it so that
the systemloader is tried in that case. Without the patch, an unhandled
NPE is thrown.

Cheers,
Deepak
diff -up ./src/org/mozilla/javascript/Kit.java.sav 
./src/org/mozilla/javascript/Kit.java
--- ./src/org/mozilla/javascript/Kit.java.sav   2008-07-28 10:45:16.000000000 
-0400
+++ ./src/org/mozilla/javascript/Kit.java       2009-05-07 09:59:01.000000000 
-0400
@@ -89,7 +89,10 @@ public class Kit
     public static Class<?> classOrNull(ClassLoader loader, String className)
     {
         try {
-            return loader.loadClass(className);
+            if (loader == null)
+                return ClassLoader.getSystemClassLoader().loadClass(className);
+            else
+                return loader.loadClass(className);
         } catch (ClassNotFoundException ex) {
         } catch (SecurityException ex) {
         } catch (LinkageError ex) {
_______________________________________________
dev-tech-js-engine-rhino mailing list
[email protected]
https://lists.mozilla.org/listinfo/dev-tech-js-engine-rhino

Reply via email to