Re: RFR: 8155246: Throw error if default java.security file is missing [v2]

2022-08-15 Thread Sean Coffey
On Wed, 10 Aug 2022 16:10:02 GMT, Bernd wrote: >> I think if you just said "If this properties file ..." it would be >> sufficient, as the previous paragraph provided enough context as to what you >> are referring to. > > Btw the current code does not handlmissing or unreadable overwrite files

Integrated: 8155246: Throw error if default java.security file is missing

2022-08-15 Thread Sean Coffey
On Thu, 4 Aug 2022 15:22:23 GMT, Sean Coffey wrote: > In the broken case where the conf/security/java.security configuration file > doesn't exist, the JDK should throw an Error. > > CSR in progress. This pull request has now been integrated. Changeset: 9d7c13eb Author:Sean Coffey URL:

Re: RFR: 8291511: Redefinition of EXIT_FAILURE in libw2k_lsa_auth [v4]

2022-08-15 Thread Weijun Wang
On Sun, 14 Aug 2022 06:39:58 GMT, Julian Waters wrote: >> EXIT_FAILURE is defined as -1 in NativeCreds.c, and is not only unused, but >> also incorrect, as it incorrectly redefines the existing EXIT_FAILURE, which >> is actually 1 on Windows, not -1. Can be safely commented out as it isn't >>

Re: RFR: 8291511: Redefinition of EXIT_FAILURE in libw2k_lsa_auth [v4]

2022-08-15 Thread Julian Waters
On Sun, 14 Aug 2022 06:39:58 GMT, Julian Waters wrote: >> EXIT_FAILURE is defined as -1 in NativeCreds.c, and is not only unused, but >> also incorrect, as it incorrectly redefines the existing EXIT_FAILURE, which >> is actually 1 on Windows, not -1. Can be safely commented out as it isn't >>

Re: RFR: 8291511: Redefinition of EXIT_FAILURE in libw2k_lsa_auth [v4]

2022-08-15 Thread Weijun Wang
On Sun, 14 Aug 2022 06:39:58 GMT, Julian Waters wrote: >> EXIT_FAILURE is defined as -1 in NativeCreds.c, and is not only unused, but >> also incorrect, as it incorrectly redefines the existing EXIT_FAILURE, which >> is actually 1 on Windows, not -1. Can be safely commented out as it isn't >>

Re: RFR: 8215916: The failure reason of an optional JAAS LoginModule is not logged [v5]

2022-08-15 Thread Weijun Wang
On Mon, 8 Aug 2022 17:34:59 GMT, Jayashree Huttanagoudar wrote: >> Could you please review the changes? >> This patch is to address : >> https://bugs.openjdk.org/browse/JDK-8215916?jql=labels%20%3D%20starter-bug > > Jayashree Huttanagoudar has updated the pull request incrementally with one >

Re: RFR: 8291511: Redefinition of EXIT_FAILURE in libw2k_lsa_auth [v5]

2022-08-15 Thread Julian Waters
> EXIT_FAILURE is defined as -1 in NativeCreds.c, and is not only unused, but > also incorrect, as it incorrectly redefines the existing EXIT_FAILURE, which > is actually 1 on Windows, not -1. Can be safely commented out as it isn't > used anywhere, except for in a call to ExitProcess which has

Re: RFR: 8291511: Redefinition of EXIT_FAILURE in libw2k_lsa_auth [v5]

2022-08-15 Thread Weijun Wang
On Mon, 15 Aug 2022 16:17:42 GMT, Julian Waters wrote: >> EXIT_FAILURE is defined as -1 in NativeCreds.c, and is not only unused, but >> also incorrect, as it incorrectly redefines the existing EXIT_FAILURE, which >> is actually 1 on Windows, not -1. Can be safely commented out as it isn't >>

Integrated: 8291511: Redefinition of EXIT_FAILURE in libw2k_lsa_auth

2022-08-15 Thread Julian Waters
On Thu, 28 Jul 2022 17:23:05 GMT, Julian Waters wrote: > EXIT_FAILURE is defined as -1 in NativeCreds.c, and is not only unused, but > also incorrect, as it incorrectly redefines the existing EXIT_FAILURE, which > is actually 1 on Windows, not -1. Can be safely commented out as it isn't > used

Integrated: 8291949: Unexpected extending of SupportedGroups

2022-08-15 Thread Xue-Lei Andrew Fan
On Fri, 5 Aug 2022 05:23:15 GMT, Xue-Lei Andrew Fan wrote: > In the SunJSSE implementation, there are a few unexpected extending of static > class SupportedGroups. It may be nice to clean them up so that the code is > easier to read. Please review this simple code clean up. This pull request