[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