diff options
author | Jakub Hrozek <jhrozek@redhat.com> | 2013-10-18 17:05:38 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-11-14 12:20:18 +0100 |
commit | 30016dd78db3de6eb6ec9c74abe4bcb380504fb2 (patch) | |
tree | 1e48725420ae77f33a6f228e4f435c22ecbbc00a | |
parent | ddc11f44963a079835b2c5560a5b89eb81e13498 (diff) | |
download | sssd-30016dd78db3de6eb6ec9c74abe4bcb380504fb2.tar.gz sssd-30016dd78db3de6eb6ec9c74abe4bcb380504fb2.tar.xz sssd-30016dd78db3de6eb6ec9c74abe4bcb380504fb2.zip |
Inherit ID limits of parent domains if set
https://fedorahosted.org/sssd/ticket/2123
Previously, the subdomains were always unbound even if the administrator
limited the ranges with min_id/max_id. This could have posed problems
when running programs that scan the whole ID space, such as "groupadd
-r".
-rw-r--r-- | src/util/domain_info_utils.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/util/domain_info_utils.c b/src/util/domain_info_utils.c index a6aa5c733..4da0c9d2a 100644 --- a/src/util/domain_info_utils.c +++ b/src/util/domain_info_utils.c @@ -74,9 +74,11 @@ struct sss_domain_info *new_subdomain(TALLOC_CTX *mem_ctx, dom->enumerate = false; dom->fqnames = false; - /* FIXME: get ranges from the server */ - dom->id_min = 0; - dom->id_max = 0xffffffff; + /* If the parent domain explicitly limits ID ranges, the subdomain + * should honour the limits as well. + */ + dom->id_min = parent->id_min ? parent->id_min : 0; + dom->id_max = parent->id_max ? parent->id_max : 0xffffffff; dom->pwd_expiration_warning = parent->pwd_expiration_warning; dom->cache_credentials = parent->cache_credentials; dom->case_sensitive = false; |