On Wed, 18 Jun 2025 23:53:52 GMT, Weijun Wang <wei...@openjdk.org> wrote:
>> src/java.security.jgss/share/classes/sun/security/krb5/Config.java line 608: >> >>> 606: // Already parsed. This is allowed. >>> 607: return; >>> 608: } >> >> Should this block of code be moved up before the line of `dups = new >> HashSet<>(dups);`? > > That would allow a file to recursively include itself. I see. Would be nice to add a comment about this, so others are aware of this also. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25421#discussion_r2155848853
