[SSSD] [PATCH] Allow new option to specify principal for FAST

Jan Zelený jzeleny at redhat.com
Wed Mar 30 14:13:03 UTC 2011


Stephen Gallagher <sgallagh at redhat.com> wrote:
> On 03/30/2011 08:44 AM, Stephen Gallagher wrote:
> > On 03/29/2011 09:23 AM, Jan Zelený wrote:
> >> Stephen Gallagher <sgallagh at redhat.com> wrote:
> >>> On 03/29/2011 03:53 AM, Jan Zelený wrote:
> >>>> I'm not entirely sure if I got the feature request right but in case I
> >>>> didn't I have already another patch prepared.
> >>>> 
> >>>> https://fedorahosted.org/sssd/ticket/700
> >>> 
> >>> Nack.
> >>> 
> >>> If we have explicitly set a principal, we should NOT fall back to NULL
> >>> on KRB5_KT_NOTFOUND.
> >>> 
> >>> In krb5_child_setup(), I'd rather that you send fast_principal into
> >>> krb5_parse_name() and then get the realm from
> >>> krb5_principal_get_realm().
> >> 
> >> Fixed patch in attachment.
> > 
> > Ack
> 
> Sorry, Nack.
> 
> I just noticed one small bug: at line 1473 of krb5_child.c, you 'goto
> failed' without setting kerr.

Fixed

Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jzeleny-010-3-fast-principal.patch
Type: text/x-patch
Size: 8795 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110330/336114a9/attachment.bin>


More information about the sssd-devel mailing list