Mercurial > hg > release > icedtea6-1.9

changeset 2334:e4336b504f8a

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Backport fix for PR542 2010-11-01 Deepak Bhole <dbhole@redhat.com> PR542: Plugin fails with NPE on http://www.openprocessing.org/visuals/iframe.php?visualID=2615 * netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java (initializeResources): If cacheFile is null (JAR couldn't be downloaded), try to continue, rather than allowing the exception to cause an abort. * NEWS: Updated.
author Omair Majid <omajid@redhat.com>
date Fri, 25 Mar 2011 15:04:00 -0400
parents 77b802f946bb
children 495cac553ab0
files ChangeLog NEWS netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java
diffstat 3 files changed, 20 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Mon Mar 07 17:22:45 2011 +0100
+++ b/ChangeLog	Fri Mar 25 15:04:00 2011 -0400
@@ -1,3 +1,12 @@
+2010-11-01  Deepak Bhole <dbhole@redhat.com>
+
+	PR542: Plugin fails with NPE on
+	http://www.openprocessing.org/visuals/iframe.php?visualID=2615
+	* netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java
+	(initializeResources): If cacheFile is null (JAR couldn't be downloaded),
+	try to continue, rather than allowing the exception to cause an abort.
+	* NEWS: Updated.
+
 2011-03-07  Pavel Tisnovsky  <ptisnovs@redhat.co>
 
 	* Makefile.am: Add new patch.
--- a/NEWS	Mon Mar 07 17:22:45 2011 +0100
+++ b/NEWS	Fri Mar 25 15:04:00 2011 -0400
@@ -14,6 +14,8 @@
   - S6675802: Regression: heavyweight popups cause SecurityExceptions in applets
   - S6691503: Malicious applet can show always-on-top popup menu which has whole screen size 
   - S6980392, PR642: simple correction in testcase, added missing bracket
+* Plugin
+  - PR542: Plugin fails with NPE on http://www.openprocessing.org/visuals/iframe.php?visualID=2615
 
 New in release 1.9.7 (2011-02-15):
 
--- a/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java	Mon Mar 07 17:22:45 2011 +0100
+++ b/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java	Fri Mar 25 15:04:00 2011 -0400
@@ -424,7 +424,15 @@
 
                 for (JARDesc jarDesc: file.getResources().getJARs()) {
                         try {
-                                URL location = tracker.getCacheFile(jarDesc.getLocation()).toURI().toURL();
+                                File cachedFile = tracker.getCacheFile(jarDesc.getLocation());
+
+                                if (cachedFile == null) {
+                                    System.err.println("JAR " + jarDesc.getLocation() + " not found. Continuing.");
+                                    continue; // JAR not found. Keep going.
+                                }
+
+                                URL location = cachedFile.toURL();
+
                                 SecurityDesc jarSecurity = file.getSecurity();
 
                                 if (file instanceof PluginBridge) {