[icedtea-web] RFC: Fix for RH718693: MindTerm SSH Applet doesn't work

[Dr Andrew John Hughes]

On 16:58 Thu 04 Aug     , Deepak Bhole wrote:
> Hi,
> 
> Attached path fixes rhbz 718693:
> https://bugzilla.redhat.com/show_bug.cgi?id=718693
> 
> The patch adds a class that certain apps may call because Netscape used
> to need it. IcedTea-Web does not need this, and therefore the methods
> are empty.
> 
> Okay for HEAD and 1.1?
> 
> ChangeLog:
> 2011-08-04  Deepak Bhole <dbhole at redhat.com>
> 
>     RH718693: MindTerm SSH Applet doesn't work
>     * plugin/icedteanp/java/netscape/security/PrivilegeManager.java: New file.
>     Stub class, not needed with IcedTea-Web.
> 
> Cheers,
> Deepak

Needs a license header.  It would also probably benefit from some class-level
Javadoc saying what you said above.

> diff -r defa7d0051bf NEWS
> --- a/NEWS	Wed Aug 03 14:11:11 2011 -0400
> +++ b/NEWS	Thu Aug 04 16:55:23 2011 -0400
> @@ -14,6 +14,7 @@
>  	- RH718170, CVE-2011-2514: Java Web Start security warning dialog manipulation
>  * Plugin
>    - PR749: sun.applet.PluginStreamHandler#handleMessage(String) really slow
> +  - RH718693: MindTerm SSH Applet doesn't work 
>  Common
>    - PR768: Signed applets/Web Start apps don't work with OpenJDK7 and up
>  
> diff -r defa7d0051bf plugin/icedteanp/java/netscape/security/PrivilegeManager.java
> --- /dev/null	Thu Jan 01 00:00:00 1970 +0000
> +++ b/plugin/icedteanp/java/netscape/security/PrivilegeManager.java	Thu Aug 04 16:55:23 2011 -0400
> @@ -0,0 +1,24 @@
> +package netscape.security;
> +
> +import sun.applet.PluginDebug;
> +
> +public class PrivilegeManager {
> +
> +    /**
> +     * Stub for enablePrivilege. Not used by IcedTea-Web, kept for compatibility
> +     * 
> +     * @param privilege
> +     */
> +    public static void  enablePrivilege(String privilege) {
> +        PluginDebug.debug("netscape.security.enablePrivilege stub called");
> +    }
> +    
> +    /**
> +     * Stub for disablePrivilege. Not used by IcedTea-Web, kept for compatibility
> +     * 
> +     * @param privilege
> +     */
> +    public static void  disablePrivilege(String privilege) {
> +        PluginDebug.debug("netscape.security.disablePrivilege stub called");
> +    }
> +}


-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Support Free Java!
Contribute to GNU Classpath and IcedTea
http://www.gnu.org/software/classpath
http://icedtea.classpath.org
PGP Key: F5862A37 (https://keys.indymedia.org/)
Fingerprint = EA30 D855 D50F 90CD F54D  0698 0713 C3ED F586 2A37