Change looks fine. Thanks Max
> On May 20, 2016, at 6:11 PM, Sibabrata Sahoo <sibabrata.sa...@oracle.com> > wrote: > > I missed to include one more test. > Here is the updated one: http://cr.openjdk.java.net/~ssahoo/8157417/webrev.01/ > > sun/security/provider/SeedGenerator/SeedGeneratorChoice.java might also fail > in Linux without the fix provided. > > Thanks, > Siba > > From: Sibabrata Sahoo > Sent: Friday, May 20, 2016 3:06 PM > To: Weijun Wang; security-dev@openjdk.java.net > Subject: [9] RFR: 8157417: Some of SecureRandom test might get timed out in > linux. > > Hi, > > Please review the following fix for SecureRandom test which might cause > timeout in Linux. > > webrev: http://cr.openjdk.java.net/~ssahoo/8157417/webrev.00/ > JBS: https://bugs.openjdk.java.net/browse/JDK-8157417 > > The change includes: > - Added “-Djava.security.egd=file:/dev/urandom” without which the > test might timeout in Linux. The change included in “MultiThreadTest.java” > and “SerializedSeedTest.java”. > - Added finally inside “GetInstanceTest.java” to preserve the > original security property correctly in case of rare failure. > > Thanks, > Siba
