On Mon, 15 Mar 2021 20:39:13 GMT, Valerie Peng <valer...@openjdk.org> wrote:
> Can someone help review this somewhat trivial change? > > Updated JCAUtil class to return the cached SecureRandom object only when the > provider configuration has not changed. > Added a regression test to check the affected classes, i.e. > AlgorithmParameterGenerator, KeyPairGenerator, Cipher, KeyAgreement, > KeyGenerator. > > Thanks, > Valerie src/java.base/share/classes/sun/security/jca/JCAUtil.java line 62: > 60: private static SecureRandom instance = new SecureRandom(); > 61: public static SecureRandom instance(boolean checkConfig) { > 62: synchronized (CachedSecureRandomHolder.class) { Is there a performance regression because of this synchronization? ------------- PR: https://git.openjdk.java.net/jdk/pull/3018