On Wed, 17 Jul 2024 15:42:11 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
>
> Fernando Guallini has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   applied suggested patch

@fguallini 
Your change (at version 5a8f99020a837abde2f4e902c16a032f37b36772) is now ready 
to be sponsored by a Committer.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/19480#issuecomment-2236002371

Reply via email to