RFR: 8330133: libj2pkcs11.so crashes on some pkcs#11 v3.0 libraries [v2]
Valerie Peng
valeriep at openjdk.org
Tue Apr 16 22:50:01 UTC 2024
On Tue, 16 Apr 2024 17:21:11 GMT, Valerie Peng <valeriep at openjdk.org> wrote:
>> It is reported that some PKCS#11 library/vendor reports major version 3, but doesn't implement the C_GetInterface function and the resulting 'interface' variable value may be NULL and cause unexpected crash later.
>>
>> This PR would check the 'interface' variable value to be non-NULL.
>> Reproducing this would require certain 3rd party PKCS#11 library, and thus the noreg-hard label.
>>
>> Thanks~
>> FYI, I will be on vacation starting 4/17 and will address the review comments upon return.
>> Valerie
>
> Valerie Peng has updated the pull request incrementally with one additional commit since the last revision:
>
> Removed pre-existing duplicate assignment.
Thanks for the reviews!
-------------
PR Comment: https://git.openjdk.org/jdk/pull/18789#issuecomment-2060031951
More information about the security-dev
mailing list