On Wed, 18 Jun 2025 01:28:52 GMT, Valerie Peng <valer...@openjdk.org> wrote:

>> Weijun Wang has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   more random testing
>
> 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.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25421#discussion_r2155733258

Reply via email to