summaryrefslogtreecommitdiff
path: root/source4/auth
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2017-05-23 15:05:25 +0200
committerStefan Metzmacher <metze@samba.org>2017-06-16 19:34:17 +0200
commit3d96b093b7d24534ae091b626ea044c6bae7930d (patch)
tree67ca5eb4f9b99efad6f90b36d276e7d61c2e1690 /source4/auth
parent1b88c5d4c0e5da2e4092a06f6cd6bf3c8b767883 (diff)
downloadsamba-3d96b093b7d24534ae091b626ea044c6bae7930d.tar.gz
s4:gensec_gssapi: fix CID 1409781: Possible Control flow issues (DEADCODE)
Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-by: Andreas Schneider <asn@samba.org>
Diffstat (limited to 'source4/auth')
-rw-r--r--source4/auth/gensec/gensec_gssapi.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/source4/auth/gensec/gensec_gssapi.c b/source4/auth/gensec/gensec_gssapi.c
index 8bc5452f3b9..a61b2b2659c 100644
--- a/source4/auth/gensec/gensec_gssapi.c
+++ b/source4/auth/gensec/gensec_gssapi.c
@@ -457,10 +457,11 @@ static NTSTATUS gensec_gssapi_update_internal(struct gensec_security *gensec_sec
switch (gensec_security->gensec_role) {
case GENSEC_CLIENT:
{
- bool fallback = false;
#ifdef SAMBA4_USES_HEIMDAL
struct gsskrb5_send_to_kdc send_to_kdc;
krb5_error_code ret;
+#else
+ bool fallback = false;
#endif
nt_status = gensec_gssapi_client_creds(gensec_security, ev);
@@ -581,10 +582,12 @@ static NTSTATUS gensec_gssapi_update_internal(struct gensec_security *gensec_sec
return NT_STATUS_NO_MEMORY;
}
+#ifndef SAMBA4_USES_HEIMDAL
if (fallback &&
strequal(client_realm, server_realm)) {
goto init_sec_context_done;
}
+#endif /* !SAMBA4_USES_HEIMDAL */
nt_status = gensec_gssapi_setup_server_principal(gensec_gssapi_state,
target_principal,