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