[SSSD] [PATCH] Fixed timeout handling in responders

Stephen Gallagher sgallagh at redhat.com
Fri Oct 14 17:59:29 UTC 2011


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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20111014/c1a4e80d/attachment.sig>


More information about the sssd-devel mailing list