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.
signature.asc
Description: This is a digitally signed message part
_______________________________________________ sssd-devel mailing list [email protected] https://fedorahosted.org/mailman/listinfo/sssd-devel
