From: Ralph Boehme <slow@samba.org>
Date: Tue, 9 Jan 2018 15:58:06 +0000 (+0100)
Subject: winbindd: remove a redundant check from winbindd_dual_pam_auth_samlogon
X-Git-Tag: samba-4.8.0rc1~77
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=aae75d124a5555f1cb5bb1b3f081a9f09b51beb3;p=thirdparty%2Fsamba.git

winbindd: remove a redundant check from winbindd_dual_pam_auth_samlogon

result is already checked a few lines above.

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
---

diff --git a/source3/winbindd/winbindd_pam.c b/source3/winbindd/winbindd_pam.c
index 42c4aef878c..bb5bab21209 100644
--- a/source3/winbindd/winbindd_pam.c
+++ b/source3/winbindd/winbindd_pam.c
@@ -1673,7 +1673,8 @@ static NTSTATUS winbindd_dual_pam_auth_samlogon(
 	 * caller, we look up the account flags ourselves - gd */
 
 	if ((request_flags & WBFLAG_PAM_INFO3_TEXT) &&
-	    NT_STATUS_IS_OK(result) && (my_info3->base.acct_flags == 0)) {
+	    (my_info3->base.acct_flags == 0))
+	{
 
 		struct rpc_pipe_client *samr_pipe;
 		struct policy_handle samr_domain_handle, user_pol;