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 security-dev
mailing list