RFR: 8360979: Remove use of Thread.stop in krb5/auto/KDC.java [v2]
Mikhail Yankelevich
myankelevich at openjdk.org
Tue Jul 15 15:03:22 UTC 2025
On Tue, 15 Jul 2025 11:55:01 GMT, Matthew Donovan <mdonovan at openjdk.org> wrote:
>> Mikhail Yankelevich has updated the pull request incrementally with one additional commit since the last revision:
>>
>> breaking the loop when interrupt flag is set
>
> test/jdk/sun/security/krb5/auto/KDC.java line 1649:
>
>> 1647: } else {
>> 1648: try {
>> 1649: thread1.interrupt();
>
> Each thread catches and ignores `Exception` which means the `interrupt()` call is ineffective. The threads either need to exit when they catch `Exception` or explicitly catch `InterruptedException` and exit when it is thrown.
Done in the next commit
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26247#discussion_r2207756831
More information about the security-dev
mailing list