summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLukas Slebodnik <lslebodn@redhat.com>2013-10-14 14:26:20 +0200
committerJakub Hrozek <jhrozek@redhat.com>2013-10-15 22:00:18 +0200
commitc93a3ac5ff2e037471d8d6b2e61e2578fdc09315 (patch)
tree119015ad7ecc7fae02c59191e09f9a19273cd677
parentc3b1e99007d5f0069fbbd32a3df799d71ea4882e (diff)
downloadsssd-c93a3ac5ff2e037471d8d6b2e61e2578fdc09315.zip
sssd-c93a3ac5ff2e037471d8d6b2e61e2578fdc09315.tar.gz
sssd-c93a3ac5ff2e037471d8d6b2e61e2578fdc09315.tar.bz2
LDAP: Set default value for dyndns update to false
In some cases, local boolean variable "do_update" could be used without proper initialisation. Clang static analyser warning: "Assigned value is garbage or undefined" It was not a big problem, because non-zero value for boolean variable mean true.
-rw-r--r--src/providers/ldap/sdap_dyndns.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/providers/ldap/sdap_dyndns.c b/src/providers/ldap/sdap_dyndns.c
index bf5fb09..7bcf64f 100644
--- a/src/providers/ldap/sdap_dyndns.c
+++ b/src/providers/ldap/sdap_dyndns.c
@@ -251,7 +251,7 @@ sdap_dyndns_addrs_diff(struct sdap_dyndns_update_state *state, bool *_do_update)
int i;
char **str_dnslist = NULL, **str_local_list = NULL;
char **dns_only = NULL, **local_only = NULL;
- bool do_update;
+ bool do_update = false;
ret = sss_iface_addr_list_as_str_list(state,
state->dns_addrlist, &str_dnslist);