mirror of
https://github.com/samba-team/samba.git
synced 2024-12-22 13:34:15 +03:00
Fix struct_based winbind test for 'check machine account'.
The problem here was that we did not fill in the error strings in the success case, and the testsuite checked against the incorrect 'freindly' error string. Andrew Bartlett
This commit is contained in:
parent
86694d429d
commit
df05f11883
@ -262,7 +262,7 @@ static bool torture_winbind_struct_check_machacc(struct torture_context *torture
|
||||
|
||||
torture_assert_str_equal(torture,
|
||||
rep.data.auth.error_string,
|
||||
nt_errstr(NT_STATUS_OK),
|
||||
get_friendly_nt_error_msg(NT_STATUS_OK),
|
||||
"WINBINDD_CHECK_MACHACC ok: error_string");
|
||||
|
||||
torture_assert_int_equal(torture,
|
||||
|
@ -43,13 +43,14 @@ static void wbsrv_samba3_async_auth_epilogue(NTSTATUS status,
|
||||
struct winbindd_response *resp = &s3call->response;
|
||||
if (!NT_STATUS_IS_OK(status)) {
|
||||
resp->result = WINBINDD_ERROR;
|
||||
WBSRV_SAMBA3_SET_STRING(resp->data.auth.nt_status_string,
|
||||
nt_errstr(status));
|
||||
WBSRV_SAMBA3_SET_STRING(resp->data.auth.error_string,
|
||||
get_friendly_nt_error_msg(status));
|
||||
} else {
|
||||
resp->result = WINBINDD_OK;
|
||||
}
|
||||
|
||||
WBSRV_SAMBA3_SET_STRING(resp->data.auth.nt_status_string,
|
||||
nt_errstr(status));
|
||||
WBSRV_SAMBA3_SET_STRING(resp->data.auth.error_string,
|
||||
get_friendly_nt_error_msg(status));
|
||||
|
||||
resp->data.auth.pam_error = nt_status_to_pam(status);
|
||||
resp->data.auth.nt_status = NT_STATUS_V(status);
|
||||
|
Loading…
Reference in New Issue
Block a user