On Tue, 18 May 2021 04:03:16 GMT, Martin Balao <mba...@openjdk.org> wrote:

>> Hi,
>> 
>> Please find in this PR a proposal to fix JDK-8265462 [1].
>> 
>> With this fix, OpenJDK will only use the known slot IDs for the NSS Internal 
>> Module. If the NSS Internal Module has more slots (for example, as a result 
>> of an initialization sequence such as the one triggered from the 
>> libnsssysinit.so library), they will be ignored. The goal is to handle 
>> multiple-slots scenarios while keeping OpenJDK's previous behavior.
>> 
>> No regressions observed in the jdk/sun/security/pkcs11 tests category.
>> 
>> A new regression test was not added as part of this changeset because of its 
>> complexity. It would depend on a specific NSS configuration, or the NSS 
>> library would need to be mocked. I've done manual testing in my environment 
>> and JDK-8265462 [1] has further information about it.
>> 
>> Thanks,
>> Martin.-
>> 
>> --
>> [1] - https://bugs.openjdk.java.net/browse/JDK-8265462
>
> Martin Balao has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Documentation improvements and minor reordering

The updates look good. Thanks! Valerie

-------------

Marked as reviewed by valeriep (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/3661

Reply via email to