diff options
author | Pavel Reichl <preichl@redhat.com> | 2014-04-25 13:26:19 +0100 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-04-30 19:08:20 +0200 |
commit | 7344bde2d0995a3295b2bea2d347ed1926cbdf2d (patch) | |
tree | a84dfc06ad4c0aee8eaf13025e3468fd6928b2e1 /src/providers | |
parent | 917ff0aa43d3858df1b82f75608601622c12ba57 (diff) | |
download | sssd-7344bde2d0995a3295b2bea2d347ed1926cbdf2d.tar.gz sssd-7344bde2d0995a3295b2bea2d347ed1926cbdf2d.tar.xz sssd-7344bde2d0995a3295b2bea2d347ed1926cbdf2d.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>
(cherry picked from commit cef2384a3a6fc1a1637c6a55e2bced93d28e8fca)
Diffstat (limited to 'src/providers')
-rw-r--r-- | src/providers/ad/ad_subdomains.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/providers/ad/ad_subdomains.c b/src/providers/ad/ad_subdomains.c index a797ceb91..79aa608b8 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); |