JDK 9 RFR of JDK-8079107: Update TestKeyPairGenerator.java to use random number generator library
Sean Mullan
sean.mullan at oracle.com
Thu Apr 30 19:25:29 UTC 2015
Looks fine to me.
--Sean
On 04/29/2015 08:24 PM, joe darcy wrote:
> Hello,
>
> Please review the patch below to address
>
> JDK-8079107: Update TestKeyPairGenerator.java to use random number
> generator library
>
> The patch implements the recommendations recently sent to jdk9-dev:
>
> http://mail.openjdk.java.net/pipermail/jdk9-dev/2015-April/002164.html
>
> Thanks,
>
> -Joe
>
> diff -r 409888e3ba56 test/sun/security/pkcs11/rsa/TestKeyPairGenerator.java
> --- a/test/sun/security/pkcs11/rsa/TestKeyPairGenerator.java Wed Apr
> 29 16:34:49 2015 -0700
> +++ b/test/sun/security/pkcs11/rsa/TestKeyPairGenerator.java Wed Apr
> 29 17:22:35 2015 -0700
> @@ -24,9 +24,11 @@
> /**
> * @test
> * @bug 4856966
> - * @summary Verify that the RSA KeyPairGenerator works
> + * @summary Verify that the RSA KeyPairGenerator works (use -Dseed=X to
> set PRNG seed)
> * @author Andreas Sterbenz
> * @library ..
> + * @library /lib/testlibrary
> + * @build jdk.testlibrary.*
> * @run main/othervm TestKeyPairGenerator
> * @key intermittent randomness
> */
> @@ -106,7 +108,7 @@
> int[] keyLengths = {512, 512, 1024};
> BigInteger[] pubExps = {null, BigInteger.valueOf(3), null};
> KeyPair[] keyPairs = new KeyPair[3];
> - new Random().nextBytes(data);
> + RandomFactory.getRandom().nextBytes(data);
> KeyPairGenerator kpg = KeyPairGenerator.getInstance("RSA",
> provider);
> for (int i = 0; i < keyLengths.length; i++) {
> int len = keyLengths[i];
>
More information about the security-dev
mailing list