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