> On Apr 29, 2016, at 11:25 PM, Alan Bateman <alan.bate...@oracle.com> wrote:
> 
> 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.


I originally asked Sherman to identify the permissions for jdk.charsets as a 
follow up issue.  I took another look and define the list:
   http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8155513/webrev.01/

Mandy

Reply via email to