[SSSD] [PATCHES] Handle invalid domain configuration better

Simo Sorce ssorce at redhat.com
Wed Sep 9 18:20:03 UTC 2009


On Wed, 2009-09-09 at 13:33 -0400, Stephen Gallagher wrote:
> Patch 0001: Ensure that daemonization happens after reading in the
> configuration, so critical configuration errors repord "FAILED" during
> init script startup.

Nack, monitor_process_init() performs a number of operations (like
forking children) that assumes we are operating in the final monitor
process.
Moving become_daemon() after it would make these children loose, and
might interfere with other things like signal handlers etc.

In general I don't think we should do this.

> Patch 0002: Ensure that the minId and maxId values are sensible (minId
> > maxId, neither < 0)

ack this one.

Simo.

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




More information about the sssd-devel mailing list