diff options
| author | Ondrej Kos <okos@redhat.com> | 2013-05-24 13:48:42 +0200 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-05-27 19:20:26 +0200 |
| commit | bfdb2eeed95bde6cd065a9a47a7cb1773990ccfb (patch) | |
| tree | 7290db581e81c34be0ba75e97e4313606af76f5b /src/providers | |
| parent | c89e13316166957c853ed61496a17306e59afc79 (diff) | |
| download | sssd-bfdb2eeed95bde6cd065a9a47a7cb1773990ccfb.tar.gz sssd-bfdb2eeed95bde6cd065a9a47a7cb1773990ccfb.tar.bz2 sssd-bfdb2eeed95bde6cd065a9a47a7cb1773990ccfb.zip | |
Fail with misconfigured id-mapping ranges
https://fedorahosted.org/sssd/ticket/1930
On misconfigured id-mapping range variables, the provider should not
start. We were internally correctly setting error code for failure, but
interruption of startup was not performed.
Also raised the debug level of message for this misconfiguration.
Diffstat (limited to 'src/providers')
| -rw-r--r-- | src/providers/ldap/sdap_idmap.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/providers/ldap/sdap_idmap.c b/src/providers/ldap/sdap_idmap.c index 43db0c83..a81bc98b 100644 --- a/src/providers/ldap/sdap_idmap.c +++ b/src/providers/ldap/sdap_idmap.c @@ -81,10 +81,11 @@ sdap_idmap_init(TALLOC_CTX *mem_ctx, || idmap_upper <= idmap_lower || (idmap_upper-idmap_lower) < rangesize) { - DEBUG(SSSDBG_CRIT_FAILURE, + DEBUG(SSSDBG_FATAL_FAILURE, ("Invalid settings for range selection: [%d][%d][%d]\n", idmap_lower, idmap_upper, rangesize)); ret = EINVAL; + goto done; } if (((idmap_upper - idmap_lower) % rangesize) != 0) { |