Change looks fine.
Thanks
Max
On 8/30/2016 18:45, Sibabrata Sahoo wrote:
Hi Max,
Here is the updated webrev:
http://cr.openjdk.java.net/~ssahoo/8164922/webrev.01/
I did a minor change other than addressing the following comment. I allowed all
the test case to execute irrespective exception thrown.
Thanks,
Siba
-----Original Message-----
From: Weijun Wang
Sent: Tuesday, August 30, 2016 1:40 PM
To: Sibabrata Sahoo; security-dev@openjdk.java.net
Subject: Re: [9] RFR: 8164922:
sun/security/provider/SecureRandom/AutoReseed.java failed with timeout in
Ubuntu Linux.
Please just use othervm, and remove the finally block. If oldegd is null the
code change will change the environment permanently.
--Max
On 8/29/2016 23:20, Sibabrata Sahoo wrote:
Hi,
Please review the patch for
"sun/security/provider/SecureRandom/AutoReseed.java failed with timeout"
JBS: https://bugs.openjdk.java.net/browse/JDK-8164922
Webrev: http://cr.openjdk.java.net/~ssahoo/8164922/webrev.00/
Description:
The Test was blocked while generating seed. I have providing a fix by
choosing a non-blocking mechanism while seeding by setting
"java.security.egd" System property to "file:/dev/urandom".
Thanks,
Siba
