Fix confdb issues.

Avoid uninitialized memory messages in valgrind (in _btreemap_get_keys).
Do not free memory we just stored in the btree (in confdb_get_domains_list).
Streamline confdb_get_domains() and remove extra calls when we already have
all the information handy.
Do not store basedn in domain info, the base dn is always calculated out of
the domain name.
Remove the "provider" attribute, it was really used only to distinguish between
LOCAL and other domains, directly check for LOCAL as a special case instead.

1 files changed, 2 insertions, 3 deletions

diff --git a/server/confdb/confdb.h b/server/confdb/confdb.h
index 3bd0d0387..de6790355 100644
--- a/server/confdb/confdb.h
+++ b/server/confdb/confdb.h
@@ -31,10 +31,8 @@
 
 struct sss_domain_info {
     char *name;
-    char *basedn;
+    int timeout;
     int enumerate;
-    bool has_provider;
-    char *provider;
     bool legacy;
 };
 
@@ -76,6 +74,7 @@ int confdb_get_domains(struct confdb_ctx *cdb,
 
 int confdb_get_domains_list(struct confdb_ctx *cdb,
                             TALLOC_CTX *mem_ctx,
+                            struct btreemap **domain_map,
                             const char ***domain_names,
                             int *count);