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

Stephen Gallagher sgallagh at redhat.com
Mon Apr 25 12:13:21 UTC 2011


On Wed, 2011-04-20 at 10:44 -0400, Stephen Gallagher wrote:
> 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.


Pushed to master.
-------------- 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/20110425/f8857ca1/attachment.sig>


More information about the sssd-devel mailing list