<div dir="ltr"><div dir="ltr"><div>Hi Valerie,</div><div><br></div><div>I fixed all previously discussed issues in Webrev.13:</div><div><br></div><div> * <a href="http://cr.openjdk.java.net/~mbalao/webrevs/6913047/6913047.webrev.13/">http://cr.openjdk.java.net/~mbalao/webrevs/6913047/6913047.webrev.13/</a></div><div> * <a href="http://cr.openjdk.java.net/~mbalao/webrevs/6913047/6913047.webrev.13.zip">http://cr.openjdk.java.net/~mbalao/webrevs/6913047/6913047.webrev.13.zip</a></div><div><br></div><div>In addition to that, I fixed a couple of bugs introduced in p11_keymgmt.c.</div><div><br></div><div>In Java_sun_security_pkcs11_wrapper_PKCS11_getNativeKeyInfo function, the first call to C_GetAttributeValue (to get CKA_CLASS, CKA_KEY_TYPE, CKA_SENSITIVE and CKA_NETSCAPE_DB attributes) may fail because the key may not have a CKA_NETSCAPE_DB attribute. That is fine. It just means that we are not going to get that attribute -which does not exist- and it won't be needed for key unwrapping.</div><div><br></div><div>Later in Java_sun_security_pkcs11_wrapper_PKCS11_getNativeKeyInfo function, a similar issue happened. The call to get buffer lengths may return an error if one of the attributes does not exist. This is fine because length values are obtained anyways and based on that we were not going to query for non-existent attributes later.</div><div><br></div><div>These bugs were silently making all keys not to be extracted.</div><div><br></div><div>Thanks,</div><div>Martin.-</div><div><br></div></div></div>