On Mon, 20 May 2024 18:38:15 GMT, Weijun Wang <[email protected]> wrote:
>> Add a new system property to control the name comparison in keytab and
>> ccache entry lookup.
>
> 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.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19216#discussion_r1607361497
