RFR: 8260274: Cipher.init(int, key) does not use highest priority provider for random bytes [v2]

Valerie Peng valeriep at openjdk.java.net
Wed Mar 17 19:25:13 UTC 2021


> 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


More information about the security-dev mailing list