s3-auth Rename NT_USER_TOKEN user_sids -> sids
[amitay/samba.git] / source3 / auth / auth_util.c
index 0e94fd9dd40658ee6e5f22f257c70de9cf56e51c..3b07bad6bf86d3cfa8abfe95e5837eb22754a79c 100644 (file)
 #include "includes.h"
 #include "smbd/globals.h"
 #include "../libcli/auth/libcli_auth.h"
+#include "../lib/crypto/arcfour.h"
+#include "rpc_client/init_lsa.h"
 
 #undef DBGC_CLASS
 #define DBGC_CLASS DBGC_AUTH
 
-/****************************************************************************
- Ensure primary group SID is always at position 0 in a 
- auth_serversupplied_info struct.
-****************************************************************************/
-
-static void sort_sid_array_for_smbd(struct auth_serversupplied_info *result,
-                               const DOM_SID *pgroup_sid)
-{
-       unsigned int i;
-
-       if (!result->sids) {
-               return;
-       }
-
-       if (sid_compare(&result->sids[0], pgroup_sid)==0) {
-               return;
-       }
-
-       for (i = 1; i < result->num_sids; i++) {
-               if (sid_compare(pgroup_sid,
-                               &result->sids[i]) == 0) {
-                       sid_copy(&result->sids[i], &result->sids[0]);
-                       sid_copy(&result->sids[0], pgroup_sid);
-                       return;
-               }
-       }
-}
-
 /****************************************************************************
  Create a UNIX user on demand.
 ****************************************************************************/
@@ -103,87 +77,6 @@ static int _smb_create_user(const char *domain, const char *unix_username, const
        return ret;
 }
 
-/****************************************************************************
- Create an auth_usersupplied_data structure
-****************************************************************************/
-
-static NTSTATUS make_user_info(struct auth_usersupplied_info **user_info,
-                               const char *smb_name,
-                               const char *internal_username,
-                               const char *client_domain,
-                               const char *domain,
-                               const char *wksta_name,
-                               DATA_BLOB *lm_pwd, DATA_BLOB *nt_pwd,
-                               DATA_BLOB *lm_interactive_pwd, DATA_BLOB *nt_interactive_pwd,
-                               DATA_BLOB *plaintext,
-                               bool encrypted)
-{
-
-       DEBUG(5,("attempting to make a user_info for %s (%s)\n", internal_username, smb_name));
-
-       *user_info = SMB_MALLOC_P(struct auth_usersupplied_info);
-       if (*user_info == NULL) {
-               DEBUG(0,("malloc failed for user_info (size %lu)\n", (unsigned long)sizeof(*user_info)));
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       ZERO_STRUCTP(*user_info);
-
-       DEBUG(5,("making strings for %s's user_info struct\n", internal_username));
-
-       (*user_info)->smb_name = SMB_STRDUP(smb_name);
-       if ((*user_info)->smb_name == NULL) { 
-               free_user_info(user_info);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       (*user_info)->internal_username = SMB_STRDUP(internal_username);
-       if ((*user_info)->internal_username == NULL) { 
-               free_user_info(user_info);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       (*user_info)->domain = SMB_STRDUP(domain);
-       if ((*user_info)->domain == NULL) { 
-               free_user_info(user_info);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       (*user_info)->client_domain = SMB_STRDUP(client_domain);
-       if ((*user_info)->client_domain == NULL) { 
-               free_user_info(user_info);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       (*user_info)->wksta_name = SMB_STRDUP(wksta_name);
-       if ((*user_info)->wksta_name == NULL) { 
-               free_user_info(user_info);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       DEBUG(5,("making blobs for %s's user_info struct\n", internal_username));
-
-       if (lm_pwd)
-               (*user_info)->lm_resp = data_blob(lm_pwd->data, lm_pwd->length);
-       if (nt_pwd)
-               (*user_info)->nt_resp = data_blob(nt_pwd->data, nt_pwd->length);
-       if (lm_interactive_pwd)
-               (*user_info)->lm_interactive_pwd = data_blob(lm_interactive_pwd->data, lm_interactive_pwd->length);
-       if (nt_interactive_pwd)
-               (*user_info)->nt_interactive_pwd = data_blob(nt_interactive_pwd->data, nt_interactive_pwd->length);
-
-       if (plaintext)
-               (*user_info)->plaintext_password = data_blob(plaintext->data, plaintext->length);
-
-       (*user_info)->encrypted = encrypted;
-
-       (*user_info)->logon_parameters = 0;
-
-       DEBUG(10,("made an %sencrypted user_info for %s (%s)\n", encrypted ? "":"un" , internal_username, smb_name));
-
-       return NT_STATUS_OK;
-}
-
 /****************************************************************************
  Create an auth_usersupplied_data structure after appropriate mapping.
 ****************************************************************************/
@@ -191,24 +84,23 @@ static NTSTATUS make_user_info(struct auth_usersupplied_info **user_info,
 NTSTATUS make_user_info_map(struct auth_usersupplied_info **user_info,
                            const char *smb_name,
                            const char *client_domain,
-                           const char *wksta_name,
+                           const char *workstation_name,
                            DATA_BLOB *lm_pwd,
                            DATA_BLOB *nt_pwd,
-                           DATA_BLOB *lm_interactive_pwd,
-                           DATA_BLOB *nt_interactive_pwd,
-                           DATA_BLOB *plaintext,
-                           bool encrypted)
+                           const struct samr_Password *lm_interactive_pwd,
+                           const struct samr_Password *nt_interactive_pwd,
+                           const char *plaintext,
+                           enum auth_password_state password_state)
 {
-       struct smbd_server_connection *sconn = smbd_server_conn;
        const char *domain;
        NTSTATUS result;
        bool was_mapped;
        fstring internal_username;
        fstrcpy(internal_username, smb_name);
-       was_mapped = map_username(sconn, internal_username);
+       was_mapped = map_username(internal_username);
 
        DEBUG(5, ("Mapping user [%s]\\[%s] from workstation [%s]\n",
-                client_domain, smb_name, wksta_name));
+                client_domain, smb_name, workstation_name));
 
        domain = client_domain;
 
@@ -229,7 +121,7 @@ NTSTATUS make_user_info_map(struct auth_usersupplied_info **user_info,
                        domain = get_global_sam_name();
                DEBUG(5, ("Mapped domain from [%s] to [%s] for user [%s] from "
                          "workstation [%s]\n",
-                         client_domain, domain, smb_name, wksta_name));
+                         client_domain, domain, smb_name, workstation_name));
        }
 
        /* We know that the given domain is trusted (and we are allowing them),
@@ -237,11 +129,14 @@ NTSTATUS make_user_info_map(struct auth_usersupplied_info **user_info,
         * primary domain name */
 
        result = make_user_info(user_info, smb_name, internal_username,
-                             client_domain, domain, wksta_name,
+                             client_domain, domain, workstation_name,
                              lm_pwd, nt_pwd,
                              lm_interactive_pwd, nt_interactive_pwd,
-                             plaintext, encrypted);
+                             plaintext, password_state);
        if (NT_STATUS_IS_OK(result)) {
+               /* We have tried mapping */
+               (*user_info)->mapped_state = True;
+               /* did we actually map the user to a different name? */
                (*user_info)->was_mapped = was_mapped;
        }
        return result;
@@ -255,7 +150,7 @@ NTSTATUS make_user_info_map(struct auth_usersupplied_info **user_info,
 bool make_user_info_netlogon_network(struct auth_usersupplied_info **user_info,
                                     const char *smb_name, 
                                     const char *client_domain, 
-                                    const char *wksta_name, 
+                                    const char *workstation_name,
                                     uint32 logon_parameters,
                                     const uchar *lm_network_pwd,
                                     int lm_pwd_len,
@@ -269,11 +164,11 @@ bool make_user_info_netlogon_network(struct auth_usersupplied_info **user_info,
 
        status = make_user_info_map(user_info,
                                    smb_name, client_domain, 
-                                   wksta_name, 
+                                   workstation_name,
                                    lm_pwd_len ? &lm_blob : NULL, 
                                    nt_pwd_len ? &nt_blob : NULL,
                                    NULL, NULL, NULL,
-                                   True);
+                                   AUTH_PASSWORD_RESPONSE);
 
        if (NT_STATUS_IS_OK(status)) {
                (*user_info)->logon_parameters = logon_parameters;
@@ -293,15 +188,15 @@ bool make_user_info_netlogon_network(struct auth_usersupplied_info **user_info,
 bool make_user_info_netlogon_interactive(struct auth_usersupplied_info **user_info,
                                         const char *smb_name, 
                                         const char *client_domain, 
-                                        const char *wksta_name, 
+                                        const char *workstation_name,
                                         uint32 logon_parameters,
                                         const uchar chal[8], 
                                         const uchar lm_interactive_pwd[16], 
                                         const uchar nt_interactive_pwd[16], 
                                         const uchar *dc_sess_key)
 {
-       unsigned char lm_pwd[16];
-       unsigned char nt_pwd[16];
+       struct samr_Password lm_pwd;
+       struct samr_Password nt_pwd;
        unsigned char local_lm_response[24];
        unsigned char local_nt_response[24];
        unsigned char key[16];
@@ -309,42 +204,42 @@ bool make_user_info_netlogon_interactive(struct auth_usersupplied_info **user_in
        memcpy(key, dc_sess_key, 16);
 
        if (lm_interactive_pwd)
-               memcpy(lm_pwd, lm_interactive_pwd, sizeof(lm_pwd));
+               memcpy(lm_pwd.hash, lm_interactive_pwd, sizeof(lm_pwd.hash));
 
        if (nt_interactive_pwd)
-               memcpy(nt_pwd, nt_interactive_pwd, sizeof(nt_pwd));
+               memcpy(nt_pwd.hash, nt_interactive_pwd, sizeof(nt_pwd.hash));
 
 #ifdef DEBUG_PASSWORD
        DEBUG(100,("key:"));
        dump_data(100, key, sizeof(key));
 
        DEBUG(100,("lm owf password:"));
-       dump_data(100, lm_pwd, sizeof(lm_pwd));
+       dump_data(100, lm_pwd.hash, sizeof(lm_pwd.hash));
 
        DEBUG(100,("nt owf password:"));
-       dump_data(100, nt_pwd, sizeof(nt_pwd));
+       dump_data(100, nt_pwd.hash, sizeof(nt_pwd.hash));
 #endif
 
        if (lm_interactive_pwd)
-               arcfour_crypt(lm_pwd, key, sizeof(lm_pwd));
+               arcfour_crypt(lm_pwd.hash, key, sizeof(lm_pwd.hash));
 
        if (nt_interactive_pwd)
-               arcfour_crypt(nt_pwd, key, sizeof(nt_pwd));
+               arcfour_crypt(nt_pwd.hash, key, sizeof(nt_pwd.hash));
 
 #ifdef DEBUG_PASSWORD
        DEBUG(100,("decrypt of lm owf password:"));
-       dump_data(100, lm_pwd, sizeof(lm_pwd));
+       dump_data(100, lm_pwd.hash, sizeof(lm_pwd));
 
        DEBUG(100,("decrypt of nt owf password:"));
-       dump_data(100, nt_pwd, sizeof(nt_pwd));
+       dump_data(100, nt_pwd.hash, sizeof(nt_pwd));
 #endif
 
        if (lm_interactive_pwd)
-               SMBOWFencrypt(lm_pwd, chal,
+               SMBOWFencrypt(lm_pwd.hash, chal,
                              local_lm_response);
 
        if (nt_interactive_pwd)
-               SMBOWFencrypt(nt_pwd, chal,
+               SMBOWFencrypt(nt_pwd.hash, chal,
                              local_nt_response);
 
        /* Password info paranoia */
@@ -356,33 +251,24 @@ bool make_user_info_netlogon_interactive(struct auth_usersupplied_info **user_in
                DATA_BLOB local_lm_blob;
                DATA_BLOB local_nt_blob;
 
-               DATA_BLOB lm_interactive_blob;
-               DATA_BLOB nt_interactive_blob;
-
                if (lm_interactive_pwd) {
                        local_lm_blob = data_blob(local_lm_response,
                                                  sizeof(local_lm_response));
-                       lm_interactive_blob = data_blob(lm_pwd,
-                                                       sizeof(lm_pwd));
-                       ZERO_STRUCT(lm_pwd);
                }
 
                if (nt_interactive_pwd) {
                        local_nt_blob = data_blob(local_nt_response,
                                                  sizeof(local_nt_response));
-                       nt_interactive_blob = data_blob(nt_pwd,
-                                                       sizeof(nt_pwd));
-                       ZERO_STRUCT(nt_pwd);
                }
 
                nt_status = make_user_info_map(
                        user_info, 
-                       smb_name, client_domain, wksta_name, 
+                       smb_name, client_domain, workstation_name,
                        lm_interactive_pwd ? &local_lm_blob : NULL,
                        nt_interactive_pwd ? &local_nt_blob : NULL,
-                       lm_interactive_pwd ? &lm_interactive_blob : NULL,
-                       nt_interactive_pwd ? &nt_interactive_blob : NULL,
-                       NULL, True);
+                       lm_interactive_pwd ? &lm_pwd : NULL,
+                       nt_interactive_pwd ? &nt_pwd : NULL,
+                       NULL, AUTH_PASSWORD_HASH);
 
                if (NT_STATUS_IS_OK(nt_status)) {
                        (*user_info)->logon_parameters = logon_parameters;
@@ -391,8 +277,6 @@ bool make_user_info_netlogon_interactive(struct auth_usersupplied_info **user_in
                ret = NT_STATUS_IS_OK(nt_status) ? True : False;
                data_blob_free(&local_lm_blob);
                data_blob_free(&local_nt_blob);
-               data_blob_free(&lm_interactive_blob);
-               data_blob_free(&nt_interactive_blob);
                return ret;
        }
 }
@@ -412,14 +296,13 @@ bool make_user_info_for_reply(struct auth_usersupplied_info **user_info,
        DATA_BLOB local_lm_blob;
        DATA_BLOB local_nt_blob;
        NTSTATUS ret = NT_STATUS_UNSUCCESSFUL;
-
+       char *plaintext_password_string;
        /*
         * Not encrypted - do so.
         */
 
        DEBUG(5,("make_user_info_for_reply: User passwords not in encrypted "
                 "format.\n"));
-
        if (plaintext_password.data && plaintext_password.length) {
                unsigned char local_lm_response[24];
 
@@ -442,14 +325,26 @@ bool make_user_info_for_reply(struct auth_usersupplied_info **user_info,
                local_nt_blob = data_blob_null; 
        }
 
+       plaintext_password_string = talloc_strndup(talloc_tos(),
+                                                  (const char *)plaintext_password.data,
+                                                  plaintext_password.length);
+       if (!plaintext_password_string) {
+               return False;
+       }
+
        ret = make_user_info_map(
                user_info, smb_name, client_domain, 
                get_remote_machine_name(),
                local_lm_blob.data ? &local_lm_blob : NULL,
                local_nt_blob.data ? &local_nt_blob : NULL,
                NULL, NULL,
-               plaintext_password.data && plaintext_password.length ? &plaintext_password : NULL, 
-               False);
+               plaintext_password_string,
+               AUTH_PASSWORD_PLAIN);
+
+       if (plaintext_password_string) {
+               memset(plaintext_password_string, '\0', strlen(plaintext_password_string));
+               talloc_free(plaintext_password_string);
+       }
 
        data_blob_free(&local_lm_blob);
        return NT_STATUS_IS_OK(ret) ? True : False;
@@ -467,10 +362,10 @@ NTSTATUS make_user_info_for_reply_enc(struct auth_usersupplied_info **user_info,
        return make_user_info_map(user_info, smb_name, 
                                  client_domain, 
                                  get_remote_machine_name(), 
-                                 lm_resp.data ? &lm_resp : NULL, 
-                                 nt_resp.data ? &nt_resp : NULL, 
+                                 lm_resp.data && (lm_resp.length > 0) ? &lm_resp : NULL,
+                                 nt_resp.data && (nt_resp.length > 0) ? &nt_resp : NULL,
                                  NULL, NULL, NULL,
-                                 True);
+                                 AUTH_PASSWORD_RESPONSE);
 }
 
 /****************************************************************************
@@ -488,7 +383,7 @@ bool make_user_info_guest(struct auth_usersupplied_info **user_info)
                                   NULL, NULL, 
                                   NULL, NULL, 
                                   NULL,
-                                  True);
+                                  AUTH_PASSWORD_RESPONSE);
 
        return NT_STATUS_IS_OK(nt_status) ? True : False;
 }
@@ -510,12 +405,12 @@ static NTSTATUS log_nt_token(NT_USER_TOKEN *token)
        for (i=1; i<token->num_sids; i++) {
                group_sidstr = talloc_asprintf(
                        frame, "%s %s", group_sidstr,
-                       sid_string_talloc(frame, &token->user_sids[i]));
+                       sid_string_talloc(frame, &token->sids[i]));
        }
 
        command = talloc_string_sub(
                frame, lp_log_nt_token_command(),
-               "%s", sid_string_talloc(frame, &token->user_sids[0]));
+               "%s", sid_string_talloc(frame, &token->sids[0]));
        command = talloc_string_sub(frame, command, "%t", group_sidstr);
 
        if (command == NULL) {
@@ -535,7 +430,7 @@ static NTSTATUS log_nt_token(NT_USER_TOKEN *token)
 }
 
 /*
- * Create the token to use from server_info->sam_account and
+ * Create the token to use from server_info->info3 and
  * server_info->sids (the info3/sam groups). Find the unix gids.
  */
 
@@ -562,13 +457,11 @@ NTSTATUS create_local_token(struct auth_serversupplied_info *server_info)
                                                    &server_info->ptok);
 
        } else {
-               server_info->ptok = create_local_nt_token(
-                       server_info,
-                       pdb_get_user_sid(server_info->sam_account),
-                       server_info->guest,
-                       server_info->num_sids, server_info->sids);
-               status = server_info->ptok ?
-                       NT_STATUS_OK : NT_STATUS_NO_SUCH_USER;
+               status = create_local_nt_token_from_info3(server_info,
+                                                         server_info->guest,
+                                                         server_info->info3,
+                                                         &server_info->extra,
+                                                         &server_info->ptok);
        }
 
        if (!NT_STATUS_IS_OK(status)) {
@@ -584,7 +477,7 @@ NTSTATUS create_local_token(struct auth_serversupplied_info *server_info)
 
        for (i=1; i<server_info->ptok->num_sids; i++) {
                gid_t gid;
-               DOM_SID *sid = &server_info->ptok->user_sids[i];
+               struct dom_sid *sid = &server_info->ptok->sids[i];
 
                if (!sid_to_gid(sid, &gid)) {
                        DEBUG(10, ("Could not convert SID %s to gid, "
@@ -609,22 +502,16 @@ NTSTATUS create_local_token(struct auth_serversupplied_info *server_info)
         * the nt token.
         */
 
-       if (!uid_to_unix_users_sid(server_info->utok.uid, &tmp_sid)) {
-               DEBUG(1,("create_local_token: Failed to create SID "
-                       "for uid %u!\n", (unsigned int)server_info->utok.uid));
-       }
+       uid_to_unix_users_sid(server_info->utok.uid, &tmp_sid);
+
        add_sid_to_array_unique(server_info->ptok, &tmp_sid,
-                               &server_info->ptok->user_sids,
+                               &server_info->ptok->sids,
                                &server_info->ptok->num_sids);
 
        for ( i=0; i<server_info->utok.ngroups; i++ ) {
-               if (!gid_to_unix_groups_sid( server_info->utok.groups[i], &tmp_sid ) ) {
-                       DEBUG(1,("create_local_token: Failed to create SID "
-                               "for gid %u!\n", (unsigned int)server_info->utok.groups[i]));
-                       continue;
-               }
+               gid_to_unix_groups_sid(server_info->utok.groups[i], &tmp_sid);
                add_sid_to_array_unique(server_info->ptok, &tmp_sid,
-                                       &server_info->ptok->user_sids,
+                                       &server_info->ptok->sids,
                                        &server_info->ptok->num_sids);
        }
 
@@ -650,54 +537,12 @@ NTSTATUS make_server_info_pw(struct auth_serversupplied_info **server_info,
 {
        NTSTATUS status;
        struct samu *sampass = NULL;
-       gid_t *gids;
        char *qualified_name = NULL;
        TALLOC_CTX *mem_ctx = NULL;
-       DOM_SID u_sid;
+       struct dom_sid u_sid;
        enum lsa_SidType type;
        struct auth_serversupplied_info *result;
 
-       if ( !(sampass = samu_new( NULL )) ) {
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       status = samu_set_unix( sampass, pwd );
-       if (!NT_STATUS_IS_OK(status)) {
-               return status;
-       }
-
-       result = make_server_info(NULL);
-       if (result == NULL) {
-               TALLOC_FREE(sampass);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       result->sam_account = sampass;
-
-       result->unix_name = talloc_strdup(result, unix_username);
-       result->sanitized_username = sanitize_username(result, unix_username);
-
-       if ((result->unix_name == NULL)
-           || (result->sanitized_username == NULL)) {
-               TALLOC_FREE(sampass);
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       result->utok.uid = pwd->pw_uid;
-       result->utok.gid = pwd->pw_gid;
-
-       status = pdb_enum_group_memberships(result, sampass,
-                                           &result->sids, &gids,
-                                           &result->num_sids);
-
-       if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(10, ("pdb_enum_group_memberships failed: %s\n",
-                          nt_errstr(status)));
-               TALLOC_FREE(result);
-               return status;
-       }
-
        /*
         * The SID returned in server_info->sam_account is based
         * on our SAM sid even though for a pure UNIX account this should
@@ -713,7 +558,6 @@ NTSTATUS make_server_info_pw(struct auth_serversupplied_info **server_info,
 
        mem_ctx = talloc_init("make_server_info_pw_tmp");
        if (!mem_ctx) {
-               TALLOC_FREE(result);
                return NT_STATUS_NO_MEMORY;
        }
 
@@ -721,7 +565,6 @@ NTSTATUS make_server_info_pw(struct auth_serversupplied_info **server_info,
                                        unix_users_domain_name(),
                                        unix_username );
        if (!qualified_name) {
-               TALLOC_FREE(result);
                TALLOC_FREE(mem_ctx);
                return NT_STATUS_NO_MEMORY;
        }
@@ -729,7 +572,6 @@ NTSTATUS make_server_info_pw(struct auth_serversupplied_info **server_info,
        if (!lookup_name(mem_ctx, qualified_name, LOOKUP_NAME_ALL,
                                                NULL, NULL,
                                                &u_sid, &type)) {
-               TALLOC_FREE(result);
                TALLOC_FREE(mem_ctx);
                return NT_STATUS_NO_SUCH_USER;
        }
@@ -737,28 +579,107 @@ NTSTATUS make_server_info_pw(struct auth_serversupplied_info **server_info,
        TALLOC_FREE(mem_ctx);
 
        if (type != SID_NAME_USER) {
-               TALLOC_FREE(result);
                return NT_STATUS_NO_SUCH_USER;
        }
 
-       status = add_sid_to_array_unique(result, &u_sid,
-                                        &result->sids,
-                                        &result->num_sids);
+       if ( !(sampass = samu_new( NULL )) ) {
+               return NT_STATUS_NO_MEMORY;
+       }
+
+       status = samu_set_unix( sampass, pwd );
+       if (!NT_STATUS_IS_OK(status)) {
+               return status;
+       }
+
+       /* In pathological cases the above call can set the account
+        * name to the DOMAIN\username form. Reset the account name
+        * using unix_username */
+       pdb_set_username(sampass, unix_username, PDB_SET);
+
+       /* set the user sid to be the calculated u_sid */
+       pdb_set_user_sid(sampass, &u_sid, PDB_SET);
+
+       result = make_server_info(NULL);
+       if (result == NULL) {
+               TALLOC_FREE(sampass);
+               return NT_STATUS_NO_MEMORY;
+       }
+
+       status = samu_to_SamInfo3(result, sampass, global_myname(),
+                                 &result->info3, &result->extra);
+       TALLOC_FREE(sampass);
        if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(10, ("Failed to convert samu to info3: %s\n",
+                          nt_errstr(status)));
                TALLOC_FREE(result);
                return status;
        }
 
-       /* For now we throw away the gids and convert via sid_to_gid
-        * later. This needs fixing, but I'd like to get the code straight and
-        * simple first. */
-       TALLOC_FREE(gids);
+       result->unix_name = talloc_strdup(result, unix_username);
+       result->sanitized_username = sanitize_username(result, unix_username);
+
+       if ((result->unix_name == NULL)
+           || (result->sanitized_username == NULL)) {
+               TALLOC_FREE(result);
+               return NT_STATUS_NO_MEMORY;
+       }
+
+       result->utok.uid = pwd->pw_uid;
+       result->utok.gid = pwd->pw_gid;
 
        *server_info = result;
 
        return NT_STATUS_OK;
 }
 
+static NTSTATUS get_guest_info3(TALLOC_CTX *mem_ctx,
+                               struct netr_SamInfo3 *info3)
+{
+       const char *guest_account = lp_guestaccount();
+       struct dom_sid domain_sid;
+       struct passwd *pwd;
+       const char *tmp;
+       NTSTATUS status;
+
+       pwd = getpwnam_alloc(mem_ctx, guest_account);
+       if (pwd == NULL) {
+               DEBUG(0,("SamInfo3_for_guest: Unable to locate guest "
+                        "account [%s]!\n", guest_account));
+               return NT_STATUS_NO_SUCH_USER;
+       }
+
+       /* Set acount name */
+       tmp = talloc_strdup(mem_ctx, pwd->pw_name);
+       if (tmp == NULL) {
+               return NT_STATUS_NO_MEMORY;
+       }
+       init_lsa_String(&info3->base.account_name, tmp);
+
+       /* Set domain name */
+       tmp = talloc_strdup(mem_ctx, get_global_sam_name());
+       if (tmp == NULL) {
+               return NT_STATUS_NO_MEMORY;
+       }
+       init_lsa_StringLarge(&info3->base.domain, tmp);
+
+       /* Domain sid */
+       sid_copy(&domain_sid, get_global_sam_sid());
+
+       info3->base.domain_sid = sid_dup_talloc(mem_ctx, &domain_sid);
+       if (info3->base.domain_sid == NULL) {
+               return NT_STATUS_NO_MEMORY;
+       }
+
+       /* Guest rid */
+       info3->base.rid = DOMAIN_RID_GUEST;
+
+       /* Primary gid */
+       info3->base.primary_gid = BUILTIN_RID_GUESTS;
+
+       TALLOC_FREE(pwd);
+       return status;
+}
+
 /***************************************************************************
  Make (and fill) a user_info struct for a guest login.
  This *must* succeed for smbd to start. If there is no mapping entry for
@@ -767,32 +688,33 @@ NTSTATUS make_server_info_pw(struct auth_serversupplied_info **server_info,
 
 static NTSTATUS make_new_server_info_guest(struct auth_serversupplied_info **server_info)
 {
+       static const char zeros[16] = {0};
+       const char *guest_account = lp_guestaccount();
+       const char *domain = global_myname();
+       struct netr_SamInfo3 info3;
+       TALLOC_CTX *tmp_ctx;
        NTSTATUS status;
-       struct samu *sampass = NULL;
-       DOM_SID guest_sid;
-       bool ret;
-       static const char zeros[16] = {0, };
        fstring tmp;
 
-       if ( !(sampass = samu_new( NULL )) ) {
+       tmp_ctx = talloc_stackframe();
+       if (tmp_ctx == NULL) {
                return NT_STATUS_NO_MEMORY;
        }
 
-       sid_compose(&guest_sid, get_global_sam_sid(), DOMAIN_USER_RID_GUEST);
+       ZERO_STRUCT(info3);
 
-       become_root();
-       ret = pdb_getsampwsid(sampass, &guest_sid);
-       unbecome_root();
-
-       if (!ret) {
-               TALLOC_FREE(sampass);
-               return NT_STATUS_NO_SUCH_USER;
+       status = get_guest_info3(tmp_ctx, &info3);
+       if (!NT_STATUS_IS_OK(status)) {
+               goto done;
        }
 
-       status = make_server_info_sam(server_info, sampass);
+       status = make_server_info_info3(tmp_ctx,
+                                       guest_account,
+                                       domain,
+                                       server_info,
+                                       &info3);
        if (!NT_STATUS_IS_OK(status)) {
-               TALLOC_FREE(sampass);
-               return status;
+               goto done;
        }
 
        (*server_info)->guest = True;
@@ -801,7 +723,7 @@ static NTSTATUS make_new_server_info_guest(struct auth_serversupplied_info **ser
        if (!NT_STATUS_IS_OK(status)) {
                DEBUG(10, ("create_local_token failed: %s\n",
                           nt_errstr(status)));
-               return status;
+               goto done;
        }
 
        /* annoying, but the Guest really does have a session key, and it is
@@ -809,9 +731,42 @@ static NTSTATUS make_new_server_info_guest(struct auth_serversupplied_info **ser
        (*server_info)->user_session_key = data_blob(zeros, sizeof(zeros));
        (*server_info)->lm_session_key = data_blob(zeros, sizeof(zeros));
 
-       alpha_strcpy(tmp, pdb_get_username(sampass), ". _-$", sizeof(tmp));
+       alpha_strcpy(tmp, (*server_info)->info3->base.account_name.string,
+                    ". _-$", sizeof(tmp));
        (*server_info)->sanitized_username = talloc_strdup(*server_info, tmp);
 
+       status = NT_STATUS_OK;
+done:
+       TALLOC_FREE(tmp_ctx);
+       return NT_STATUS_OK;
+}
+
+/***************************************************************************
+ Make (and fill) a user_info struct for a system user login.
+ This *must* succeed for smbd to start.
+***************************************************************************/
+
+static NTSTATUS make_new_server_info_system(TALLOC_CTX *mem_ctx,
+                                           struct auth_serversupplied_info **server_info)
+{
+       struct passwd *pwd;
+       NTSTATUS status;
+
+       pwd = getpwuid_alloc(mem_ctx, sec_initial_uid());
+       if (pwd == NULL) {
+               return NT_STATUS_NO_MEMORY;
+       }
+
+       status = make_serverinfo_from_username(mem_ctx,
+                                            pwd->pw_name,
+                                            false,
+                                            server_info);
+       if (!NT_STATUS_IS_OK(status)) {
+               return status;
+       }
+
+       (*server_info)->system = true;
+
        return NT_STATUS_OK;
 }
 
@@ -867,6 +822,7 @@ struct auth_serversupplied_info *copy_serverinfo(TALLOC_CTX *mem_ctx,
        }
 
        dst->guest = src->guest;
+       dst->system = src->system;
        dst->utok.uid = src->utok.uid;
        dst->utok.gid = src->utok.gid;
        dst->utok.ngroups = src->utok.ngroups;
@@ -892,16 +848,12 @@ struct auth_serversupplied_info *copy_serverinfo(TALLOC_CTX *mem_ctx,
        dst->lm_session_key = data_blob_talloc(dst, src->lm_session_key.data,
                                                src->lm_session_key.length);
 
-       dst->sam_account = samu_new(NULL);
-       if (!dst->sam_account) {
-               TALLOC_FREE(dst);
-               return NULL;
-       }
-
-       if (!pdb_copy_sam_account(dst->sam_account, src->sam_account)) {
+       dst->info3 = copy_netr_SamInfo3(dst, src->info3);
+       if (!dst->info3) {
                TALLOC_FREE(dst);
                return NULL;
        }
+       dst->extra = src->extra;
 
        dst->pam_handle = NULL;
        dst->unix_name = talloc_strdup(dst, src->unix_name);
@@ -952,6 +904,24 @@ NTSTATUS make_server_info_guest(TALLOC_CTX *mem_ctx,
        return (*server_info != NULL) ? NT_STATUS_OK : NT_STATUS_NO_MEMORY;
 }
 
+static struct auth_serversupplied_info *system_info = NULL;
+
+bool init_system_info(void)
+{
+       if (system_info != NULL)
+               return True;
+
+       return NT_STATUS_IS_OK(make_new_server_info_system(talloc_autofree_context(), &system_info));
+}
+
+NTSTATUS make_server_info_system(TALLOC_CTX *mem_ctx,
+                               struct auth_serversupplied_info **server_info)
+{
+       if (system_info == NULL) return NT_STATUS_UNSUCCESSFUL;
+       *server_info = copy_serverinfo(mem_ctx, system_info);
+       return (*server_info != NULL) ? NT_STATUS_OK : NT_STATUS_NO_MEMORY;
+}
+
 bool copy_current_user(struct current_user *dst, struct current_user *src)
 {
        gid_t *groups;
@@ -981,18 +951,13 @@ bool copy_current_user(struct current_user *dst, struct current_user *src)
 
 /***************************************************************************
  Purely internal function for make_server_info_info3
- Fill the sam account from getpwnam
 ***************************************************************************/
-static NTSTATUS fill_sam_account(TALLOC_CTX *mem_ctx, 
-                                const char *domain,
-                                const char *username,
-                                char **found_username,
-                                uid_t *uid, gid_t *gid,
-                                struct samu *account,
-                                bool *username_was_mapped)
+
+static NTSTATUS check_account(TALLOC_CTX *mem_ctx, const char *domain,
+                             const char *username, char **found_username,
+                             struct passwd **pwd,
+                             bool *username_was_mapped)
 {
-       struct smbd_server_connection *sconn = smbd_server_conn;
-       NTSTATUS nt_status;
        fstring dom_user, lower_username;
        fstring real_username;
        struct passwd *passwd;
@@ -1003,15 +968,18 @@ static NTSTATUS fill_sam_account(TALLOC_CTX *mem_ctx,
        fstr_sprintf(dom_user, "%s%c%s", domain, *lp_winbind_separator(), 
                lower_username);
 
-       /* Get the passwd struct.  Try to create the account is necessary. */
+       /* Get the passwd struct.  Try to create the account if necessary. */
 
-       *username_was_mapped = map_username(sconn, dom_user);
+       *username_was_mapped = map_username(dom_user);
 
-       if ( !(passwd = smb_getpwnam( NULL, dom_user, real_username, True )) )
+       passwd = smb_getpwnam( NULL, dom_user, real_username, True );
+       if (!passwd) {
+               DEBUG(3, ("Failed to find authenticated user %s via "
+                         "getpwnam(), denying access.\n", dom_user));
                return NT_STATUS_NO_SUCH_USER;
+       }
 
-       *uid = passwd->pw_uid;
-       *gid = passwd->pw_gid;
+       *pwd = passwd;
 
        /* This is pointless -- there is no suport for differing 
           unix and windows names.  Make sure to always store the 
@@ -1021,13 +989,7 @@ static NTSTATUS fill_sam_account(TALLOC_CTX *mem_ctx,
 
        *found_username = talloc_strdup( mem_ctx, real_username );
 
-       DEBUG(5,("fill_sam_account: located username was [%s]\n", *found_username));
-
-       nt_status = samu_set_unix( account, passwd );
-
-       TALLOC_FREE(passwd);
-
-       return nt_status;
+       return NT_STATUS_OK;
 }
 
 /****************************************************************************
@@ -1036,7 +998,7 @@ static NTSTATUS fill_sam_account(TALLOC_CTX *mem_ctx,
  the username if we fallback to the username only.
  ****************************************************************************/
 
-struct passwd *smb_getpwnam( TALLOC_CTX *mem_ctx, char *domuser,
+struct passwd *smb_getpwnam( TALLOC_CTX *mem_ctx, const char *domuser,
                             fstring save_username, bool create )
 {
        struct passwd *pw = NULL;
@@ -1131,15 +1093,11 @@ NTSTATUS make_server_info_info3(TALLOC_CTX *mem_ctx,
        char *found_username = NULL;
        const char *nt_domain;
        const char *nt_username;
-       struct samu *sam_account = NULL;
-       DOM_SID user_sid;
-       DOM_SID group_sid;
        bool username_was_mapped;
-
-       uid_t uid = (uid_t)-1;
-       gid_t gid = (gid_t)-1;
-
+       struct passwd *pwd;
        struct auth_serversupplied_info *result;
+       struct dom_sid *group_sid;
+       struct netr_SamInfo3 *i3;
 
        /* 
           Here is where we should check the list of
@@ -1147,15 +1105,6 @@ NTSTATUS make_server_info_info3(TALLOC_CTX *mem_ctx,
           matches.
        */
 
-       if (!sid_compose(&user_sid, info3->base.domain_sid, info3->base.rid)) {
-               return NT_STATUS_INVALID_PARAMETER;
-       }
-
-       if (!sid_compose(&group_sid, info3->base.domain_sid,
-                        info3->base.primary_gid)) {
-               return NT_STATUS_INVALID_PARAMETER;
-       }
-
        nt_username = talloc_strdup(mem_ctx, info3->base.account_name.string);
        if (!nt_username) {
                /* If the server didn't give us one, just use the one we sent
@@ -1170,8 +1119,7 @@ NTSTATUS make_server_info_info3(TALLOC_CTX *mem_ctx,
                nt_domain = domain;
        }
 
-       /* try to fill the SAM account..  If getpwnam() fails, then try the 
-          add user script (2.2.x behavior).
+       /* If getpwnam() fails try the add user script (2.2.x behavior).
 
           We use the _unmapped_ username here in an attempt to provide
           consistent username mapping behavior between kerberos and NTLM[SSP]
@@ -1182,129 +1130,22 @@ NTSTATUS make_server_info_info3(TALLOC_CTX *mem_ctx,
           that is how the current code is designed.  Making the change here
           is the least disruptive place.  -- jerry */
 
-       if ( !(sam_account = samu_new( NULL )) ) {
-               return NT_STATUS_NO_MEMORY;
-       }
-
        /* this call will try to create the user if necessary */
 
-       nt_status = fill_sam_account(mem_ctx, nt_domain, sent_nt_username,
-                                    &found_username, &uid, &gid, sam_account,
+       nt_status = check_account(mem_ctx, nt_domain, sent_nt_username,
+                                    &found_username, &pwd,
                                     &username_was_mapped);
 
-
-       /* if we still don't have a valid unix account check for 
-         'map to guest = bad uid' */
-
        if (!NT_STATUS_IS_OK(nt_status)) {
-               TALLOC_FREE( sam_account );
-               if ( lp_map_to_guest() == MAP_TO_GUEST_ON_BAD_UID ) {
-                       make_server_info_guest(NULL, server_info);
-                       return NT_STATUS_OK;
-               }
                return nt_status;
        }
 
-       if (!pdb_set_nt_username(sam_account, nt_username, PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_username(sam_account, nt_username, PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_domain(sam_account, nt_domain, PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_user_sid(sam_account, &user_sid, PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_UNSUCCESSFUL;
-       }
-
-       if (!pdb_set_group_sid(sam_account, &group_sid, PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_UNSUCCESSFUL;
-       }
-
-       if (!pdb_set_fullname(sam_account,
-                             info3->base.full_name.string,
-                             PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_logon_script(sam_account,
-                                 info3->base.logon_script.string,
-                                 PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_profile_path(sam_account,
-                                 info3->base.profile_path.string,
-                                 PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_homedir(sam_account,
-                            info3->base.home_directory.string,
-                            PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_dir_drive(sam_account,
-                              info3->base.home_drive.string,
-                              PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_acct_ctrl(sam_account, info3->base.acct_flags, PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_pass_last_set_time(
-                   sam_account,
-                   nt_time_to_unix(info3->base.last_password_change),
-                   PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_pass_can_change_time(
-                   sam_account,
-                   nt_time_to_unix(info3->base.allow_password_change),
-                   PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_pass_must_change_time(
-                   sam_account,
-                   nt_time_to_unix(info3->base.force_password_change),
-                   PDB_CHANGED)) {
-               TALLOC_FREE(sam_account);
-               return NT_STATUS_NO_MEMORY;
-       }
-
        result = make_server_info(NULL);
        if (result == NULL) {
                DEBUG(4, ("make_server_info failed!\n"));
-               TALLOC_FREE(sam_account);
                return NT_STATUS_NO_MEMORY;
        }
 
-       /* save this here to _net_sam_logon() doesn't fail (it assumes a 
-          valid struct samu) */
-
-       result->sam_account = sam_account;
        result->unix_name = talloc_strdup(result, found_username);
 
        result->sanitized_username = sanitize_username(result,
@@ -1314,30 +1155,43 @@ NTSTATUS make_server_info_info3(TALLOC_CTX *mem_ctx,
                return NT_STATUS_NO_MEMORY;
        }
 
-       /* Fill in the unix info we found on the way */
-
-       result->utok.uid = uid;
-       result->utok.gid = gid;
-
-       /* Create a 'combined' list of all SIDs we might want in the SD */
+       /* copy in the info3 */
+       result->info3 = i3 = copy_netr_SamInfo3(result, info3);
+       if (result->info3 == NULL) {
+               TALLOC_FREE(result);
+               return NT_STATUS_NO_MEMORY;
+       }
 
-       result->num_sids = 0;
-       result->sids = NULL;
+       /* Fill in the unix info we found on the way */
+       result->utok.uid = pwd->pw_uid;
+       result->utok.gid = pwd->pw_gid;
 
-       nt_status = sid_array_from_info3(result, info3,
-                                        &result->sids,
-                                        &result->num_sids,
-                                        false, false);
+       /* We can't just trust that the primary group sid sent us is something
+        * we can really use. Obtain the useable sid, and store the original
+        * one as an additional group if it had to be replaced */
+       nt_status = get_primary_group_sid(mem_ctx, found_username,
+                                         &pwd, &group_sid);
        if (!NT_STATUS_IS_OK(nt_status)) {
                TALLOC_FREE(result);
                return nt_status;
        }
 
-       /* Ensure the primary group sid is at position 0. */
-       sort_sid_array_for_smbd(result, &group_sid);
-
-       result->login_server = talloc_strdup(result,
-                                            info3->base.logon_server.string);
+       /* store and check if it is the same we got originally */
+       sid_peek_rid(group_sid, &i3->base.primary_gid);
+       if (i3->base.primary_gid != info3->base.primary_gid) {
+               uint32_t n = i3->base.groups.count;
+               /* not the same, store the original as an additional group */
+               i3->base.groups.rids =
+                       talloc_realloc(i3, i3->base.groups.rids,
+                                       struct samr_RidWithAttribute, n + 1);
+               if (i3->base.groups.rids == NULL) {
+                       TALLOC_FREE(result);
+                       return NT_STATUS_NO_MEMORY;
+               }
+               i3->base.groups.rids[n].rid = info3->base.primary_gid;
+               i3->base.groups.rids[n].attributes = SE_GROUP_ENABLED;
+               i3->base.groups.count = n + 1;
+       }
 
        /* ensure we are never given NULL session keys */
 
@@ -1374,264 +1228,16 @@ NTSTATUS make_server_info_wbcAuthUserInfo(TALLOC_CTX *mem_ctx,
                                          const struct wbcAuthUserInfo *info,
                                          struct auth_serversupplied_info **server_info)
 {
-       static const char zeros[16] = {0, };
-
-       NTSTATUS nt_status = NT_STATUS_OK;
-       char *found_username = NULL;
-       const char *nt_domain;
-       const char *nt_username;
-       struct samu *sam_account = NULL;
-       DOM_SID user_sid;
-       DOM_SID group_sid;
-       bool username_was_mapped;
-       uint32_t i;
-
-       uid_t uid = (uid_t)-1;
-       gid_t gid = (gid_t)-1;
-
-       struct auth_serversupplied_info *result;
-
-       result = make_server_info(NULL);
-       if (result == NULL) {
-               DEBUG(4, ("make_server_info failed!\n"));
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       /*
-          Here is where we should check the list of
-          trusted domains, and verify that the SID
-          matches.
-       */
-
-       memcpy(&user_sid, &info->sids[0].sid, sizeof(user_sid));
-       memcpy(&group_sid, &info->sids[1].sid, sizeof(group_sid));
-
-       if (info->account_name) {
-               nt_username = talloc_strdup(result, info->account_name);
-       } else {
-               /* If the server didn't give us one, just use the one we sent
-                * them */
-               nt_username = talloc_strdup(result, sent_nt_username);
-       }
-       if (!nt_username) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (info->domain_name) {
-               nt_domain = talloc_strdup(result, info->domain_name);
-       } else {
-               /* If the server didn't give us one, just use the one we sent
-                * them */
-               nt_domain = talloc_strdup(result, domain);
-       }
-       if (!nt_domain) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       /* try to fill the SAM account..  If getpwnam() fails, then try the
-          add user script (2.2.x behavior).
-
-          We use the _unmapped_ username here in an attempt to provide
-          consistent username mapping behavior between kerberos and NTLM[SSP]
-          authentication in domain mode security.  I.E. Username mapping
-          should be applied to the fully qualified username
-          (e.g. DOMAIN\user) and not just the login name.  Yes this means we
-          called map_username() unnecessarily in make_user_info_map() but
-          that is how the current code is designed.  Making the change here
-          is the least disruptive place.  -- jerry */
-
-       if ( !(sam_account = samu_new( result )) ) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       /* this call will try to create the user if necessary */
-
-       nt_status = fill_sam_account(result, nt_domain, sent_nt_username,
-                                    &found_username, &uid, &gid, sam_account,
-                                    &username_was_mapped);
-
-       /* if we still don't have a valid unix account check for
-         'map to guest = bad uid' */
-
-       if (!NT_STATUS_IS_OK(nt_status)) {
-               TALLOC_FREE( result );
-               if ( lp_map_to_guest() == MAP_TO_GUEST_ON_BAD_UID ) {
-                       make_server_info_guest(NULL, server_info);
-                       return NT_STATUS_OK;
-               }
-               return nt_status;
-       }
-
-       if (!pdb_set_nt_username(sam_account, nt_username, PDB_CHANGED)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_username(sam_account, nt_username, PDB_CHANGED)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_domain(sam_account, nt_domain, PDB_CHANGED)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
+       struct netr_SamInfo3 *info3;
 
-       if (!pdb_set_user_sid(sam_account, &user_sid, PDB_CHANGED)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_UNSUCCESSFUL;
-       }
-
-       if (!pdb_set_group_sid(sam_account, &group_sid, PDB_CHANGED)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_UNSUCCESSFUL;
-       }
-
-       if (!pdb_set_fullname(sam_account, info->full_name, PDB_CHANGED)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_logon_script(sam_account, info->logon_script, PDB_CHANGED)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_profile_path(sam_account, info->profile_path, PDB_CHANGED)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_homedir(sam_account, info->home_directory, PDB_CHANGED)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_dir_drive(sam_account, info->home_drive, PDB_CHANGED)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_acct_ctrl(sam_account, info->acct_flags, PDB_CHANGED)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_pass_last_set_time(
-                   sam_account,
-                   nt_time_to_unix(info->pass_last_set_time),
-                   PDB_CHANGED)) {
-               TALLOC_FREE(result);
+       info3 = wbcAuthUserInfo_to_netr_SamInfo3(mem_ctx, info);
+       if (!info3) {
                return NT_STATUS_NO_MEMORY;
        }
 
-       if (!pdb_set_pass_can_change_time(
-                   sam_account,
-                   nt_time_to_unix(info->pass_can_change_time),
-                   PDB_CHANGED)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       if (!pdb_set_pass_must_change_time(
-                   sam_account,
-                   nt_time_to_unix(info->pass_must_change_time),
-                   PDB_CHANGED)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       /* save this here to _net_sam_logon() doesn't fail (it assumes a
-          valid struct samu) */
-
-       result->sam_account = sam_account;
-       result->unix_name = talloc_strdup(result, found_username);
-
-       result->sanitized_username = sanitize_username(result,
-                                                      result->unix_name);
-       result->login_server = talloc_strdup(result, info->logon_server);
-
-       if ((result->unix_name == NULL)
-           || (result->sanitized_username == NULL)
-           || (result->login_server == NULL)) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       /* Fill in the unix info we found on the way */
-
-       result->utok.uid = uid;
-       result->utok.gid = gid;
-
-       /* Create a 'combined' list of all SIDs we might want in the SD */
-
-       result->num_sids = info->num_sids - 2;
-       result->sids = talloc_array(result, DOM_SID, result->num_sids);
-       if (result->sids == NULL) {
-               TALLOC_FREE(result);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       for (i=0; i < result->num_sids; i++) {
-               memcpy(&result->sids[i], &info->sids[i+2].sid, sizeof(result->sids[i]));
-       }
-
-       /* Ensure the primary group sid is at position 0. */
-       sort_sid_array_for_smbd(result, &group_sid);
-
-       /* ensure we are never given NULL session keys */
-
-       if (memcmp(info->user_session_key, zeros, sizeof(zeros)) == 0) {
-               result->user_session_key = data_blob_null;
-       } else {
-               result->user_session_key = data_blob_talloc(
-                       result, info->user_session_key,
-                       sizeof(info->user_session_key));
-       }
-
-       if (memcmp(info->lm_session_key, zeros, 8) == 0) {
-               result->lm_session_key = data_blob_null;
-       } else {
-               result->lm_session_key = data_blob_talloc(
-                       result, info->lm_session_key,
-                       sizeof(info->lm_session_key));
-       }
-
-       result->nss_token |= username_was_mapped;
-
-       *server_info = result;
-
-       return NT_STATUS_OK;
-}
-
-/***************************************************************************
- Free a user_info struct
-***************************************************************************/
-
-void free_user_info(struct auth_usersupplied_info **user_info)
-{
-       DEBUG(5,("attempting to free (and zero) a user_info structure\n"));
-       if (*user_info != NULL) {
-               if ((*user_info)->smb_name) {
-                       DEBUG(10,("structure was created for %s\n",
-                                 (*user_info)->smb_name));
-               }
-               SAFE_FREE((*user_info)->smb_name);
-               SAFE_FREE((*user_info)->internal_username);
-               SAFE_FREE((*user_info)->client_domain);
-               SAFE_FREE((*user_info)->domain);
-               SAFE_FREE((*user_info)->wksta_name);
-               data_blob_free(&(*user_info)->lm_resp);
-               data_blob_free(&(*user_info)->nt_resp);
-               data_blob_clear_free(&(*user_info)->lm_interactive_pwd);
-               data_blob_clear_free(&(*user_info)->nt_interactive_pwd);
-               data_blob_clear_free(&(*user_info)->plaintext_password);
-               ZERO_STRUCT(**user_info);
-       }
-       SAFE_FREE(*user_info);
+       return make_server_info_info3(mem_ctx,
+                                     sent_nt_username, domain,
+                                     server_info, info3);
 }
 
 /**
@@ -1644,7 +1250,7 @@ void free_user_info(struct auth_usersupplied_info **user_info)
 
 bool is_trusted_domain(const char* dom_name)
 {
-       DOM_SID trustdom_sid;
+       struct dom_sid trustdom_sid;
        bool ret;
 
        /* no trusted domains for a standalone server */