Your fix looks good to me. Thanks. > On 17 Aug 2016, at 02:24, Valerie Peng <valerie.p...@oracle.com> wrote: > > > Anyone has time to review a straightforward fix? The current PKCS11 code > assume that if public exponent is available for RSA Private Key, then it's a > RSA CRT key. However, not all vendor implementation works this way. Changing > to a tighter check and did minor code-refactoring to avoid re-retrieving the > attribute values. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8078661 > Webrev: http://cr.openjdk.java.net/~valeriep/8078661/webrev.00/ > > Thanks, > Valerie
- [9] RFR 8078661: [SunPKCS11] Fails to cast into RSAPrivate... Valerie Peng
- Re: [9] RFR 8078661: [SunPKCS11] Fails to cast into R... Vincent Ryan
- Re: [9] RFR 8078661: [SunPKCS11] Fails to cast into R... Michael StJohns
- Re: [9] RFR 8078661: [SunPKCS11] Fails to cast in... Valerie Peng
- Re: [9] RFR 8078661: [SunPKCS11] Fails to cast in... Valerie Peng
- Re: [9] RFR 8078661: [SunPKCS11] Fails to cas... Michael StJohns
- Re: [9] RFR 8078661: [SunPKCS11] Fails to... Valerie Peng
- Re: [9] RFR 8078661: [SunPKCS11] Fai... Michael StJohns
- Re: [9] RFR 8078661: [SunPKCS11]... Valerie Peng
- Re: [9] RFR 8078661: [SunPKC... Michael StJohns
- RSA Key Interfaces Was: Re: [9] RFR 8078661: ... Michael StJohns