Re: RFR: 8154009: Some methods of java.security.Security require more permissions, than necessary

2016-06-02 Thread Sean Mullan
Looks good. --Sean On 06/01/2016 10:56 AM, Artem Kosarev wrote: Hi Sean. Thanks for suggestion. New WebRev: http://cr.openjdk.java.net/~akosarev/8154009/webrev.01/ There are only 2 changes from original one: 1) *test/java/security/Security/EmptyPolicy.policy* was updated in the way you propo

Re: RFR: 8154009: Some methods of java.security.Security require more permissions, than necessary

2016-06-01 Thread Artem Kosarev
Hi Sean. Thanks for suggestion. New WebRev: http://cr.openjdk.java.net/~akosarev/8154009/webrev.01/ There are only 2 changes from original one: 1) *test/java/security/Security/EmptyPolicy.policy* was updated in the way you proposed. 2) I removed 2 tests from *test/ProblemList.txt*, which were

Re: RFR: 8154009: Some methods of java.security.Security require more permissions, than necessary

2016-06-01 Thread Sean Mullan
I think it would be helpful to add a comment to EmptyPolicy.policy so it contains something, ex: // empty policy file for testing Otherwise, looks fine. --Sean On 05/30/2016 09:03 AM, Artem Kosarev wrote: Hello. Could you please review the proposed fix issue which is NOT applicable for JDK

RFR: 8154009: Some methods of java.security.Security require more permissions, than necessary

2016-05-30 Thread Artem Kosarev
Hello. Could you please review the proposed fix issue which is NOT applicable for JDK 9: BUGURL: https://bugs.openjdk.java.net/browse/JDK-8154009 WEBREV: http://cr.openjdk.java.net/~akosarev/8154009/webrev.00/ PROBLEM: **/AddProvider/, /RemoveProvider///& /GetProviders///methods of*//**/java