RFR: 8356997: /etc/krb5.conf parser should not forbid include/includedir directives after sections [v2]

Weijun Wang weijun at openjdk.org
Wed Jun 18 23:56:30 UTC 2025


On Wed, 18 Jun 2025 01:28:52 GMT, Valerie Peng <valeriep at 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


More information about the security-dev mailing list