r7415: * big change -- volker's new async winbindd from trunk
[samba.git] / source / nsswitch / winbindd_pam.c
index 81d4e69ccd6b819175e6fd60d3fa6e7c966b2bb6..97dc35c0e7e2ded1981898b3c597f35d7f549af0 100644 (file)
@@ -43,7 +43,7 @@ static NTSTATUS append_info3_as_ndr(TALLOC_CTX *mem_ctx,
        }
 
        size = prs_data_size(&ps);
-       state->response.extra_data = malloc(size);
+       state->response.extra_data = SMB_MALLOC(size);
        if (!state->response.extra_data) {
                prs_mem_free(&ps);
                return NT_STATUS_NO_MEMORY;
@@ -59,7 +59,7 @@ static NTSTATUS check_info3_in_group(TALLOC_CTX *mem_ctx,
                                     NET_USER_INFO_3 *info3,
                                     const char *group_sid) 
 {
-       DOM_SID required_membership_sid;
+       DOM_SID require_membership_of_sid;
        DOM_SID *all_sids;
        size_t num_all_sids = (2 + info3->num_groups2 + info3->num_other_sids);
        size_t i, j = 0;
@@ -71,14 +71,14 @@ static NTSTATUS check_info3_in_group(TALLOC_CTX *mem_ctx,
                return NT_STATUS_OK;
        }
        
-       if (!string_to_sid(&required_membership_sid, group_sid)) {
+       if (!string_to_sid(&require_membership_of_sid, group_sid)) {
                DEBUG(0, ("check_info3_in_group: could not parse %s as a SID!", 
                          group_sid));
 
                return NT_STATUS_INVALID_PARAMETER;
        }
 
-       all_sids = talloc(mem_ctx, sizeof(DOM_SID) * num_all_sids);
+       all_sids = TALLOC_ARRAY(mem_ctx, DOM_SID, num_all_sids);
        if (!all_sids)
                return NT_STATUS_NO_MEMORY;
 
@@ -108,9 +108,9 @@ static NTSTATUS check_info3_in_group(TALLOC_CTX *mem_ctx,
        
                sid_copy(&all_sids[j], &(info3->dom_sid.sid));
                
-               if (!sid_append_rid(&all_sids[j], info3->gids[j].g_rid)) {
+               if (!sid_append_rid(&all_sids[j], info3->gids[i].g_rid)) {
                        DEBUG(3,("could not append additional group rid 0x%x\n",
-                               info3->gids[j].g_rid));                 
+                               info3->gids[i].g_rid));                 
                                
                        return NT_STATUS_INVALID_PARAMETER;
                }
@@ -123,9 +123,9 @@ static NTSTATUS check_info3_in_group(TALLOC_CTX *mem_ctx,
            http://www.microsoft.com/windows2000/techinfo/administration/security/sidfilter.asp
          */
 
-       for (i = 0; i < info3->num_other_sids; j++) {
+       for (i = 0; i < info3->num_other_sids; i++) {
                sid_copy(&all_sids[info3->num_groups2 + i + 2],
-                        &info3->other_sids[j].sid);
+                        &info3->other_sids[i].sid);
                j++;
        }
 
@@ -133,9 +133,9 @@ static NTSTATUS check_info3_in_group(TALLOC_CTX *mem_ctx,
                fstring sid1, sid2;
                DEBUG(10, ("User has SID: %s\n", 
                           sid_to_string(sid1, &all_sids[i])));
-               if (sid_equal(&required_membership_sid, &all_sids[i])) {
+               if (sid_equal(&require_membership_of_sid, &all_sids[i])) {
                        DEBUG(10, ("SID %s matches %s - user permitted to authenticate!\n", 
-                                  sid_to_string(sid1, &required_membership_sid), sid_to_string(sid2, &all_sids[i])));
+                                  sid_to_string(sid1, &require_membership_of_sid), sid_to_string(sid2, &all_sids[i])));
                        return NT_STATUS_OK;
                }
        }
@@ -145,24 +145,99 @@ static NTSTATUS check_info3_in_group(TALLOC_CTX *mem_ctx,
        return NT_STATUS_LOGON_FAILURE;
 }
 
+static struct winbindd_domain *find_auth_domain(const char *domain_name)
+{
+       struct winbindd_domain *domain;
+
+       if (IS_DC) {
+               domain = find_domain_from_name_noinit(domain_name);
+               if (domain == NULL) {
+                       DEBUG(3, ("Authentication for domain [%s] "
+                                 "as it is not a trusted domain\n", 
+                                 domain_name));
+               }
+               return domain;
+       }
+
+       if (is_myname(domain_name)) {
+               DEBUG(3, ("Authentication for domain %s (local domain "
+                         "to this server) not supported at this "
+                         "stage\n", domain_name));
+               return NULL;
+       }
+
+       return find_our_domain();
+}
+
+static void set_auth_errors(struct winbindd_response *resp, NTSTATUS result)
+{
+       resp->data.auth.nt_status = NT_STATUS_V(result);
+       fstrcpy(resp->data.auth.nt_status_string, nt_errstr(result));
+
+       /* we might have given a more useful error above */
+       if (*resp->data.auth.error_string == '\0') 
+               fstrcpy(resp->data.auth.error_string,
+                       get_friendly_nt_error_msg(result));
+       resp->data.auth.pam_error = nt_status_to_pam(result);
+}
+
 /**********************************************************************
  Authenticate a user with a clear text password
 **********************************************************************/
 
-enum winbindd_result winbindd_pam_auth(struct winbindd_cli_state *state) 
+enum winbindd_result winbindd_pam_auth(struct winbindd_cli_state *state)
+{
+       struct winbindd_domain *domain;
+       fstring name_domain, name_user;
+
+       /* Ensure null termination */
+       state->request.data.auth.user
+               [sizeof(state->request.data.auth.user)-1]='\0';
+
+       /* Ensure null termination */
+       state->request.data.auth.pass
+               [sizeof(state->request.data.auth.pass)-1]='\0';
+
+       DEBUG(3, ("[%5lu]: pam auth %s\n", (unsigned long)state->pid,
+                 state->request.data.auth.user));
+
+       /* Parse domain and username */
+       
+       parse_domain_user(state->request.data.auth.user,
+                         name_domain, name_user);
+
+       domain = find_auth_domain(name_domain);
+
+       if (domain == NULL) {
+               set_auth_errors(&state->response, NT_STATUS_NO_SUCH_USER);
+               DEBUG(5, ("Plain text authentication for %s returned %s "
+                         "(PAM: %d)\n",
+                         state->request.data.auth.user, 
+                         state->response.data.auth.nt_status_string,
+                         state->response.data.auth.pam_error));
+               return WINBINDD_ERROR;
+       }
+
+       async_domain_request(state->mem_ctx, domain,
+                            &state->request, &state->response,
+                            request_finished_cont, state);
+       return WINBINDD_PENDING;
+}
+
+enum winbindd_result winbindd_dual_pam_auth(struct winbindd_domain *domain,
+                                           struct winbindd_cli_state *state) 
 {
        NTSTATUS result;
        fstring name_domain, name_user;
-       unsigned char trust_passwd[16];
-       time_t last_change_time;
-       uint32 sec_channel_type;
+       const char *srv_name_slash;
         NET_USER_INFO_3 info3;
-        struct cli_state *cli = NULL;
+       unsigned char *session_key;
+       struct rpc_pipe_client *pipe_cli;
        uchar chal[8];
-       TALLOC_CTX *mem_ctx = NULL;
        DATA_BLOB lm_resp;
        DATA_BLOB nt_resp;
        DOM_CRED ret_creds;
+       DOM_CRED *credentials;
        int attempts = 0;
        unsigned char local_lm_response[24];
        unsigned char local_nt_response[24];
@@ -178,25 +253,67 @@ enum winbindd_result winbindd_pam_auth(struct winbindd_cli_state *state)
        DEBUG(3, ("[%5lu]: pam auth %s\n", (unsigned long)state->pid,
                  state->request.data.auth.user));
 
-       if (!(mem_ctx = talloc_init("winbind pam auth for %s", state->request.data.auth.user))) {
-               DEBUG(0, ("winbindd_pam_auth: could not talloc_init()!\n"));
-               result = NT_STATUS_NO_MEMORY;
-               goto done;
-       }
-
        /* Parse domain and username */
        
        parse_domain_user(state->request.data.auth.user, name_domain, name_user);
 
        /* do password magic */
        
+
        generate_random_buffer(chal, 8);
-       SMBencrypt(state->request.data.auth.pass, chal, local_lm_response);
+       if (lp_client_ntlmv2_auth()) {
+               DATA_BLOB server_chal;
+               DATA_BLOB names_blob;
+               DATA_BLOB nt_response;
+               DATA_BLOB lm_response;
+               server_chal = data_blob_talloc(state->mem_ctx, chal, 8); 
+               
+               /* note that the 'workgroup' here is a best guess - we don't know
+                  the server's domain at this point.  The 'server name' is also
+                  dodgy... 
+               */
+               names_blob = NTLMv2_generate_names_blob(global_myname(), lp_workgroup());
                
-       SMBNTencrypt(state->request.data.auth.pass, chal, local_nt_response);
+               if (!SMBNTLMv2encrypt(name_user, name_domain, 
+                                     state->request.data.auth.pass, 
+                                     &server_chal, 
+                                     &names_blob,
+                                     &lm_response, &nt_response, NULL)) {
+                       data_blob_free(&names_blob);
+                       data_blob_free(&server_chal);
+                       DEBUG(0, ("winbindd_pam_auth: SMBNTLMv2encrypt() failed!\n"));
+                       result = NT_STATUS_NO_MEMORY;
+                       goto done;
+               }
+               data_blob_free(&names_blob);
+               data_blob_free(&server_chal);
+               lm_resp = data_blob_talloc(state->mem_ctx, lm_response.data,
+                                          lm_response.length);
+               nt_resp = data_blob_talloc(state->mem_ctx, nt_response.data,
+                                          nt_response.length);
+               data_blob_free(&lm_response);
+               data_blob_free(&nt_response);
+
+       } else {
+               if (lp_client_lanman_auth() 
+                   && SMBencrypt(state->request.data.auth.pass, 
+                                 chal, 
+                                 local_lm_response)) {
+                       lm_resp = data_blob_talloc(state->mem_ctx, 
+                                                  local_lm_response, 
+                                                  sizeof(local_lm_response));
+               } else {
+                       lm_resp = data_blob(NULL, 0);
+               }
+               SMBNTencrypt(state->request.data.auth.pass, 
+                            chal,
+                            local_nt_response);
+
+               nt_resp = data_blob_talloc(state->mem_ctx, 
+                                          local_nt_response, 
+                                          sizeof(local_nt_response));
+       }
 
-       lm_resp = data_blob_talloc(mem_ctx, local_lm_response, sizeof(local_lm_response));
-       nt_resp = data_blob_talloc(mem_ctx, local_nt_response, sizeof(local_nt_response));
        
        /* what domain should we contact? */
        
@@ -215,83 +332,98 @@ enum winbindd_result winbindd_pam_auth(struct winbindd_cli_state *state)
                        goto done;
                }
 
-               if (!(contact_domain = find_our_domain())) {
-                       DEBUG(1, ("Authentication for [%s] -> [%s]\\[%s] in our domain failed - we can't find our domain!\n", 
-                                 state->request.data.auth.user, name_domain, name_user)); 
-                       result = NT_STATUS_NO_SUCH_USER;
-                       goto done;
-               }
+               contact_domain = find_our_domain();
        }
 
-       if ( !get_trust_pw(contact_domain->name, trust_passwd, &last_change_time, &sec_channel_type) ) {
-               result = NT_STATUS_CANT_ACCESS_DOMAIN_INFO;
-               goto done;
+       srv_name_slash = talloc_asprintf(state->mem_ctx, "\\\\%s",
+                                        contact_domain->dcname);
+       if (srv_name_slash == NULL) {
+               DEBUG(0, ("talloc_asprintf failed\n"));
+               return WINBINDD_ERROR;
        }
-
+               
        /* check authentication loop */
 
        do {
+               DOM_CRED clnt_creds;
+
                ZERO_STRUCT(info3);
                ZERO_STRUCT(ret_creds);
                retry = False;
-       
-               /* Don't shut this down - it belongs to the connection cache code */
-               result = cm_get_netlogon_cli(contact_domain, trust_passwd, 
-                                            sec_channel_type, False, &cli);
+
+               result = cm_connect_netlogon(contact_domain, state->mem_ctx,
+                                            &pipe_cli, &session_key,
+                                            &credentials);
 
                if (!NT_STATUS_IS_OK(result)) {
                        DEBUG(3, ("could not open handle to NETLOGON pipe\n"));
                        goto done;
                }
 
-               result = cli_netlogon_sam_network_logon(cli, mem_ctx,
-                                                       &ret_creds,
-                                                       name_user, name_domain, 
-                                                       global_myname(), chal, 
-                                                       lm_resp, nt_resp,
-                                                       &info3);
+               credentials->timestamp.time = time(NULL);
+               memcpy(&clnt_creds, credentials, sizeof(clnt_creds));
+
+               /* Calculate the new credentials. */
+               cred_create(session_key, &credentials->challenge,
+                           clnt_creds.timestamp, &(clnt_creds.challenge));
+
+               result = rpccli_netlogon_sam_network_logon(pipe_cli,
+                                                          state->mem_ctx,
+                                                          srv_name_slash,
+                                                          &clnt_creds,
+                                                          &ret_creds,
+                                                          name_user,
+                                                          name_domain, 
+                                                          global_myname(),
+                                                          chal, lm_resp,
+                                                          nt_resp, &info3,
+                                                          session_key);
                attempts += 1;
-               
-               /* We have to try a second time as cm_get_netlogon_cli
+
+               /* We have to try a second time as cm_connect_netlogon
                   might not yet have noticed that the DC has killed
                   our connection. */
 
-               if ( cli->fd == -1 ) {
+               if (NT_STATUS_EQUAL(result, NT_STATUS_UNSUCCESSFUL)) {
                        retry = True;
                        continue;
-               } 
+               }
                
-               /* if we get access denied, a possible cuase was that we had and open
-                  connection to the DC, but someone changed our machine account password
-                  out from underneath us using 'net rpc changetrustpw' */
+               /* if we get access denied, a possible cause was that we had
+                  and open connection to the DC, but someone changed our
+                  machine account password out from underneath us using 'net
+                  rpc changetrustpw' */
                   
-               if ( NT_STATUS_V(result) == NT_STATUS_V(NT_STATUS_ACCESS_DENIED) ) {
-                       DEBUG(3,("winbindd_pam_auth: sam_logon returned ACCESS_DENIED.  Maybe the trust account "
-                               "password was changed and we didn't know it.  Killing connections to domain %s\n",
+               if ( NT_STATUS_EQUAL(result, NT_STATUS_ACCESS_DENIED) ) {
+                       DEBUG(3,("winbindd_pam_auth: sam_logon returned "
+                                "ACCESS_DENIED.  Maybe the trust account "
+                               "password was changed and we didn't know it. "
+                                "Killing connections to domain %s\n",
                                name_domain));
-                       winbindd_cm_flush();
+                       invalidate_cm_connection(&contact_domain->conn);
                        retry = True;
-                       cli = NULL;
                } 
                
        } while ( (attempts < 2) && retry );
 
-        if (cli != NULL) {
-               /* We might have come out of the loop above with cli == NULL,
-                  so don't dereference that. */
-               clnt_deal_with_creds(cli->sess_key, &(cli->clnt_cred), &ret_creds);
+       if (NT_STATUS_IS_OK(result) &&
+           (!clnt_deal_with_creds(session_key, credentials,
+                                 &ret_creds))) {
+               DEBUG(3, ("DC %s sent wrong credentials\n",
+                         pipe_cli->cli->srv_name_slash));
+               result = NT_STATUS_ACCESS_DENIED;
        }
-       
+
        if (NT_STATUS_IS_OK(result)) {
-               netsamlogon_cache_store( cli->mem_ctx, &info3 );
+               netsamlogon_cache_store(state->mem_ctx, name_user, &info3);
                wcache_invalidate_samlogon(find_domain_from_name(name_domain), &info3);
 
                /* Check if the user is in the right group */
 
-               if (!NT_STATUS_IS_OK(result = check_info3_in_group(mem_ctx, &info3, state->request.data.auth.required_membership_sid))) {
+               if (!NT_STATUS_IS_OK(result = check_info3_in_group(state->mem_ctx, &info3, state->request.data.auth.require_membership_of_sid))) {
                        DEBUG(3, ("User %s is not in the required group (%s), so plaintext authentication is rejected\n",
                                  state->request.data.auth.user, 
-                                 state->request.data.auth.required_membership_sid));
+                                 state->request.data.auth.require_membership_of_sid));
                }
        }
 
@@ -317,7 +449,7 @@ done:
        if ( NT_STATUS_IS_OK(result) &&
             (state->request.flags & WBFLAG_PAM_AFS_TOKEN) ) {
 
-               char *afsname = strdup(lp_afs_username_map());
+               char *afsname = SMB_STRDUP(lp_afs_username_map());
                char *cell;
 
                if (afsname == NULL) goto no_token;
@@ -326,10 +458,22 @@ done:
                afsname = realloc_string_sub(afsname, "%u", name_user);
                afsname = realloc_string_sub(afsname, "%U", name_user);
 
+               {
+                       DOM_SID user_sid;
+                       fstring sidstr;
+
+                       sid_copy(&user_sid, &info3.dom_sid.sid);
+                       sid_append_rid(&user_sid, info3.user_rid);
+                       sid_to_string(sidstr, &user_sid);
+                       afsname = realloc_string_sub(afsname, "%s", sidstr);
+               }
+
                if (afsname == NULL) goto no_token;
 
                strlower_m(afsname);
 
+               DEBUG(10, ("Generating token for user %s\n", afsname));
+
                cell = strchr(afsname, '@');
 
                if (cell == NULL) goto no_token;
@@ -349,9 +493,6 @@ done:
                SAFE_FREE(afsname);
        }
                
-       if (mem_ctx) 
-               talloc_destroy(mem_ctx);
-       
        return NT_STATUS_IS_OK(result) ? WINBINDD_OK : WINBINDD_ERROR;
 }
 
@@ -359,16 +500,80 @@ done:
  Challenge Response Authentication Protocol 
 **********************************************************************/
 
-enum winbindd_result winbindd_pam_auth_crap(struct winbindd_cli_state *state) 
+enum winbindd_result winbindd_crap_auth(struct winbindd_cli_state *state)
 {
+       struct winbindd_domain *domain = NULL;
+       const char *domain_name = NULL;
        NTSTATUS result;
-       unsigned char trust_passwd[16];
-       time_t last_change_time;
-       uint32 sec_channel_type;
+
+       if (!state->privileged) {
+               char *error_string = NULL;
+               DEBUG(2, ("winbindd_pam_auth_crap: non-privileged access "
+                         "denied.  !\n"));
+               DEBUGADD(2, ("winbindd_pam_auth_crap: Ensure permissions "
+                            "on %s are set correctly.\n",
+                            get_winbind_priv_pipe_dir()));
+               /* send a better message than ACCESS_DENIED */
+               error_string = talloc_asprintf(state->mem_ctx,
+                                              "winbind client not authorized "
+                                              "to use winbindd_pam_auth_crap."
+                                              " Ensure permissions on %s "
+                                              "are set correctly.",
+                                              get_winbind_priv_pipe_dir());
+               fstrcpy(state->response.data.auth.error_string, error_string);
+               result = NT_STATUS_ACCESS_DENIED;
+               goto done;
+       }
+
+       /* Ensure null termination */
+       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;
+
+       DEBUG(3, ("[%5lu]: pam auth crap domain: [%s] user: %s\n",
+                 (unsigned long)state->pid,
+                 state->request.data.auth_crap.domain,
+                 state->request.data.auth_crap.user));
+
+       if (*state->request.data.auth_crap.domain != '\0') {
+               domain_name = state->request.data.auth_crap.domain;
+       } else if (lp_winbind_use_default_domain()) {
+               domain_name = lp_workgroup();
+       }
+
+       if (domain_name != NULL)
+               domain = find_auth_domain(domain_name);
+
+       if (domain != NULL) {
+               async_domain_request(state->mem_ctx, domain,
+                                    &state->request, &state->response,
+                                    request_finished_cont, state);
+               return WINBINDD_PENDING;
+       }
+
+       result = NT_STATUS_NO_SUCH_USER;
+
+ done:
+       set_auth_errors(&state->response, result);
+       DEBUG(5, ("CRAP authentication for %s returned %s (PAM: %d)\n",
+                 state->request.data.auth.user, 
+                 state->response.data.auth.nt_status_string,
+                 state->response.data.auth.pam_error));
+       return WINBINDD_ERROR;
+}
+
+
+enum winbindd_result winbindd_dual_pam_auth_crap(struct winbindd_domain *domain,
+                                                struct winbindd_cli_state *state) 
+{
+       NTSTATUS result;
+       const char *srv_name_slash;
         NET_USER_INFO_3 info3;
-        struct cli_state *cli = NULL;
-       TALLOC_CTX *mem_ctx = NULL;
-       char *name_user = NULL;
+       unsigned char *session_key;
+       struct rpc_pipe_client *pipe_cli;
+       DOM_CRED *credentials;
+       const char *name_user = NULL;
        const char *name_domain = NULL;
        const char *workstation;
        struct winbindd_domain *contact_domain;
@@ -378,44 +583,17 @@ enum winbindd_result winbindd_pam_auth_crap(struct winbindd_cli_state *state)
 
        DATA_BLOB lm_resp, nt_resp;
 
-       if (!state->privileged) {
-               char *error_string = NULL;
-               DEBUG(2, ("winbindd_pam_auth_crap: non-privileged access denied.  !\n"));
-               DEBUGADD(2, ("winbindd_pam_auth_crap: Ensure permissions on %s are set correctly.\n", 
-                            get_winbind_priv_pipe_dir()));
-               /* send a better message than ACCESS_DENIED */
-               asprintf(&error_string, "winbind client not authorized to use winbindd_pam_auth_crap.  Ensure permissions on %s are set correctly.",
-                        get_winbind_priv_pipe_dir());
-               push_utf8_fstring(state->response.data.auth.error_string, error_string);
-               SAFE_FREE(error_string);
-               result =  NT_STATUS_ACCESS_DENIED;
-               goto done;
-       }
+       /* This is child-only, so no check for privileged access is needed
+          anymore */
 
        /* Ensure null termination */
        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 (!(mem_ctx = talloc_init("winbind pam auth crap for (utf8) %s", state->request.data.auth_crap.user))) {
-               DEBUG(0, ("winbindd_pam_auth_crap: could not talloc_init()!\n"));
-               result = NT_STATUS_NO_MEMORY;
-               goto done;
-       }
-
-        if (pull_utf8_talloc(mem_ctx, &name_user, state->request.data.auth_crap.user) == (size_t)-1) {
-               DEBUG(0, ("winbindd_pam_auth_crap: pull_utf8_talloc failed!\n"));
-               result = NT_STATUS_UNSUCCESSFUL;
-               goto done;
-       }
+       name_user = state->request.data.auth_crap.user;
 
        if (*state->request.data.auth_crap.domain) {
-               char *dom = NULL;
-               if (pull_utf8_talloc(mem_ctx, &dom, state->request.data.auth_crap.domain) == (size_t)-1) {
-                       DEBUG(0, ("winbindd_pam_auth_crap: pull_utf8_talloc failed!\n"));
-                       result = NT_STATUS_UNSUCCESSFUL;
-                       goto done;
-               }
-               name_domain = dom;
+               name_domain = state->request.data.auth_crap.domain;
        } else if (lp_winbind_use_default_domain()) {
                name_domain = lp_workgroup();
        } else {
@@ -429,13 +607,7 @@ enum winbindd_result winbindd_pam_auth_crap(struct winbindd_cli_state *state)
                  name_domain, name_user));
           
        if (*state->request.data.auth_crap.workstation) {
-               char *wrk = NULL;
-               if (pull_utf8_talloc(mem_ctx, &wrk, state->request.data.auth_crap.workstation) == (size_t)-1) {
-                       DEBUG(0, ("winbindd_pam_auth_crap: pull_utf8_talloc failed!\n"));
-                       result = NT_STATUS_UNSUCCESSFUL;
-                       goto done;
-               }
-               workstation = wrk;
+               workstation = state->request.data.auth_crap.workstation;
        } else {
                workstation = global_myname();
        }
@@ -449,8 +621,8 @@ enum winbindd_result winbindd_pam_auth_crap(struct winbindd_cli_state *state)
                goto done;
        }
 
-       lm_resp = data_blob_talloc(mem_ctx, state->request.data.auth_crap.lm_resp, state->request.data.auth_crap.lm_resp_len);
-       nt_resp = data_blob_talloc(mem_ctx, state->request.data.auth_crap.nt_resp, state->request.data.auth_crap.nt_resp_len);
+       lm_resp = data_blob_talloc(state->mem_ctx, state->request.data.auth_crap.lm_resp, state->request.data.auth_crap.lm_resp_len);
+       nt_resp = data_blob_talloc(state->mem_ctx, state->request.data.auth_crap.nt_resp, state->request.data.auth_crap.nt_resp_len);
        
 
        /* what domain should we contact? */
@@ -470,26 +642,25 @@ enum winbindd_result winbindd_pam_auth_crap(struct winbindd_cli_state *state)
                        goto done;
                }
 
-               if (!(contact_domain = find_our_domain())) {
-                       DEBUG(1, ("Authenticatoin for [%s] -> [%s]\\[%s] in our domain failed - we can't find our domain!\n", 
-                                 state->request.data.auth_crap.user, name_domain, name_user)); 
-                       result = NT_STATUS_NO_SUCH_USER;
-                       goto done;
-               }
-       }
-               
-       if ( !get_trust_pw(contact_domain->name, trust_passwd, &last_change_time, &sec_channel_type) ) {
-               result = NT_STATUS_CANT_ACCESS_DOMAIN_INFO;
-               goto done;
+               contact_domain = find_our_domain();
        }
 
+       srv_name_slash = talloc_asprintf(state->mem_ctx, "\\\\%s",
+                                        contact_domain->dcname);
+       if (srv_name_slash == NULL) {
+               DEBUG(0, ("talloc_asprintf failed\n"));
+               return WINBINDD_ERROR;
+       }
+               
        do {
+               DOM_CRED clnt_creds;
                ZERO_STRUCT(info3);
                ZERO_STRUCT(ret_creds);
                retry = False;
 
-               /* Don't shut this down - it belongs to the connection cache code */
-               result = cm_get_netlogon_cli(contact_domain, trust_passwd, sec_channel_type, False, &cli);
+               result = cm_connect_netlogon(contact_domain, state->mem_ctx,
+                                            &pipe_cli, &session_key,
+                                            &credentials);
 
                if (!NT_STATUS_IS_OK(result)) {
                        DEBUG(3, ("could not open handle to NETLOGON pipe (error: %s)\n",
@@ -497,71 +668,86 @@ enum winbindd_result winbindd_pam_auth_crap(struct winbindd_cli_state *state)
                        goto done;
                }
 
-               result = cli_netlogon_sam_network_logon(cli, mem_ctx,
-                                                       &ret_creds,
-                                                       name_user, name_domain,
-                                                       workstation,
-                                                       state->request.data.auth_crap.chal, 
-                                                       lm_resp, nt_resp, 
-                                                       &info3);
+               credentials->timestamp.time = time(NULL);
+               memcpy(&clnt_creds, credentials, sizeof(clnt_creds));
+
+               /* Calculate the new credentials. */
+               cred_create(session_key, &credentials->challenge,
+                           clnt_creds.timestamp, &(clnt_creds.challenge));
+
+               result = rpccli_netlogon_sam_network_logon(pipe_cli,
+                                                          state->mem_ctx,
+                                                          srv_name_slash,
+                                                          &clnt_creds,
+                                                          &ret_creds,
+                                                          name_user,
+                                                          name_domain, 
+                                                          global_myname(),
+                                                          state->request.data.auth_crap.chal,
+                                                          lm_resp,
+                                                          nt_resp, &info3,
+                                                          session_key);
 
                attempts += 1;
 
-               /* We have to try a second time as cm_get_netlogon_cli
+               /* We have to try a second time as cm_connect_netlogon
                   might not yet have noticed that the DC has killed
                   our connection. */
 
-               if ( cli->fd == -1 ) {
+               if (NT_STATUS_EQUAL(result, NT_STATUS_UNSUCCESSFUL)) {
                        retry = True;
                        continue;
-               } 
+               }
 
                /* if we get access denied, a possible cause was that we had and open
                   connection to the DC, but someone changed our machine account password
                   out from underneath us using 'net rpc changetrustpw' */
                   
-               if ( NT_STATUS_V(result) == NT_STATUS_V(NT_STATUS_ACCESS_DENIED) ) {
-                       DEBUG(3,("winbindd_pam_auth_crap: sam_logon returned ACCESS_DENIED.  Maybe the trust account "
-                               "password was changed and we didn't know it.  Killing connections to domain %s\n",
-                               contact_domain->name));
-                       winbindd_cm_flush();
+               if ( NT_STATUS_EQUAL(result, NT_STATUS_ACCESS_DENIED) ) {
+                       DEBUG(3,("winbindd_pam_auth: sam_logon returned "
+                                "ACCESS_DENIED.  Maybe the trust account "
+                               "password was changed and we didn't know it. "
+                                "Killing connections to domain %s\n",
+                               name_domain));
+                       invalidate_cm_connection(&contact_domain->conn);
                        retry = True;
-                       cli = NULL;
                } 
-               
+
        } while ( (attempts < 2) && retry );
 
-       if (cli != NULL) {
-               /* We might have come out of the loop above with cli == NULL,
-                  so don't dereference that. */
-               clnt_deal_with_creds(cli->sess_key, &(cli->clnt_cred), &ret_creds);
+       if (NT_STATUS_IS_OK(result) &&
+           (!clnt_deal_with_creds(session_key, credentials,
+                                 &ret_creds))) {
+               DEBUG(3, ("DC %s sent wrong credentials\n",
+                         pipe_cli->cli->srv_name_slash));
+               result = NT_STATUS_ACCESS_DENIED;
        }
 
        if (NT_STATUS_IS_OK(result)) {
-               netsamlogon_cache_store( cli->mem_ctx, &info3 );
+               netsamlogon_cache_store( state->mem_ctx, name_user, &info3 );
                wcache_invalidate_samlogon(find_domain_from_name(name_domain), &info3);
                
-               if (!NT_STATUS_IS_OK(result = check_info3_in_group(mem_ctx, &info3, state->request.data.auth_crap.required_membership_sid))) {
+               if (!NT_STATUS_IS_OK(result = check_info3_in_group(state->mem_ctx, &info3, state->request.data.auth_crap.require_membership_of_sid))) {
                        DEBUG(3, ("User %s is not in the required group (%s), so plaintext authentication is rejected\n",
                                  state->request.data.auth_crap.user, 
-                                 state->request.data.auth_crap.required_membership_sid));
+                                 state->request.data.auth_crap.require_membership_of_sid));
                        goto done;
                }
 
                if (state->request.flags & WBFLAG_PAM_INFO3_NDR) {
-                       result = append_info3_as_ndr(mem_ctx, state, &info3);
+                       result = append_info3_as_ndr(state->mem_ctx, state, &info3);
                } else if (state->request.flags & WBFLAG_PAM_UNIX_NAME) {
                        /* ntlm_auth should return the unix username, per 
                           'winbind use default domain' settings and the like */
                        
                        fstring username_out;
                        const char *nt_username, *nt_domain;
-                       if (!(nt_username = unistr2_tdup(mem_ctx, &(info3.uni_user_name)))) {
+                       if (!(nt_username = unistr2_tdup(state->mem_ctx, &(info3.uni_user_name)))) {
                                /* If the server didn't give us one, just use the one we sent them */
                                nt_username = name_user;
                        }
                        
-                       if (!(nt_domain = unistr2_tdup(mem_ctx, &(info3.uni_logon_dom)))) {
+                       if (!(nt_domain = unistr2_tdup(state->mem_ctx, &(info3.uni_logon_dom)))) {
                                /* If the server didn't give us one, just use the one we sent them */
                                nt_domain = name_domain;
                        }
@@ -570,8 +756,8 @@ enum winbindd_result winbindd_pam_auth_crap(struct winbindd_cli_state *state)
 
                        DEBUG(5, ("Setting unix username to [%s]\n", username_out));
 
-                       /* this interface is in UTF8 */
-                       if (push_utf8_allocate((char **)&state->response.extra_data, username_out) == -1) {
+                       state->response.extra_data = SMB_STRDUP(username_out);
+                       if (!state->response.extra_data) {
                                result = NT_STATUS_NO_MEMORY;
                                goto done;
                        }
@@ -582,7 +768,7 @@ enum winbindd_result winbindd_pam_auth_crap(struct winbindd_cli_state *state)
                        memcpy(state->response.data.auth.user_session_key, info3.user_sess_key, sizeof(state->response.data.auth.user_session_key) /* 16 */);
                }
                if (state->request.flags & WBFLAG_PAM_LMKEY) {
-                       memcpy(state->response.data.auth.first_8_lm_hash, info3.padding, sizeof(state->response.data.auth.first_8_lm_hash) /* 8 */);
+                       memcpy(state->response.data.auth.first_8_lm_hash, info3.lm_sess_key, sizeof(state->response.data.auth.first_8_lm_hash) /* 8 */);
                }
        }
 
@@ -597,11 +783,11 @@ done:
        }
        
        state->response.data.auth.nt_status = NT_STATUS_V(result);
-       push_utf8_fstring(state->response.data.auth.nt_status_string, nt_errstr(result));
+       fstrcpy(state->response.data.auth.nt_status_string, nt_errstr(result));
        
        /* we might have given a more useful error above */
        if (!*state->response.data.auth.error_string) 
-               push_utf8_fstring(state->response.data.auth.error_string, get_friendly_nt_error_msg(result));
+               fstrcpy(state->response.data.auth.error_string, get_friendly_nt_error_msg(result));
        state->response.data.auth.pam_error = nt_status_to_pam(result);
 
        DEBUG(NT_STATUS_IS_OK(result) ? 5 : 2, 
@@ -611,9 +797,6 @@ done:
               state->response.data.auth.nt_status_string,
               state->response.data.auth.pam_error));         
 
-       if (mem_ctx) 
-               talloc_destroy(mem_ctx);
-       
        return NT_STATUS_IS_OK(result) ? WINBINDD_OK : WINBINDD_ERROR;
 }
 
@@ -624,20 +807,13 @@ enum winbindd_result winbindd_pam_chauthtok(struct winbindd_cli_state *state)
        NTSTATUS result;
        char *oldpass, *newpass;
        fstring domain, user;
-       CLI_POLICY_HND *hnd;
-       TALLOC_CTX *mem_ctx;
+       POLICY_HND dom_pol;
        struct winbindd_domain *contact_domain;
+       struct rpc_pipe_client *cli;
 
        DEBUG(3, ("[%5lu]: pam chauthtok %s\n", (unsigned long)state->pid,
                state->request.data.chauthtok.user));
 
-       if (!(mem_ctx = talloc_init("winbind password change for (utf8) %s", 
-                                   state->request.data.chauthtok.user))) {
-               DEBUG(0, ("winbindd_pam_auth_crap: could not talloc_init()!\n"));
-               result = NT_STATUS_NO_MEMORY;
-               goto done;
-       }
-
        /* Setup crap */
 
        if (state == NULL)
@@ -659,12 +835,15 @@ enum winbindd_result winbindd_pam_chauthtok(struct winbindd_cli_state *state)
 
        /* Get sam handle */
 
-       if (!NT_STATUS_IS_OK(result = cm_get_sam_handle(contact_domain, &hnd)) ) {
+       result = cm_connect_sam(contact_domain, state->mem_ctx, &cli,
+                               &dom_pol);
+       if (!NT_STATUS_IS_OK(result)) {
                DEBUG(1, ("could not get SAM handle on DC for %s\n", domain));
                goto done;
        }
 
-       result = cli_samr_chgpasswd_user(hnd->cli, mem_ctx, user, newpass, oldpass);
+       result = rpccli_samr_chgpasswd_user(cli, state->mem_ctx, user, newpass,
+                                           oldpass);
 
 done:    
        state->response.data.auth.nt_status = NT_STATUS_V(result);
@@ -679,8 +858,5 @@ done:
               state->response.data.auth.nt_status_string,
               state->response.data.auth.pam_error));         
 
-       if (mem_ctx)
-               talloc_destroy(mem_ctx);
-
        return NT_STATUS_IS_OK(result) ? WINBINDD_OK : WINBINDD_ERROR;
 }