Your fix still looks fine to me.
> On 27 Oct 2016, at 18:00, Sean Mullan <sean.mul...@oracle.com> wrote: > > I accidentally used the wrong bugid. I have updated the Subject line and > links to reflect that this fix is for the java.smartcardio module, not the > jdk.crypto.ucrypto module: > > https://bugs.openjdk.java.net/browse/JDK-8168851 > http://cr.openjdk.java.net/~mullan/webrevs/8168851/webrev.00/ > > --Sean > > On 10/27/2016 12:28 PM, Vincent Ryan wrote: >> 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 >>
