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

Valerie Peng valeriep at openjdk.org
Thu Jun 19 01:48:37 UTC 2025


On Wed, 18 Jun 2025 23:53:52 GMT, Weijun Wang <weijun at 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


More information about the security-dev mailing list