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.
-----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@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 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@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@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel
- -- Stephen Gallagher RHCE 804006346421761
Looking to carve out IT costs? www.redhat.com/carveoutcosts/
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.
sssd-devel@lists.fedorahosted.org