System.java (setSecurityManager): Javadoc fix.

2006-08-09  Gary Benson  <gbenson@redhat.com>

	* java/lang/System.java (setSecurityManager): Javadoc fix.
	* java/security/VMAccessController.java (getContext): Comment fix.

From-SVN: r116033

diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index 27061ae8b6d4db3d43797f0a18bcd63c20d429c6..9be7e4ac6cc3b8051f8bb2ffa9532839ebe00750 100644 (file)
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -1,3 +1,8 @@
+2006-08-09  Gary Benson  <gbenson@redhat.com>
+
+       * java/lang/System.java (setSecurityManager): Javadoc fix.
+       * java/security/VMAccessController.java (getContext): Comment fix.
+
 2006-08-09  Gary Benson  <gbenson@redhat.com>
 
        * include/java-stack.h (accesscontrol_trace_fn): Declare.

diff --git a/libjava/java/lang/System.java b/libjava/java/lang/System.java
index 8ba0e8960cbc4862ac4ce6623c1eef66150fc4b9..6b125c90a2a006ebe4ac3b611489696df3e8e76e 100644 (file)
--- a/libjava/java/lang/System.java
+++ b/libjava/java/lang/System.java
@@ -167,11 +167,6 @@ public final class System
    * first. Since this permission is denied by the default security manager,
    * setting the security manager is often an irreversible action.
    *
-   * <STRONG>Spec Note:</STRONG> Don't ask me, I didn't write it.  It looks
-   * pretty vulnerable; whoever gets to the gate first gets to set the policy.
-   * There is probably some way to set the original security manager as a
-   * command line argument to the VM, but I don't know it.
-   *
    * @param sm the new SecurityManager
    * @throws SecurityException if permission is denied
    */

diff --git a/libjava/java/security/VMAccessController.java b/libjava/java/security/VMAccessController.java
index c39475334bb46de728b260a4fe6ff8e22eec703e..04c90d2ee784400d0fcbf440bc1e22bd8185d350 100644 (file)
--- a/libjava/java/security/VMAccessController.java
+++ b/libjava/java/security/VMAccessController.java
@@ -159,8 +159,6 @@ final class VMAccessController
   {
     // If the VM is initializing return the all-permissive context
     // so that any security checks succeed.
-    //
-    // XXX this might not be necessary, but it seems prudent.
     if (Thread.currentThread() == null)
       return DEFAULT_CONTEXT;