Hi Valerie, How about the suggested fix from the bug reporter?
Thanks, Max > On Jul 3, 2020, at 4:52 AM, Valerie Peng <valerie.p...@oracle.com> wrote: > > Hi Max and Sean, > > Can you help reviewing this fix for JDK-8248505? This is the followup fix for > JDK-8246613 "Choose the default SecureRandom algo based on registration > ordering" which you reviewed earlier. Based on the feedback, BCFIPS provider > overrides putService/getService() calls which does not work well with the fix > for JDK-8246613. Thus, I adapted to store the SecureRandom algorithm names > internally and then return the result from getService(...) when > Provider.getDefaultSecureRandomService() is called. Updated the regression > test to include a custom provider which simulates the BCFIPS provider > behavior. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8248505 > > Webrev: http://cr.openjdk.java.net/~valeriep/8248505/webrev.00/ > > Thanks, > Valerie > >