From c2048db59d52051c045386a0075a11aea2f5bef7 Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Wed, 8 Sep 2010 17:56:55 -0700 Subject: 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 --- source3/winbindd/winbindd_pam.c | 10 ---------- 1 file changed, 10 deletions(-) (limited to 'source3/winbindd') 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) { -- cgit