In some places we use tevent_req_post() after tevent_req_error(). We shouldn't do that, since this causes the callback to be called two times.
Fields changed
owner: somebody => mnagy
If this is done ina _send() function it is the proper way to do it. the tevent_req_error() function sets the error, while tevent_req_post() makes sure the callback is called after is being set.
Rember there is no callback set until the _send() function returns, so just calling tevent_req_error() is wrong.
If this happens on non-_send() functions it is probably a bug, but I am aware of no such case.
I can't fins any place in the code where we use tevent_req_post() in a wrong way. Closing.
resolution: => invalid status: new => closed
Right, sorry. I meant to check to see if there is any such usage, thanks for doing that.
rhbz: => 0
Metadata Update from @mnagy: - Issue assigned to mnagy - Issue set to the milestone: SSSD Patches welcome
SSSD is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in SSSD's github repository.
This issue has been cloned to Github and is available here: - https://github.com/SSSD/sssd/issues/1280
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.