From c83a40ddbfe0710b9206f98f811bbf9056870ebe Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Tue, 9 Nov 2010 20:29:17 +0100 Subject: 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 Autobuild-Date: Tue Nov 9 20:15:59 UTC 2010 on sn-devel-104 --- source3/auth/auth_winbind.c | 4 +--- 1 file changed, 1 insertion(+), 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) { -- cgit