[SSSD] [PATCH] Eliminate unnecessary explicit timeout for DP account requests

Stephen Gallagher sgallagh at redhat.com
Wed Aug 12 13:13:26 UTC 2009


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

On 08/12/2009 09:09 AM, Simo Sorce wrote:
> On Wed, 2009-08-12 at 09:04 -0400, Stephen Gallagher wrote:
>> On 08/12/2009 08:51 AM, Stephen Gallagher wrote:
>>> D-BUS handles timeouts itself and reports DBUS_ERROR_NO_REPLY if
>>> a timeout fires, so we can rely on this instead of having an
>>> explicit timeout ourselves. Furthermore, the two timeouts present
>>> a potential race condition.
> 
>> Patch was nacked on IRC. I forgot to take advantage of the 'timeout'
>> argument passed in.
>>
>> New patch attached.
>>
> 
> remove also the tevent_timer structure from ndp_req and it is a ack :-)
> 
> Simo.
> 
> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://fedorahosted.org/mailman/listinfo/sssd-devel

Fixed and pushed to master.

- -- 
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/

iEYEARECAAYFAkqCv/IACgkQeiVVYja6o6OjdwCdFfS4ZZ/DDKbhJVLR2OQUUfp6
C/4An1yf8pDt2zZ5eg6RYrSy7nONLqDN
=KwA5
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list