diff options
| author | Volker Lendecke <vl@samba.org> | 2010-09-08 17:56:55 -0700 |
|---|---|---|
| committer | Volker Lendecke <vl@samba.org> | 2010-09-09 06:19:23 +0200 |
| commit | c2048db59d52051c045386a0075a11aea2f5bef7 (patch) | |
| tree | e9a118f1ff2c5f7c494222ea4406e80fd4d83081 | |
| parent | 4e4228bd5d970350516162cc716ec2e083ead56f (diff) | |
| download | samba-c2048db59d52051c045386a0075a11aea2f5bef7.tar.gz samba-c2048db59d52051c045386a0075a11aea2f5bef7.tar.bz2 samba-c2048db59d52051c045386a0075a11aea2f5bef7.zip | |
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
| -rw-r--r-- | source3/winbindd/winbindd_pam.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/source3/winbindd/winbindd_pam.c b/source3/winbindd/winbindd_pam.c index 5576417850..2d2f466f80 100644 --- a/source3/winbindd/winbindd_pam.c +++ b/source3/winbindd/winbindd_pam.c @@ -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) { |