[SSSD] [PATCH] Make basic options typed

Simo Sorce ssorce at redhat.com
Mon Sep 14 17:10:59 UTC 2009


On Mon, 2009-09-14 at 18:21 +0200, Sumit Bose wrote:
> On Mon, Sep 14, 2009 at 11:32:40AM -0400, Simo Sorce wrote:
> > On Mon, 2009-09-14 at 14:34 +0200, Sumit Bose wrote:
> > > > 
> > > > Nack. Generates a segfault running "getent passwd user at LDAP"
> > > > 
> > > 
> > > I cannot reproduce this with my setup, but
> > > 
> > > providers/ldap/ldap_id.c:982: warning: 'ldap_id_cleanup_done' defined
> > > but not used
> > > 
> > > and there are some typos in the title and description of the patch.
> > 
> > Ok thanks to Steve I could jump on a machine where this was reproducible
> > (different build flags ?). It was a bug in the code that initialized the
> > blob type in case the option was not specified.
> > 
> > the warning was instead a leftover of another patch I am working on, it
> > shouldn't happen now.
> > 
> > New patch attached.
> > 
> 
> It's still working for me :-). It would be nice to use this scheme in
> other places soon.
> 
> ACK

pushed

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York




More information about the sssd-devel mailing list