summaryrefslogtreecommitdiffstats
path: root/src/providers/ad
diff options
context:
space:
mode:
authorPavel Reichl <preichl@redhat.com>2014-04-25 13:26:19 +0100
committerJakub Hrozek <jhrozek@redhat.com>2014-04-30 19:06:38 +0200
commitcef2384a3a6fc1a1637c6a55e2bced93d28e8fca (patch)
treecea04815d6e0ac3d1b20cc8de8b2490679d86555 /src/providers/ad
parent9faab6d48145d3a0d7b9a225ed35bdcaa32eca2c (diff)
downloadsssd-cef2384a3a6fc1a1637c6a55e2bced93d28e8fca.tar.gz
sssd-cef2384a3a6fc1a1637c6a55e2bced93d28e8fca.tar.xz
sssd-cef2384a3a6fc1a1637c6a55e2bced93d28e8fca.zip
AD Provider: bug-fix uninitialized variable
ad_subdomains_refresh() always set value to output parameter 'changes' if EOK is returned. Reviewed-by: Sumit Bose <sbose@redhat.com>
Diffstat (limited to 'src/providers/ad')
-rw-r--r--src/providers/ad/ad_subdomains.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/providers/ad/ad_subdomains.c b/src/providers/ad/ad_subdomains.c
index abcc93cfa..58510c3f9 100644
--- a/src/providers/ad/ad_subdomains.c
+++ b/src/providers/ad/ad_subdomains.c
@@ -404,6 +404,7 @@ static errno_t ad_subdomains_refresh(struct ad_subdomains_ctx *ctx,
if (count == h) {
/* all domains were already accounted for and have been updated */
ret = EOK;
+ *changes = false;
goto done;
}
@@ -664,7 +665,7 @@ static void ad_subdomains_get_root_domain_done(struct tevent_req *req)
struct sysdb_attrs **reply = NULL;
struct ad_subdomains_req_ctx *ctx;
int dp_error = DP_ERR_FATAL;
- bool has_changes;
+ bool has_changes = false;
ctx = tevent_req_callback_data(req, struct ad_subdomains_req_ctx);