diff options
| author | Simo Sorce <ssorce@redhat.com> | 2009-08-12 06:15:01 -0400 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2009-08-12 07:09:36 -0400 |
| commit | d0682cba5cd82d7e0eaffe0a3a8e67fb2b782448 (patch) | |
| tree | 5a17d210e99c5c81eedb9177cdb5d64d64ae22a1 | |
| parent | 2de7dab64517fcaee5198342e9b7890cdf1c7776 (diff) | |
| download | sssd-d0682cba5cd82d7e0eaffe0a3a8e67fb2b782448.tar.gz sssd-d0682cba5cd82d7e0eaffe0a3a8e67fb2b782448.tar.xz sssd-d0682cba5cd82d7e0eaffe0a3a8e67fb2b782448.zip | |
Fix reversal of parent and member in groupmod
ticket #101
| -rw-r--r-- | server/tools/sss_groupmod.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/server/tools/sss_groupmod.c b/server/tools/sss_groupmod.c index 5420fc033..175d08d13 100644 --- a/server/tools/sss_groupmod.c +++ b/server/tools/sss_groupmod.c @@ -156,16 +156,16 @@ static void remove_from_groups(struct ops_ctx *data) struct ldb_dn *member_dn; struct tevent_req *req; - member_dn = sysdb_group_dn(data->ctx->sysdb, data, + parent_dn = sysdb_group_dn(data->ctx->sysdb, data, data->domain->name, data->name); - if (!member_dn) { + if (!parent_dn) { return mod_group_done(data, ENOMEM); } - parent_dn = sysdb_group_dn(data->ctx->sysdb, data, + member_dn = sysdb_group_dn(data->ctx->sysdb, data, data->domain->name, data->rmgroups[data->cur]); - if (!parent_dn) { + if (!member_dn) { return mod_group_done(data, ENOMEM); } @@ -213,16 +213,16 @@ static void add_to_groups(struct ops_ctx *data) struct ldb_dn *member_dn; struct tevent_req *req; - member_dn = sysdb_group_dn(data->ctx->sysdb, data, + parent_dn = sysdb_group_dn(data->ctx->sysdb, data, data->domain->name, data->name); - if (!member_dn) { + if (!parent_dn) { return mod_group_done(data, ENOMEM); } - parent_dn = sysdb_group_dn(data->ctx->sysdb, data, + member_dn = sysdb_group_dn(data->ctx->sysdb, data, data->domain->name, data->addgroups[data->cur]); - if (!parent_dn) { + if (!member_dn) { return mod_group_done(data, ENOMEM); } |