On Fri, 2011-10-14 at 13:27 -0400, Stephen Gallagher wrote: > On Fri, 2011-10-14 at 17:02 +0200, Jan Zelený wrote: > > > On Fri, 2011-10-14 at 16:10 +0200, Jan Zelený wrote: > > > > https://fedorahosted.org/sssd/ticket/1038 > > > > > > > > With the change I also implemented what Stephen suggested. I schedule > > > > all > > > > callbacks to fire at tevent_timeval_current() directly, instead of > > > > scheduling them one by one. > > > > > > Nack. > > > > > > Don't remove the debug message in sss_dp_req_destructor() if > > > hash_delete() fails. > > > > > > > > > > > > There's no point to the block: > > > /* Free any remaining callback */ > > > if (sdp_req->err_maj == DP_ERR_OK) { > > > sdp_req->err_maj = DP_ERR_FATAL; > > > sdp_req->err_min = EIO; > > > sdp_req->err_msg = discard_const_p(char, "Internal Error"); > > > } > > > Because we're no longer passing this info into callbacks (and sdp_req is > > > being freed). > > > > > > > > > Otherwise it looks fine. No obvious regressions while testing, either. > > > > Thanks for that tip, I'm sending updated patch. > > > Simo made an astute observation off-list. We shouldn't be stealing the > timer event onto NULL. It should move to the parent of sdp_req (which is > rctx). I'm going to make this minor change before committing. Otherwise, > ack.
Change made and pushed to master, sssd-1-6 and sssd-1-5.
signature.asc
Description: This is a digitally signed message part
_______________________________________________ sssd-devel mailing list [email protected] https://fedorahosted.org/mailman/listinfo/sssd-devel
