[SSSD] [PATCH] Move and extend function for finding principal in keytab

Jan Zelený jzeleny at redhat.com
Tue Mar 29 13:20:47 UTC 2011


Stephen Gallagher <sgallagh at redhat.com> wrote:
> On 03/29/2011 03:50 AM, Jan Zelený wrote:
> > Jan Zelený <jzeleny at redhat.com> wrote:
> >> Jan Zelený <jzeleny at redhat.com> wrote:
> >>> This functionality will be required in tickets #781 and #700.
> >>> 
> >>> Jan
> >> 
> >> This is corrected version - removes some compilation-time error
> >> messages.
> >> 
> >> Jan
> > 
> > Another correction: the code mixup has been eliminated.
> 
> Nack.
> 
> In match_principal(), use
> krb5_unparse_name_flags(ctx, principal,
>                         KRB5_PRINCIPAL_UNPARSE_NO_REALM,
>                         &primary);
> 
> Please use krb5_principal_get_realm() instead of krb5_princ_realm(). The
> latter is deprecated. This will also make your comparisons later
> somewhat easier, as krb5_principal_get_realm() returns a string instead
> of krb5_data.

Fixed patch in attachment.

Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jzeleny-007-4-extended-principal-search.patch
Type: text/x-patch
Size: 11165 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110329/94086bf6/attachment.bin>


More information about the sssd-devel mailing list