summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthias Dieter Wallnöfer <mdw@samba.org>2010-12-04 16:36:27 +0100
committerMatthias Dieter Wallnöfer <mdw@samba.org>2010-12-04 16:40:25 +0100
commitee311beabe0dd9f904f05a4c8d8bab065eda1fb7 (patch)
tree2c6f2b2f9aac58cbdcd86c2e7a7e2c2f38a682b9
parent3fb5ae600efaeeb1cb7ee02d465b49bc52d111bf (diff)
downloadsamba-ee311beabe0dd9f904f05a4c8d8bab065eda1fb7.tar.gz
samba-ee311beabe0dd9f904f05a4c8d8bab065eda1fb7.tar.bz2
samba-ee311beabe0dd9f904f05a4c8d8bab065eda1fb7.zip
s4:auth/gensec/gensec_krb5.c - remove a pointless "nt_status" test
There is no operation which sets the "nt_status" before the "if".
-rw-r--r--source4/auth/gensec/gensec_krb5.c11
1 files changed, 3 insertions, 8 deletions
diff --git a/source4/auth/gensec/gensec_krb5.c b/source4/auth/gensec/gensec_krb5.c
index b0ab3f61e7..deb22adb2d 100644
--- a/source4/auth/gensec/gensec_krb5.c
+++ b/source4/auth/gensec/gensec_krb5.c
@@ -670,15 +670,10 @@ static NTSTATUS gensec_krb5_session_info(struct gensec_security *gensec_security
} else {
DEBUG(1, ("Unable to find PAC in ticket from %s, failing to allow access\n",
principal_string));
- return NT_STATUS_ACCESS_DENIED;
- }
-
- krb5_free_principal(context, client_principal);
- free(principal_string);
-
- if (!NT_STATUS_IS_OK(nt_status)) {
+ free(principal_string);
+ krb5_free_principal(context, client_principal);
talloc_free(mem_ctx);
- return nt_status;
+ return NT_STATUS_ACCESS_DENIED;
}
} else {
/* Found pac */