Review request: 8040059 Change default policy for extensions to no permission

Alan Bateman Alan.Bateman at oracle.com
Tue Apr 29 14:57:34 UTC 2014


On 25/04/2014 00:07, Mandy Chung wrote:
> Thanks Sean.
>
> I have updated the webrev:
> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8040059/webrev.01/
>
> Erik - I'm including build-dev to review the build change for 
> java.policy file.
Just catching up on this thread.

The update to ZipFileSystem and the policy file look good to me.

I only skimmed over the test changes and agree with the the comments 
that tests usually just want to configure the policy for the test and 
not have to repeat the grants in the system policy.

A minor comment on the policy files is that there is mix of indentation 
styles (4 and 8 in the same file in some cases).  There is also 
inconsistent indentation in Ext_AllPolicy.java (pre-dates your changes).

-Alan





More information about the core-libs-dev mailing list