mirror of
https://github.com/samba-team/samba.git
synced 2025-03-10 12:58:35 +03:00
s3: Remove redundant flag checks
We're checking these in the parent already (winbindd_pam_auth_send and winbindd_pam_auth_crap_send). No point in doing it in the child as well
This commit is contained in:
parent
4e4228bd5d
commit
c2048db59d
@ -1429,11 +1429,6 @@ enum winbindd_result winbindd_dual_pam_auth(struct winbindd_domain *domain,
|
||||
DEBUG(3, ("[%5lu]: dual pam auth %s\n", (unsigned long)state->pid,
|
||||
state->request->data.auth.user));
|
||||
|
||||
if (!check_request_flags(state->request->flags)) {
|
||||
result = NT_STATUS_INVALID_PARAMETER_MIX;
|
||||
goto done;
|
||||
}
|
||||
|
||||
/* Parse domain and username */
|
||||
|
||||
name_map_status = normalize_name_unmap(state->mem_ctx,
|
||||
@ -1693,11 +1688,6 @@ enum winbindd_result winbindd_dual_pam_auth_crap(struct winbindd_domain *domain,
|
||||
state->request->data.auth_crap.user[sizeof(state->request->data.auth_crap.user)-1]=0;
|
||||
state->request->data.auth_crap.domain[sizeof(state->request->data.auth_crap.domain)-1]=0;
|
||||
|
||||
if (!check_request_flags(state->request->flags)) {
|
||||
result = NT_STATUS_INVALID_PARAMETER_MIX;
|
||||
goto done;
|
||||
}
|
||||
|
||||
name_user = state->request->data.auth_crap.user;
|
||||
|
||||
if (*state->request->data.auth_crap.domain) {
|
||||
|
Loading…
x
Reference in New Issue
Block a user