/hg/icedtea-web: Preventing rare class cast exception in erroneo...

[jvanek at icedtea.classpath.org]

changeset 6d62f68fb037 in /hg/icedtea-web
details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=6d62f68fb037
author: Jiri Vanek <jvanek at redhat.com>
date: Mon Sep 22 17:10:07 2014 +0200

	Preventing rare class cast exception in erroneous detached applets


diffstat:

 ChangeLog                                                              |  10 ++++++++++
 netx/net/sourceforge/jnlp/runtime/AppletEnvironment.java               |   9 ++++++---
 netx/net/sourceforge/jnlp/splashscreen/SplashUtils.java                |   2 +-
 tests/netx/unit/net/sourceforge/jnlp/splashscreen/SplashUtilsTest.java |  10 ++++++++++
 4 files changed, 27 insertions(+), 4 deletions(-)

diffs (74 lines):

diff -r 9fbfc6ef9237 -r 6d62f68fb037 ChangeLog
--- a/ChangeLog	Fri Sep 19 15:51:19 2014 -0400
+++ b/ChangeLog	Mon Sep 22 17:10:07 2014 +0200
@@ -1,3 +1,13 @@
+2014-09-22  Jiri Vanek  <jvanek at redhat.com>
+
+	Preventing rare class cast exception in erroneous detached applets
+	* netx/net/sourceforge/jnlp/runtime/AppletEnvironment.java: getSplashControler
+	renamed to getSplashController. (getSplashController) added check for 
+	SplashController instance. Returning null if not so.
+	* netx/net/sourceforge/jnlp/splashscreen/SplashUtils.java: adapted to renaming
+	* tests/netx/unit/net/sourceforge/jnlp/splashscreen/SplashUtilsTest.java:
+	added (assertNulsAreOkInShow) test to check null values for showError methods
+
 2014-09-19  Jie Kang  <jkang at redhat.com>
 
     Modified Makefile.am to use whitelist when processing reproducers.
diff -r 9fbfc6ef9237 -r 6d62f68fb037 netx/net/sourceforge/jnlp/runtime/AppletEnvironment.java
--- a/netx/net/sourceforge/jnlp/runtime/AppletEnvironment.java	Fri Sep 19 15:51:19 2014 -0400
+++ b/netx/net/sourceforge/jnlp/runtime/AppletEnvironment.java	Mon Sep 22 17:10:07 2014 +0200
@@ -138,9 +138,12 @@
      * container must be SplashContoler
      * 
      */
-    public SplashController getSplashControler() {
-        
-        return (SplashController)cont;
+    public SplashController getSplashController() {
+        if (cont instanceof SplashController) {
+            return (SplashController) cont;
+        } else {
+            return null;
+        }
     }
 
     /**
diff -r 9fbfc6ef9237 -r 6d62f68fb037 netx/net/sourceforge/jnlp/splashscreen/SplashUtils.java
--- a/netx/net/sourceforge/jnlp/splashscreen/SplashUtils.java	Fri Sep 19 15:51:19 2014 -0400
+++ b/netx/net/sourceforge/jnlp/splashscreen/SplashUtils.java	Mon Sep 22 17:10:07 2014 +0200
@@ -94,7 +94,7 @@
         if (ae == null) {
             return;
         }
-        SplashController p = ae.getSplashControler();
+        SplashController p = ae.getSplashController();
         showError(ex, p);
     }
 
diff -r 9fbfc6ef9237 -r 6d62f68fb037 tests/netx/unit/net/sourceforge/jnlp/splashscreen/SplashUtilsTest.java
--- a/tests/netx/unit/net/sourceforge/jnlp/splashscreen/SplashUtilsTest.java	Fri Sep 19 15:51:19 2014 -0400
+++ b/tests/netx/unit/net/sourceforge/jnlp/splashscreen/SplashUtilsTest.java	Mon Sep 22 17:10:07 2014 +0200
@@ -40,6 +40,8 @@
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.Map;
+import net.sourceforge.jnlp.runtime.AppletEnvironment;
+import net.sourceforge.jnlp.runtime.AppletInstance;
 import net.sourceforge.jnlp.runtime.JNLPRuntime;
 import net.sourceforge.jnlp.splashscreen.impls.*;
 import org.junit.Assert;
@@ -251,5 +253,13 @@
       field.setAccessible(true);
       field.set(null, newValue);
    }
+     
+    @Test
+    public void assertNulsAreOkInShow() {
+        SplashUtils.showError(null, (AppletEnvironment)null);
+        SplashUtils.showError(null, (AppletInstance)null);
+        SplashUtils.showError(null, (SplashController)null);
+    }
+     
 
 }