Request for approval to sync Jan 2014 CPU fixes

Andrew gnu.andrew at redhat.com
Thu Jan 16 14:44:59 PST 2014



----- Original Message -----
> * Andrew <gnu.andrew at redhat.com> [2014-01-16 16:27]:
> > * 8022945: The update to the corresponding test is missing:
> > 
> > +diff -r c0ff56eaaa96 -r 4b7df9a8efc3
> > test/java/lang/SecurityManager/CheckPackageAccess.java
> > +--- a/test/java/lang/SecurityManager/CheckPackageAccess.java   Fri Aug 09
> > 13:50:13 2013 +0100
> > ++++ b/test/java/lang/SecurityManager/CheckPackageAccess.java   Mon Aug 19
> > 22:43:32 2013 +0800
> > +@@ -23,7 +23,7 @@
> > +
> > + /*
> > +  *  @test
> > +- *  @bug 6741606 7146431 8000450
> > ++ *  @bug 6741606 7146431 8000450 8022945
> > +  *  @summary Make sure all restricted packages listed in the
> > package.access
> > +  *           property in the java.security file are blocked
> > +  *  @run main/othervm CheckPackageAccess
> > +@@ -54,6 +54,7 @@
> > +         "com.sun.imageio.",
> > +         "com.sun.istack.internal.",
> > +         "com.sun.jmx.",
> > ++        "com.sun.naming.internal.",
> > +         "com.sun.proxy.",
> > +         "com.sun.org.apache.bcel.internal.",
> > +         "com.sun.org.apache.regexp.internal.",
> > 
> > Unless you can alter the existing 8022945, you'll need to add that via an
> > OPENJDK-6 bug.
> 
> Updated webrev with this change included:
> http://cr.openjdk.java.net/~omajid/webrevs/2014-01-14-cpu.1/
> 
> If this looks okay, I can push this shortly.
> 
> Thanks,
> Omair
> 
> --
> PGP Key: 66484681 (http://pgp.mit.edu/)
> Fingerprint = F072 555B 0A17 3957 4E95  0056 F286 F14F 6648 4681
> 

That file looks correct now.  Assuming the rest is the same, please push.

Thanks for all your hard work,
-- 
Andrew :)

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

PGP Key: 248BDC07 (https://keys.indymedia.org/)
Fingerprint = EC5A 1F5E C0AD 1D15 8F1F  8F91 3B96 A578 248B DC07



More information about the jdk6-dev mailing list