Skip to content

Commit

Permalink
Treat a missing provider entry as a config error
Browse files Browse the repository at this point in the history
  • Loading branch information
sgallagher committed Jun 8, 2009
1 parent eb92196 commit 8c85480
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions server/confdb/confdb.c
Original file line number Diff line number Diff line change
Expand Up @@ -730,6 +730,11 @@ int confdb_get_domain(struct confdb_ctx *cdb,
goto done;
}
}
else {
DEBUG(0, ("Domain [%s] does not specify a provider, disabling!\n",
domain->name));
ret = EINVAL;
}

domain->timeout = ldb_msg_find_attr_as_int(res->msgs[0],
"timeout", 0);
Expand Down Expand Up @@ -805,8 +810,8 @@ int confdb_get_domains(struct confdb_ctx *cdb,
for (i = 0; domlist[i]; i++) {
ret = confdb_get_domain(cdb, mem_ctx, domlist[i], &domain);
if (ret) {
DEBUG(0, ("Error (%d [%s]) retrieving domain %s, skipping!\n",
ret, strerror(ret), domains[i]));
DEBUG(0, ("Error (%d [%s]) retrieving domain [%s], skipping!\n",
ret, strerror(ret), domlist[i]));
continue;
}

Expand Down

0 comments on commit 8c85480

Please sign in to comment.