[RFC] Added javawebstart.version property to netx

[Deepak Bhole]

* Man Wong <mwong at redhat.com> [2010-07-20 13:24]:
> The attached patch adds javawebstart.version as a system property whenever a 
> jnlp file is being launched. It also gives the user the permission to read that 
> property when working under sandbox permission. This fixes icedtea bug 486.
> Even though bug 486 says there is a problem with the encoding support. However,
> after add the encoding support, the file could not be launched because the
> above permission was denied. 
> 
> Should I file a separate bug for the permission issue, or can I just mention
> that as part of bug 486?
> 
> Any comments?
> 
> Thanks,
> Man Lung Wong

Looks good. Please commit to 1.7 and 1.8 along with HEAD.

Cheers,
Deepak

> diff -r 16b019555725 netx/net/sourceforge/jnlp/SecurityDesc.java
> --- a/netx/net/sourceforge/jnlp/SecurityDesc.java	Tue Jul 20 09:35:12 2010 -0400
> +++ b/netx/net/sourceforge/jnlp/SecurityDesc.java	Tue Jul 20 11:11:05 2010 -0400
> @@ -94,6 +94,7 @@
>          new PropertyPermission("java.vm.version", "read"),
>          new PropertyPermission("java.vm.vendor", "read"),
>          new PropertyPermission("java.vm.name", "read"),
> +        new PropertyPermission("javawebstart.version", "read"),
>          new RuntimePermission("exitVM"),
>          new RuntimePermission("stopThread"),
>          new AWTPermission("showWindowWithoutWarningBanner"),
> diff -r 16b019555725 netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java
> --- a/netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java	Tue Jul 20 09:35:12 2010 -0400
> +++ b/netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java	Tue Jul 20 11:11:05 2010 -0400
> @@ -167,6 +167,11 @@
>          checkInitialized();
>       
>          isWebstartApplication = isApplication;
> +        
> +        //Setting the system property for javawebstart's version.
> +        //The version stored will be the same as java's version.
> +        System.setProperty("javawebstart.version", "javaws-" +
> +            System.getProperty("java.version"));
>  
>          if (headless == false)
>              checkHeadless();