Review request: 8154190 & 8155513: Deprivilege java.compiler and jdk.charsets

[Alan Bateman]

On 30/04/2016 06:02, Mandy Chung wrote:
> JDK-8154190: Deprivilege java.compiler module
> Webrev:
>     http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8154190/webrev.00/
>
> JDK-8155513: Deprivilege jdk.charsets module
> Webrev:
>    http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8155513/webrev.00/
>
> Very simple change.
>
> These patches move java.compiler and jdk.charsets module to be defined by the platform class loader and grant with AllPermissions initially.  We could grant finer-grained permissions in the future.
>
This looks okay to me. For jdk.charsets then I expect it should be easy 
to identify the permission it needs as there are only a few cases where 
it needs os.name or to read a resource file in the image.

-Alan.