summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2010-11-09 20:29:17 +0100
committerVolker Lendecke <vlendec@samba.org>2010-11-09 20:15:59 +0000
commitc83a40ddbfe0710b9206f98f811bbf9056870ebe (patch)
treef7c3633850598ce964c474aa7330ac0356543989
parent5f3747385776eac1901798aef07364ae8f79f64c (diff)
downloadsamba-c83a40ddbfe0710b9206f98f811bbf9056870ebe.tar.gz
samba-c83a40ddbfe0710b9206f98f811bbf9056870ebe.tar.bz2
samba-c83a40ddbfe0710b9206f98f811bbf9056870ebe.zip
s3: Quieten a bogus error message
This happens if you set "auth methods = winbind" without a fallback method. The return NT_STATUS_LOGON_FAILURE; is not strictly require here, because we fall through to the equivalent statement a few lines down, but it makes the code a bit clearer IMO. Autobuild-User: Volker Lendecke <vlendec@samba.org> Autobuild-Date: Tue Nov 9 20:15:59 UTC 2010 on sn-devel-104
-rw-r--r--source3/auth/auth_winbind.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/source3/auth/auth_winbind.c b/source3/auth/auth_winbind.c
index dcf7e419c1..603ec9b410 100644
--- a/source3/auth/auth_winbind.c
+++ b/source3/auth/auth_winbind.c
@@ -98,9 +98,7 @@ static NTSTATUS check_winbind_security(const struct auth_context *auth_context,
if ( auth_method )
return auth_method->auth(auth_context, auth_method->private_data,
mem_ctx, user_info, server_info);
- else
- /* log an error since this should not happen */
- DEBUG(0,("check_winbind_security: ERROR! my_private_data == NULL!\n"));
+ return NT_STATUS_LOGON_FAILURE;
}
if (wbc_status == WBC_ERR_AUTH_ERROR) {