On Thu, 30 May 2024 14:01:20 GMT, Fernando Guallini <fguall...@openjdk.org> wrote:
> As highlighted in the bug description, The test > **security/Security/SynchronizedAccess.java** have some issues: > > 1. it needs to implement the sigalg, otherwise it throws > java.security.NoSuchAlgorithmException . Even though it does not fail as it > catches the exception, it never reaches the removeProvider loop. Also, adding > an extra assertion to verify the local providers were actually removed. > 2. It is reassigning the **provs** variable with Security.getProviders(). > This will start adding/removing some of the system providers, in addition to > the local providers, which it is not the test intent. > > > Test runs in tier2 This pull request has now been integrated. Changeset: 330e520c Author: Fernando Guallini <fguall...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/330e520c1d42d3a9f3e187873dcd8ed7fd561aaf Stats: 96 lines in 1 file changed: 60 ins; 5 del; 31 mod 8028127: Regtest java/security/Security/SynchronizedAccess.java is incorrect Reviewed-by: wetmore, mdonovan, rhalade ------------- PR: https://git.openjdk.org/jdk/pull/19480
