RFR: 8356997: /etc/krb5.conf parser should not forbid include/includedir directives after sections [v2]
Weijun Wang
weijun at openjdk.org
Wed Jun 18 22:19:30 UTC 2025
On Tue, 17 Jun 2025 21:43:06 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 678:
>
>> 676: Path fullp = Paths.get(fileName).toAbsolutePath();
>> 677: Path path = Paths.get(fileName);
>> 678: if (!Files.exists(path)) {
>
> Is `path` needed? Can't you just use `fullp` for the `!Files.exists()` check?
You're right. I'll remove it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25421#discussion_r2155596225
More information about the security-dev
mailing list