[SSSD] [PATCH] Return from functions in LDAP provider after marking request as failed

Stephen Gallagher sgallagh at redhat.com
Thu Mar 24 17:42:17 UTC 2011


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

On 03/24/2011 12:41 PM, Jakub Hrozek wrote:
> Attached is a patch that fixes a couple of cases in sdap_async_accounts
> where we marke the tevent request as failed but do proceed with the
> function. This can lead to crashes - I verified that the hunk in
> sdap_group_internal_nesting_done() does, at least.
> 
> The last hunk in rfc2307bis_nested_groups_process() may not be necessary
> but I think that it makes the code read better.
> 
> 	Jakub


Ack. The last stanza of the diff is not strictly necessary, but neither
is it harmful.

- -- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAk2LgngACgkQeiVVYja6o6MSJACdFwfxcoM7pt0oDS749DsBR0Em
b8kAnjGaS8YltjzeGJDyAnxn7RCFjPIp
=AoNl
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list