FYI: Comment tweaks

Gary Benson gbenson@redhat.com
Wed Aug 9 08:48:00 GMT 2006


Hi again,

This commit removes a couple of superfluous notes in comments.

Cheers,
Gary
-------------- next part --------------
Index: ChangeLog
===================================================================
--- ChangeLog	(revision 116032)
+++ ChangeLog	(working copy)
@@ -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.
Index: java/lang/System.java
===================================================================
--- java/lang/System.java	(revision 116011)
+++ java/lang/System.java	(working copy)
@@ -167,11 +167,6 @@
    * 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
    */
Index: java/security/VMAccessController.java
===================================================================
--- java/security/VMAccessController.java	(revision 116011)
+++ java/security/VMAccessController.java	(working copy)
@@ -159,8 +159,6 @@
   {
     // 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;
 


More information about the Java-patches mailing list