diff options
| author | Volker Lendecke <vl@samba.org> | 2010-11-09 20:29:17 +0100 |
|---|---|---|
| committer | Volker Lendecke <vlendec@samba.org> | 2010-11-09 20:15:59 +0000 |
| commit | c83a40ddbfe0710b9206f98f811bbf9056870ebe (patch) | |
| tree | f7c3633850598ce964c474aa7330ac0356543989 /source3/auth/auth_winbind.c | |
| parent | 5f3747385776eac1901798aef07364ae8f79f64c (diff) | |
| download | samba-c83a40ddbfe0710b9206f98f811bbf9056870ebe.tar.gz | |
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
Diffstat (limited to 'source3/auth/auth_winbind.c')
| -rw-r--r-- | source3/auth/auth_winbind.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/source3/auth/auth_winbind.c b/source3/auth/auth_winbind.c index dcf7e419c16..603ec9b4109 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) { |