diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2009-08-12 18:21:53 +0200 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2009-08-13 09:16:19 -0400 |
| commit | 7fb74edcde011adef137357c916494409a94f1a1 (patch) | |
| tree | 242e11cc307162d3aadd3e6a8df27c8e010a0d73 /server/tools/sss_groupadd.c | |
| parent | 1771d32e945b758326951384f0e17121042ad74e (diff) | |
| download | sssd-7fb74edcde011adef137357c916494409a94f1a1.tar.gz sssd-7fb74edcde011adef137357c916494409a94f1a1.tar.xz sssd-7fb74edcde011adef137357c916494409a94f1a1.zip | |
Tools ID range fixes
The tools did not take the special case where id_max = 0 (no limit)
into account.
Also disallow adding users when ID is specified outside any domain.
Resolves trac tickets #86 and #89
Diffstat (limited to 'server/tools/sss_groupadd.c')
| -rw-r--r-- | server/tools/sss_groupadd.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/server/tools/sss_groupadd.c b/server/tools/sss_groupadd.c index 05f814a77..5c4733d7b 100644 --- a/server/tools/sss_groupadd.c +++ b/server/tools/sss_groupadd.c @@ -237,13 +237,17 @@ int main(int argc, const char **argv) break; case ID_IN_LEGACY_LOCAL: - case ID_OUTSIDE: ret = groupadd_legacy(data); if(ret != EOK) { ERROR("Cannot add group to domain using the legacy tools\n"); } goto fini; + case ID_OUTSIDE: + ERROR("The selected GID is outside all domain ranges\n"); + ret = EXIT_FAILURE; + goto fini; + case ID_IN_OTHER: DEBUG(1, ("Cannot add group to domain %s\n", dom->name)); ERROR("Unsupported domain type"); |