From ee311beabe0dd9f904f05a4c8d8bab065eda1fb7 Mon Sep 17 00:00:00 2001 From: Matthias Dieter Wallnöfer Date: Sat, 4 Dec 2010 16:36:27 +0100 Subject: s4:auth/gensec/gensec_krb5.c - remove a pointless "nt_status" test There is no operation which sets the "nt_status" before the "if". --- source4/auth/gensec/gensec_krb5.c | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) (limited to 'source4/auth/gensec/gensec_krb5.c') 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 */ -- cgit