diff options
| author | Stephen Gallagher <sgallagh@redhat.com> | 2011-04-27 15:28:07 -0400 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2011-05-04 10:14:40 -0400 |
| commit | d3750f3c3a9e232629c8b634b7b5407114667700 (patch) | |
| tree | 3f9a038c2e99edcde2d50ac7ab7ecc7e6429883b /src/util/server.c | |
| parent | b36dfa237c0d9ad3a1c9d59790d6aab3b1e2e82d (diff) | |
| download | sssd-d3750f3c3a9e232629c8b634b7b5407114667700.tar.gz sssd-d3750f3c3a9e232629c8b634b7b5407114667700.tar.xz sssd-d3750f3c3a9e232629c8b634b7b5407114667700.zip | |
Override config file debug_level with command-line
This patch also makes the following changes:
1) The [sssd] debug_level setting no longer acts as a default for
all other sections.
2) We will now skip passing the debug argument to the child
processes from the master unless the SSSD was run with a
command-line argument for the debug level.
https://fedorahosted.org/sssd/ticket/764
Diffstat (limited to 'src/util/server.c')
| -rw-r--r-- | src/util/server.c | 19 |
1 files changed, 11 insertions, 8 deletions
diff --git a/src/util/server.c b/src/util/server.c index 1e8b148d7..977c75117 100644 --- a/src/util/server.c +++ b/src/util/server.c @@ -444,14 +444,17 @@ int server_setup(const char *name, int flags, return ret; } - /* set debug level if any in conf_entry */ - ret = confdb_get_int(ctx->confdb_ctx, ctx, conf_entry, - CONFDB_SERVICE_DEBUG_LEVEL, - debug_level, &debug_level); - if (ret != EOK) { - DEBUG(0, ("Error reading from confdb (%d) [%s]\n", - ret, strerror(ret))); - return ret; + if (debug_level == SSS_UNRESOLVED_DEBUG_LEVEL) { + /* set debug level if any in conf_entry */ + ret = confdb_get_int(ctx->confdb_ctx, ctx, conf_entry, + CONFDB_SERVICE_DEBUG_LEVEL, + SSS_DEFAULT_DEBUG_LEVEL, + &debug_level); + if (ret != EOK) { + DEBUG(0, ("Error reading from confdb (%d) [%s]\n", + ret, strerror(ret))); + return ret; + } } /* same for debug timestamps */ |