summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2010-09-08 17:56:55 -0700
committerVolker Lendecke <vl@samba.org>2010-09-09 06:19:23 +0200
commitc2048db59d52051c045386a0075a11aea2f5bef7 (patch)
treee9a118f1ff2c5f7c494222ea4406e80fd4d83081
parent4e4228bd5d970350516162cc716ec2e083ead56f (diff)
downloadsamba-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.c10
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) {