diff options
author | Pavel Reichl <preichl@redhat.com> | 2015-07-23 09:52:47 -0400 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2015-08-14 23:54:48 +0200 |
commit | 76604931b11594394a05df10f8370a1b8bb3e54b (patch) | |
tree | 6f7835b6242471eae341823c306021e77c8cc13a | |
parent | 4f2a07c422fa357ef6651bca8c48b8005280fa1d (diff) | |
download | sssd-76604931b11594394a05df10f8370a1b8bb3e54b.tar.gz sssd-76604931b11594394a05df10f8370a1b8bb3e54b.tar.xz sssd-76604931b11594394a05df10f8370a1b8bb3e54b.zip |
DYNDNS: rename field of sdap_dyndns_update_state
Rename 'use_server_with_nsupdate' to more general name 'fallback_mode'.
Resolves:
https://fedorahosted.org/sssd/ticket/2495
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
-rw-r--r-- | src/providers/ldap/sdap_dyndns.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/providers/ldap/sdap_dyndns.c b/src/providers/ldap/sdap_dyndns.c index f0e3dd855..2a179fd1b 100644 --- a/src/providers/ldap/sdap_dyndns.c +++ b/src/providers/ldap/sdap_dyndns.c @@ -58,7 +58,7 @@ struct sdap_dyndns_update_state { bool update_ptr; bool check_diff; enum be_nsupdate_auth auth_type; - bool use_server_with_nsupdate; + bool fallback_mode; char *update_msg; }; @@ -100,7 +100,7 @@ sdap_dyndns_update_send(TALLOC_CTX *mem_ctx, state->hostname = hostname; state->realm = realm; state->servername = servername; - state->use_server_with_nsupdate = false; + state->fallback_mode = false; state->ttl = ttl; state->be_res = be_ctx->be_res; state->ev = ev; @@ -316,7 +316,7 @@ sdap_dyndns_update_step(struct tevent_req *req) state = tevent_req_data(req, struct sdap_dyndns_update_state); servername = NULL; - if (state->use_server_with_nsupdate == true && + if (state->fallback_mode == true && state->servername) { servername = state->servername; } @@ -359,9 +359,9 @@ sdap_dyndns_update_done(struct tevent_req *subreq) talloc_zfree(subreq); if (ret != EOK) { /* If the update didn't succeed, we can retry using the server name */ - if (state->use_server_with_nsupdate == false && state->servername && + if (state->fallback_mode == false && state->servername && WIFEXITED(child_status) && WEXITSTATUS(child_status) != 0) { - state->use_server_with_nsupdate = true; + state->fallback_mode = true; DEBUG(SSSDBG_MINOR_FAILURE, "nsupdate failed, retrying with server name\n"); ret = sdap_dyndns_update_step(req); @@ -400,7 +400,7 @@ sdap_dyndns_update_ptr_step(struct tevent_req *req) state = tevent_req_data(req, struct sdap_dyndns_update_state); servername = NULL; - if (state->use_server_with_nsupdate == true && + if (state->fallback_mode == true && state->servername) { servername = state->servername; } @@ -443,9 +443,9 @@ sdap_dyndns_update_ptr_done(struct tevent_req *subreq) talloc_zfree(subreq); if (ret != EOK) { /* If the update didn't succeed, we can retry using the server name */ - if (state->use_server_with_nsupdate == false && state->servername && + if (state->fallback_mode == false && state->servername && WIFEXITED(child_status) && WEXITSTATUS(child_status) != 0) { - state->use_server_with_nsupdate = true; + state->fallback_mode = true; DEBUG(SSSDBG_MINOR_FAILURE, "nsupdate failed, retrying with server name\n"); ret = sdap_dyndns_update_ptr_step(req); |