/hg/icedtea-web: * netx/net/sourceforge/jnlp/config/SecurityValu...

jvanek at icedtea.classpath.org jvanek at icedtea.classpath.org
Thu Feb 28 07:01:25 PST 2013


changeset 555397961654 in /hg/icedtea-web
details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=555397961654
author: Jiri Vanek <jvanek at redhat.com>
date: Thu Feb 28 16:03:39 2013 +0100

	* netx/net/sourceforge/jnlp/config/SecurityValueValidator.java: modifed
	null check - no considered as correct value as being valid value in runtime.


diffstat:

 ChangeLog                                                    |  5 +++++
 netx/net/sourceforge/jnlp/config/SecurityValueValidator.java |  5 ++++-
 2 files changed, 9 insertions(+), 1 deletions(-)

diffs (27 lines):

diff -r 56c94adc52ea -r 555397961654 ChangeLog
--- a/ChangeLog	Wed Feb 27 17:00:48 2013 +0100
+++ b/ChangeLog	Thu Feb 28 16:03:39 2013 +0100
@@ -1,3 +1,8 @@
+2013-02-28  Jiri Vanek  <jvanek at redhat.com>
+
+	* netx/net/sourceforge/jnlp/config/SecurityValueValidator.java: modifed
+	null check - no considered as correct value as being valid value in runtime.
+
 2013-02-27  Jiri Vanek  <jvanek at redhat.com>
 
 	Added backend and settings for extended applet security
diff -r 56c94adc52ea -r 555397961654 netx/net/sourceforge/jnlp/config/SecurityValueValidator.java
--- a/netx/net/sourceforge/jnlp/config/SecurityValueValidator.java	Wed Feb 27 17:00:48 2013 +0100
+++ b/netx/net/sourceforge/jnlp/config/SecurityValueValidator.java	Thu Feb 28 16:03:39 2013 +0100
@@ -45,7 +45,10 @@
     @Override
     public void validate(Object value) throws IllegalArgumentException {
         if (value == null) {
-            throw new IllegalArgumentException("Value can't be null");
+            // null is correct, it means it is not user set
+            // and so default shoudl be used whatever it is
+            // returning to prevent NPE in fromString
+            return;
         }
         if (value instanceof AppletSecurityLevel) {
             //??



More information about the distro-pkg-dev mailing list