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