diff options
| author | Pavel Reichl <preichl@redhat.com> | 2014-10-30 16:50:27 +0000 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-11-04 12:35:41 +0100 |
| commit | ad132722d6f3393ae1e6d720a222a0f880f2ea54 (patch) | |
| tree | a07b628955cc594b18a32f8ae5ba9a1cab7633bd /src/confdb | |
| parent | a1bd8bc666df7fa696523ec8ec1dfe3d79780588 (diff) | |
| download | sssd-ad132722d6f3393ae1e6d720a222a0f880f2ea54.tar.gz sssd-ad132722d6f3393ae1e6d720a222a0f880f2ea54.tar.xz sssd-ad132722d6f3393ae1e6d720a222a0f880f2ea54.zip | |
CONFDB: Detect&fix misconf opt refresh_expired_interval
Related to:
https://fedorahosted.org/sssd/ticket/2102
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
Diffstat (limited to 'src/confdb')
| -rw-r--r-- | src/confdb/confdb.c | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/src/confdb/confdb.c b/src/confdb/confdb.c index 8443fe553..c55a945a4 100644 --- a/src/confdb/confdb.c +++ b/src/confdb/confdb.c @@ -1058,6 +1058,21 @@ static int confdb_get_domain_internal(struct confdb_ctx *cdb, goto done; } + /* detect and fix misconfiguration */ + if (domain->refresh_expired_interval > entry_cache_timeout) { + DEBUG(SSSDBG_CONF_SETTINGS, + "refresh_expired_interval (%d) cannot be greater then " + "entry_cache_timeout (%u)\n", + domain->refresh_expired_interval, entry_cache_timeout); + + domain->refresh_expired_interval = 0.75 * entry_cache_timeout; + + DEBUG(SSSDBG_CONF_SETTINGS, + "refresh_expired_interval is being set to recommended value " + "entry_cache_timeout * 0.75 (%u).\n", + domain->refresh_expired_interval); + } + /* Set the PAM warning time, if specified. If not specified, pass on * the "not set" value of "-1" which means "use provider default". The * value 0 means "always display the warning if server sends one" */ |