[PATCHES] fix various minor issues

[Simo Sorce]

Series of patches I made up while working toward elimination of the
legacy options this summer.


0001 - Catch a bad buffer sent by glibc.
I reproduced this a couple of times. It was almost certainly due to bad
input sent from the server, but I wasn't able to pinpoint the main
cause. Meanwhile catch the bad buffer so at least we do not segfault
apps.

0002 - Add debug statements.
Found the need to instrument sysdb with some more debug statements to
catch some bugs I was working on.

0003 - Relax memberof constraints.
Instead of completely failing an operation if a member does not actually
exists, simply skip it. This makes the code "lossy" but allows for a
much simpler 2 pass storage technique then having to find a way to track
down dependencies and try to store groups according to them (impossible
for circular dependencies anyway).

0004 - Do not fail enums
If a single store failed for any reason, then the enumeration would stop
completely. Reworked the code to simply skip the bad element and keep
going.

Simo.

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

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 08/24/2009 09:23 AM, Simo Sorce wrote:
>  Series of patches I made up while working toward elimination of the
>  legacy options this summer.
>  
>  
>  0001 - Catch a bad buffer sent by glibc.
>  I reproduced this a couple of times. It was almost certainly due to bad
>  input sent from the server, but I wasn't able to pinpoint the main
>  cause. Meanwhile catch the bad buffer so at least we do not segfault
>  apps.
>  
>  0002 - Add debug statements.
>  Found the need to instrument sysdb with some more debug statements to
>  catch some bugs I was working on.
>  
>  0003 - Relax memberof constraints.
>  Instead of completely failing an operation if a member does not actually
>  exists, simply skip it. This makes the code "lossy" but allows for a
>  much simpler 2 pass storage technique then having to find a way to track
>  down dependencies and try to store groups according to them (impossible
>  for circular dependencies anyway).
>  
>  0004 - Do not fail enums
>  If a single store failed for any reason, then the enumeration would stop
>  completely. Reworked the code to simply skip the bad element and keep
>  going.
>  
>  Simo.
>  
>  
>  
>  ------------------------------------------------------------------------
>  
>  _______________________________________________
>  sssd-devel mailing list
>  sssd-devel(a)lists.fedorahosted.org
>  https://fedorahosted.org/mailman/listinfo/sssd-devel 
Ack to all four patches. These will be included in 0.5.0.

- -- 
Stephen Gallagher
RHCE 804006346421761

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.9 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAkqSoiAACgkQeiVVYja6o6M77wCfXdLjaekqlRRlE9QO6hZ8YMTg
> owAAn0utzGw6VmImzy08Wgp9Tdq1+hmo
> =SiO7
> -----END PGP SIGNATURE-----

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 08/24/2009 10:22 AM, Stephen Gallagher wrote:
>  On 08/24/2009 09:23 AM, Simo Sorce wrote:
> > Series of patches I made up while working toward elimination of the
> > legacy options this summer.
>  
>  
> > 0001 - Catch a bad buffer sent by glibc.
> > I reproduced this a couple of times. It was almost certainly due to bad
> > input sent from the server, but I wasn't able to pinpoint the main
> > cause. Meanwhile catch the bad buffer so at least we do not segfault
> > apps.
>  
> > 0002 - Add debug statements.
> > Found the need to instrument sysdb with some more debug statements to
> > catch some bugs I was working on.
>  
> > 0003 - Relax memberof constraints.
> > Instead of completely failing an operation if a member does not actually
> > exists, simply skip it. This makes the code "lossy" but allows for a
> > much simpler 2 pass storage technique then having to find a way to track
> > down dependencies and try to store groups according to them (impossible
> > for circular dependencies anyway).
>  
> > 0004 - Do not fail enums
> > If a single store failed for any reason, then the enumeration would stop
> > completely. Reworked the code to simply skip the bad element and keep
> > going.
>  
> > Simo.
>  
>  
>  
> > ------------------------------------------------------------------------
>  
> > _______________________________________________
> > sssd-devel mailing list
> > sssd-devel(a)lists.fedorahosted.org
> > https://fedorahosted.org/mailman/listinfo/sssd-devel
>  
>  Ack to all four patches. These will be included in 0.5.0.
>   
Actually, please rebase the last patch before pushing. It conflicts with
the UPN changes.
_______________________________________________
sssd-devel mailing list
sssd-devel(a)lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel

- -- 
Stephen Gallagher
RHCE 804006346421761

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.9 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAkqSpeoACgkQeiVVYja6o6OrlACgrx15I4UqEBc48JwogzKQaRph
> VisAnjCEybm2EtFk7I++SXJtACJPmcyn
> =Hqhg
> -----END PGP SIGNATURE-----

[Simo Sorce]

On Mon, 2009-08-24 at 10:38 -0400, Stephen Gallagher wrote:
>  > Ack to all four patches. These will be included in 0.5.0.
>  > 
>  
>  Actually, please rebase the last patch before pushing. It conflicts
>  with
>  the UPN changes. 
Ok, rebased and pushed.

Simo.

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