On Wed, 22 Sep 2021 10:36:37 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
> Can I please get a review for this change which proposes to fix the issue > noted in https://bugs.openjdk.java.net/browse/JDK-8273894? > > Given the nature of the code in `ReferralsCache`, I haven't been able to add > a new jtreg test case to reproduce the issue and verify this fix. Just to be > sure that this change indeed fixes a reproducible issue, I replicated that > code in a pretty trivial Java application and then verified a similar fix. > > Existing tests in `test/jdk/sun/security/krb5/auto/` continue to pass after > this change. This seems to trace back all the way to initial integration with JDK-8215032. The bug and the fix are understandable. Looks good! @martinuy, who I think is the original author, should ack too. ------------- Marked as reviewed by shade (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/5632
