RFR: 8360979: Remove use of Thread.stop in krb5/auto/KDC.java [v2]

Matthew Donovan mdonovan at openjdk.org
Mon Jul 21 11:56:48 UTC 2025


On Tue, 15 Jul 2025 15:03:22 GMT, Mikhail Yankelevich <myankelevich at openjdk.org> wrote:

>> Replaced Thread::stop to Thread::interrupt
>
> 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 1561:

> 1559:                     q.put(new Job(processMessage(Arrays.copyOf(inbuf, p.getLength())), udp, p));
> 1560:                 } catch (InterruptedException e){
> 1561:                     break; // Thread was stopped, so stopping the loop

The comments are probably unnecessary; the InterruptedException tells us that the thread was stopped/interrupted.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26247#discussion_r2218975708


More information about the security-dev mailing list