[8u] RFR 8165996: PKCS11 using NSS throws an error regarding secmod.db when NSS uses sqlite

[Martin Balao]

On 12/30/19 2:02 AM, Andrew John Hughes wrote:
> 
> 
> On 06/11/2019 18:39, Martin Balao wrote:
>> Hi,
>>
>> I'd like to propose the 8u backport of 8165996 [1]. This have been
>> proposed long time ago but I've now rebased the work and bumped the
>> webrev version to 02.
>>
>> Webrev.02:
>>
>>  *
>> http://cr.openjdk.java.net/~mbalao/webrevs/8165996/8165996.jdk8u.jdk.webrev.02/
>>
>> Differences with JDK baseline patch [2]:
>>
>>  * TestNssDbSqlite.java
>>   * @modules jtreg header removed
>>   * Added a check to avoid errors when initialization fails (proposed by
>> @coffeys)
> 
> I don't see this in the main OpenJDK repository. It should be added
> there first, not proposed in a backport.
> 

I've now noticed that this has been fixed by JDK-8206258 [1] [2]. We
should remove the 'sunPKCS11NSSProvider == null' check from Webrev.02 8u
backport, and look at backporting JDK-8206258 after JDK-8165996.

--
[1] - https://bugs.openjdk.java.net/browse/JDK-8206258
[2] - http://hg.openjdk.java.net/jdk/jdk11/rev/f095e3bc2d41