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

Stephen Gallagher sgallagh at redhat.com
Wed Apr 20 14:44:53 UTC 2011


On Wed, 2011-03-30 at 16:13 +0200, Jan Zelený wrote:
> 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


Ack.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110420/c25e3b0e/attachment.sig>


More information about the sssd-devel mailing list