Re: RFR: 8333857: Test sun/security/ssl/SSLSessionImpl/ResumeChecksServer.java failed: Existing session was used

2025-07-16 Thread Hai-May Chao
On Wed, 9 Jul 2025 00:03:41 GMT, Anthony Scarpino wrote: > Hi > > I need a review for a client-server threading change that will hopefully > address intermittent failures. This change is similar to > [JDK-8348309](https://bugs.openjdk.org/browse/JDK-8348309). The functional > testing is unc

Withdrawn: 8354469: Keytool exposes the password in plain text when command is piped using | grep

2025-07-16 Thread duke
On Tue, 22 Apr 2025 22:43:08 GMT, Weijun Wang wrote: > Add more description on password handling into the keytool man page. A link > to the man page is now added to the keytool help screen. > > When keytool output is redirected into a file or file, a warning is shown: > > $ keytool -genkeypair

Re: RFR: 8361868: [GCC static analyzer] complains about missing calloc - NULL checks in p11_util.c

2025-07-16 Thread Lutz Schmidt
On Tue, 15 Jul 2025 14:23:19 GMT, Matthias Baesken wrote: > When using the GCC -fanalyzer flag (see > https://developers.redhat.com/articles/2022/04/12/state-static-analysis-gcc-12-compiler# > ) , we get some complaints about missing calloc return value checks for NULL > (we check at some code

RFR: 8361961: Typo in ProtectionDomain.implies

2025-07-16 Thread Shawn Emery
Missing Javadoc meta character for ProtectionDomain.implies. Fix adds missing meta character. No regression tests related to change, but did validate that Javadoc does now render the code element. - Commit messages: - 8361961: Typo in ProtectionDomain.implies Changes: https://gi

Re: RFR: 8361894: sun/security/krb5/config/native/TestDynamicStore.java ensure that the test is run with sudo [v2]

2025-07-16 Thread Rajan Halade
On Tue, 15 Jul 2025 14:10:19 GMT, Mikhail Yankelevich wrote: >> sun/security/krb5/config/native/TestDynamicStore.java is commonly run >> without sudo, which leads to the access related failure. >> Added a ui popup reminding to run the test with sudo, making instructions >> clearer and more no

Re: RFR: 8350709: [JMH] test ProtectionDomainBench failed for 2 threads config [v2]

2025-07-16 Thread Mark Powers
On Wed, 16 Jul 2025 14:38:16 GMT, Andrey Turbanov wrote: >> Mark Powers has updated the pull request incrementally with one additional >> commit since the last revision: >> >> comments from turbanoff > > test/micro/org/openjdk/bench/java/security/ProtectionDomainBench.java line > 112: > >>

Re: RFR: 8350709: [JMH] test ProtectionDomainBench failed for 2 threads config [v2]

2025-07-16 Thread Mark Powers
> [JDK-8350709](https://bugs.openjdk.org/browse/JDK-8350709) Mark Powers has updated the pull request incrementally with one additional commit since the last revision: comments from turbanoff - Changes: - all: https://git.openjdk.org/jdk/pull/26301/files - new: https://git.op

Re: RFR: 8350709: [JMH] test ProtectionDomainBench failed for 2 threads config

2025-07-16 Thread Andrey Turbanov
On Mon, 14 Jul 2025 19:25:38 GMT, Mark Powers wrote: > [JDK-8350709](https://bugs.openjdk.org/browse/JDK-8350709) test/micro/org/openjdk/bench/java/security/ProtectionDomainBench.java line 112: > 110: throws ClassNotFoundException { > 111: if (name.equals(state.class