RFR: 8331975: Enable case-insensitive check in ccache and keytab entry lookup [v2]

[Weijun Wang]

On Mon, 20 May 2024 22:36:54 GMT, Valerie Peng <valeriep at openjdk.org> wrote:

>> Weijun Wang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   enhance test
>
> src/java.security.jgss/share/classes/sun/security/krb5/PrincipalName.java line 635:
> 
>> 633:         //      matched = false;
>> 634:         // }
>> 635:         if ((this.nameRealm != null) && (pname.nameRealm != null)) {
> 
> isn't 'nameRealm' non-null? Is this check necessary? Having this check seems to imply their value may be null and it seems incomplete that there is no code handling the cases when one or both of them are null.

You're right. The `realm` field used to be nullable but it cannot be now. I'll fix it. Thanks.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/19216#discussion_r1607363405