Review request for the incorrect check for "getClassLoader" permission

Paul Sandoz paul.sandoz at oracle.com
Mon Jun 25 22:07:17 PDT 2012


On Jun 25, 2012, at 10:12 PM, Mandy Chung wrote:

> Alan, David, Paul,
> 
> Given that j.u.c is pulled down from Doug's CVS into OpenJDK, I am having a second thought to leave the j.u.c.atomic change out from this patch.
> 
> I have added a static needsPackalgeAccessCheck method in sun.reflect.misc.ReflectUtil class that could be used by j.u.c.atomic updater once it's integrated to jdk8.  Since this j.u.c. change isn't critical for jigsaw at this time, we'll check with Doug (probably with David or Chris' help) and get appropriate change made in JSR166e source that will subsequently get pulled down in JDK 8 and jigsaw.
> 

Good idea.


> Updated webrev:
>  http://cr.openjdk.java.net/~mchung/jigsaw/webrevs/getclassloader-permission-fix.01/
> 

Looks good to me.

Paul.


More information about the jigsaw-dev mailing list