Your fix looks fine to me Sean.
> On 27 Oct 2016, at 17:09, Sean Mullan <sean.mul...@oracle.com> wrote: > > Please review this change to tighten or remove unnecessary permissions > granted to the jdk.crypto.ucrypto module: > > https://bugs.openjdk.java.net/browse/JDK-8167512 > http://cr.openjdk.java.net/~mullan/webrevs/8167512/webrev.00/ > > Thanks, > Sean