RFR: 8328785: IOException: Symbol not found: C_GetInterface for PKCS11 interface prior to V3.0 [v2]
Valerie Peng
valeriep at openjdk.org
Thu Apr 4 20:29:09 UTC 2024
On Thu, 4 Apr 2024 06:23:00 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:
>> Valerie Peng has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Address review comments
>
> src/jdk.crypto.cryptoki/unix/native/libj2pkcs11/p11_md.c line 181:
>
>> 179: C_GetFunctionList = (CK_C_GetFunctionList) dlsym(hModule,
>> 180: getFunctionListStr);
>> 181: if ((systemErrorMessage = dlerror()) != NULL){
>
> Would it make sense to move this check under `if (C_GetFunctionList == NULL)`? If `C_GetFunctionList` is non-null, do we care about the value of dlerror?
Yes, makes sense, I'll change it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18588#discussion_r1552383806
More information about the security-dev
mailing list