Your fix looks fine to me Sean.
Thanks.

> On 17 Oct 2016, at 15:51, Sean Mullan <sean.mul...@oracle.com> wrote:
> 
> Please review this fix to remove granting the unnecessary PropertyPermission 
> to the jdk.crypto.ec module:
> 
> http://cr.openjdk.java.net/~mullan/webrevs/8168078/webrev.00/
> 
> The existing TestEC test has been modified to also run under a 
> SecurityManager to verify that removing the permission does not break 
> anything.
> 
> --Sean

Reply via email to