Rajan Halade

Results 14 comments of Rajan Halade

I have update the bug summary as suggested by Sean.

I put this on hold to debug a potential issue I am seeing.

I am closing this pull request as we still don't have OCA signed from CA.

What is changed in these files? I still see the binary files.

@koushikthirupattur can you please add a regression test to provide coverage for `nextLong`?

Wherever you have added a stronger algorithm to test is fine with me. But in some tests, where possible, instead of replacing the weak algorithm, the test should be updated...

The changes in this PR should not be integrated until https://github.com/openjdk/jdk/pull/21537 is integrated. The updates in SecurityUtils should be used from https://github.com/openjdk/jdk/pull/21537.

Thanks for updating PR with suggested changes. The updates look good to me. I recommend to wait for another reviewer (@ascarpino ?) to approve as well.