[Freeipa-devel] [PATCH][SSSD] Monitor configuration changes

Simo Sorce ssorce at redhat.com
Fri Apr 10 21:06:59 UTC 2009


On Fri, 2009-04-10 at 13:25 -0400, Stephen Gallagher wrote:
> > 
> > Two additional changes spotted during code review:
> > 1) shutdown_reply() will now call monitor_kill_service if the
> impossible
> > situation happens where the reply handler is called without a valid
> > reply object.
> > 
> > 2) The addition of the service destructor was fundamentally flawed.
> It
> > has been fixed so that it will now DLIST_REMOVE() from
> > svc->mt_ctx->svc_list instead of adding a new "first" attribute.

> Missed one other small fix. Changed return of delist_service to return
> 0
> rather than EOK (for clarity).

ack and puched.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York




More information about the Freeipa-devel mailing list