On Mon, 9 Sep 2024 14:00:19 GMT, Artur Barashev <d...@openjdk.org> wrote:

> https://bugs.openjdk.org/browse/JDK-8329754

test/jdk/java/security/SecureRandom/ThreadSafe.java line 77:

> 75:             //Bad. Alias of S2, should fail because S2 is marked as 
> ThreadSafe
> 76:             put("alg.Alias.SecureRandom.AliasS2", "S2");
> 77: 

What about an alias for S1? I assume since the attribute is by default false 
the test will still pass. Just want to see it confirmed again.

Also, please add an alias for S4. Let's see if the test still pass for the 
new-service-with-an-alias case.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20916#discussion_r1750362494

Reply via email to