Re: RFR: 8159752: Grant de-privileged module permissions by default with java.security.policy override option

2016-07-15 Thread Sean Mullan
On 07/15/2016 10:50 AM, Weijun Wang wrote: Changes looks fine to me. One nit: Shall we put jdk.crypto.ucrypto into src/java.base/solaris/lib/security/default.policy? Yes, might as well fix that as part of this. Thanks, Sean

Re: RFR: 8159752: Grant de-privileged module permissions by default with java.security.policy override option

2016-07-15 Thread Weijun Wang
Changes looks fine to me. One nit: Shall we put jdk.crypto.ucrypto into src/java.base/solaris/lib/security/default.policy? --Max On 7/15/2016 4:05, Sean Mullan wrote: Please review this change to the default Policy provider implementation to grant de-privileged module permissions by default

Re: RFR: 8159752: Grant de-privileged module permissions by default with java.security.policy override option

2016-07-15 Thread Tim Bell
The Makefile changes look fine. Tim On 07/15/16 04:51, Sean Mullan wrote: Adding build-dev for review since there is one change to a Makefile in the webrev below. Thanks, Sean On 07/14/2016 04:05 PM, Sean Mullan wrote: Please review this change to the default Policy provider implementation

Re: RFR: 8159752: Grant de-privileged module permissions by default with java.security.policy override option

2016-07-15 Thread Sean Mullan
Adding build-dev for review since there is one change to a Makefile in the webrev below. Thanks, Sean On 07/14/2016 04:05 PM, Sean Mullan wrote: Please review this change to the default Policy provider implementation to grant de-privileged module permissions by default even when the java.secur

Re: RFR: 8159752: Grant de-privileged module permissions by default with java.security.policy override option

2016-07-15 Thread Mandy Chung
> On Jul 15, 2016, at 4:05 AM, Sean Mullan wrote: > > Please review this change to the default Policy provider implementation to > grant de-privileged module permissions by default even when the > java.security.policy override option is specified or when the > Policy.getInstance API is used:

Re: RFR: 8159752: Grant de-privileged module permissions by default with java.security.policy override option

2016-07-14 Thread Sean Mullan
On 07/14/2016 04:38 PM, Chris Hegarty wrote: The default.policy file is now always loaded by the default Policy provider implementation (sun/security/provider/PolicyFile). It is loaded if the java.security.policy '=' or '==' option is specified, and also if the application uses the Policy.getIns

Re: RFR: 8159752: Grant de-privileged module permissions by default with java.security.policy override option

2016-07-14 Thread Chris Hegarty
> On 14 Jul 2016, at 21:05, Sean Mullan wrote: > > Please review this change to the default Policy provider implementation to > grant de-privileged module permissions by default even when the > java.security.policy override option is specified or when the > Policy.getInstance API is used: >

RFR: 8159752: Grant de-privileged module permissions by default with java.security.policy override option

2016-07-14 Thread Sean Mullan
Please review this change to the default Policy provider implementation to grant de-privileged module permissions by default even when the java.security.policy override option is specified or when the Policy.getInstance API is used: http://cr.openjdk.java.net/~mullan/webrevs/8159752/webrev.00/