Looks fine to me. Thanks, Xuelei
On 12/6/2012 8:16 AM, Jason Uh wrote: > Could I please get a review of: > http://cr.openjdk.java.net/~juh/7193792/webrev.00/ > > Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7193792 > > This change fixes sun/security/pkcs11/ec/TestECDSA.java, which fails on > Solaris due to what seems to be an unintended back-to-back call to > Signature.initSign(privateKey). On the second initSign, the test fails > with CKR_OPERATION_ACTIVE, which means: > >> There is already an active operation (or combination of active >> operations) which prevents Cryptoki from activating the specified >> operation. ... > (http://www.cryptsoft.com/pkcs11doc/v220/pkcs11__all_8h.html#aCKR_OPERATION_ACTIVE) > > > Thanks, > Jason
