Looks fine to me.

Xuelei

On 10/14/2016 3:44 AM, Sean Mullan wrote:
Please review this fix to only grant the necessary PropertyPermissions
to the jdk.crypto.ucrypto module.

In UcryptoProvider I also moved the System.getProperty("os.name") inside
doPrivileged (just in case permission to read this property has not been
granted to the caller) and did some minor refactoring.

http://cr.openjdk.java.net/~mullan/webrevs/8167512/webrev.00/

--Sean

Reply via email to