Hi Valerie
- map.put("CertPathBuilder.PKIX ValidationAlgorithm",
- "RFC5280");
- map.put("CertPathValidator.PKIX ValidationAlgorithm",
- "RFC5280");
+ attrs.put("PKIX ValidationAlgorithm", "RFC5280"); <<<<<<<<<< !!!!
+ add(p, "CertPathBuilder", "PKIX",
+ "sun.security.provider.certpath.SunCertPathBuilder",
+ null, attrs);
+ add(p, "CertPathValidator", "PKIX",
+ "sun.security.provider.certpath.PKIXCertPathValidator",
+ null, attrs);
Here, "PKIX" is the algorithm name and not a part of the attribute name.
Other new aliases and attributes look good.
Thanks
Max
> On Nov 22, 2018, at 2:05 AM, Valerie Peng <valerie.p...@oracle.com> wrote:
>
> Hi,
>
> Can someone help reviewing this fix?
>
> Besides changing the Provider class to use ConcurrentHashMap in order to
> reduce the lock contention on Provider.getService() calls, I also changed the
> security providers in java.base module to register through putService(...)
> calls. Performance is manually verified and mach5 run is clean.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-7092821
> Webrev: http://cr.openjdk.java.net/~valeriep/7092821/webrev.00/
>
> Thanks,
> Valerie
>
>
>
>
>
