summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2009-07-30 12:02:29 +0200
committerStephen Gallagher <sgallagh@redhat.com>2009-07-31 12:08:40 -0400
commit3f21b54146399bdf3e2a56b8466dc95c3b70283e (patch)
tree3e54d494876c2e5a8039bc734fe9f32fe4cbaa5b
parenta45aea02734e57dd91cdc35b6761364057ca76ae (diff)
downloadsssd-3f21b54146399bdf3e2a56b8466dc95c3b70283e.tar.gz
sssd-3f21b54146399bdf3e2a56b8466dc95c3b70283e.tar.xz
sssd-3f21b54146399bdf3e2a56b8466dc95c3b70283e.zip
Correct check for local domain in tools
When looking for the local domain in the tools, do so by looking on provider value, not domain name. Also removes one redundant lookup of local domain.
-rw-r--r--server/tools/sss_usermod.c12
-rw-r--r--server/tools/tools_util.c4
2 files changed, 2 insertions, 14 deletions
diff --git a/server/tools/sss_usermod.c b/server/tools/sss_usermod.c
index cc94be3a3..ca6463fa0 100644
--- a/server/tools/sss_usermod.c
+++ b/server/tools/sss_usermod.c
@@ -553,18 +553,6 @@ int main(int argc, const char **argv)
"Could not add attribute to changeset\n");
}
-
- /* arguments processed, go on to actual work */
- for (dom = ctx->domains; dom; dom = dom->next) {
- if (strcasecmp(dom->name, "LOCAL") == 0) break;
- }
- if (dom == NULL) {
- ERROR("Could not get LOCAL domain info\n");
- ret = EXIT_FAILURE;
- goto fini;
- }
- data->domain = dom;
-
req = sysdb_transaction_send(ctx, ctx->ev, ctx->sysdb);
if (!req) {
DEBUG(1, ("Could not start transaction (%d)[%s]\n", ret, strerror(ret)));
diff --git a/server/tools/tools_util.c b/server/tools/tools_util.c
index 0d3220ee4..b580300b8 100644
--- a/server/tools/tools_util.c
+++ b/server/tools/tools_util.c
@@ -81,7 +81,7 @@ enum id_domain find_domain_for_id(struct tools_ctx *ctx,
if (id < dom->id_min || id > dom->id_max) {
continue;
} else {
- if (strcasecmp(dom->name, "LOCAL") == 0) {
+ if (strcasecmp(dom->provider, "local") == 0) {
*dom_ret = dom;
return ID_IN_LOCAL;
} else if (is_domain_local_legacy(ctx, dom) == 0) {
@@ -100,7 +100,7 @@ enum id_domain find_domain_for_id(struct tools_ctx *ctx,
} else {
/* No ID specified, find LOCAL */
for (dom = ctx->domains; dom; dom = dom->next) {
- if (strcasecmp(dom->name, "LOCAL") == 0) {
+ if (strcasecmp(dom->provider, "local") == 0) {
*dom_ret = dom;
return ID_IN_LOCAL;
}