diff options
| author | Matthias Dieter Wallnöfer <mdw@samba.org> | 2010-12-04 16:36:27 +0100 |
|---|---|---|
| committer | Matthias Dieter Wallnöfer <mdw@samba.org> | 2010-12-04 16:40:25 +0100 |
| commit | ee311beabe0dd9f904f05a4c8d8bab065eda1fb7 (patch) | |
| tree | 2c6f2b2f9aac58cbdcd86c2e7a7e2c2f38a682b9 /source4 | |
| parent | 3fb5ae600efaeeb1cb7ee02d465b49bc52d111bf (diff) | |
| download | samba-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".
Diffstat (limited to 'source4')
| -rw-r--r-- | source4/auth/gensec/gensec_krb5.c | 11 |
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 */ |