> 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
Valerie Peng has updated the pull request incrementally with one additional commit since the last revision: Changed to use volatile for the default SecureRandom object reference ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/3018/files - new: https://git.openjdk.java.net/jdk/pull/3018/files/48b7fa35..0ad630f3 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3018&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3018&range=00-01 Stats: 34 lines in 2 files changed: 16 ins; 13 del; 5 mod Patch: https://git.openjdk.java.net/jdk/pull/3018.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/3018/head:pull/3018 PR: https://git.openjdk.java.net/jdk/pull/3018
