RFR: 8349534: Refactor jdk/sun/security/krb5/runNameEquals.sh to java test
Mikhail Yankelevich
duke at openjdk.org
Tue Feb 11 11:20:11 UTC 2025
On Mon, 10 Feb 2025 18:32:21 GMT, Matthew Donovan <mdonovan at openjdk.org> wrote:
>> Refactored the runNameEquals.sh to java test
>
> test/jdk/sun/security/krb5/Krb5NameEquals.java line 145:
>
>> 143: GSSName.NT_HOSTBASED_SERVICE, MECH);
>> 144:
>> 145: if (!name1.equals(name1) || !name1.equals(name3) ||
>
> I don't know much about kerberos but is there ever a chance that `name1.equals(name1)` is false? And the same with name3 which looks to be created the same exact was as name1?
Yes, you're right. Also name1.equals((Object) name1) is always false, so removing in the next commit
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23542#discussion_r1950661891
More information about the security-dev
mailing list