summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSimo Sorce <ssorce@redhat.com>2009-08-12 06:15:01 -0400
committerStephen Gallagher <sgallagh@redhat.com>2009-08-12 07:09:36 -0400
commitd0682cba5cd82d7e0eaffe0a3a8e67fb2b782448 (patch)
tree5a17d210e99c5c81eedb9177cdb5d64d64ae22a1
parent2de7dab64517fcaee5198342e9b7890cdf1c7776 (diff)
downloadsssd-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.c16
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);
}