This is the 'multiple pdb backends' patch from ctrlsoft, aka Jelmer Vernooij
[kai/samba.git] / source3 / passdb / pdb_tdb.c
index a6c40eb9706c9b6125cd242f8dc1238db710887e..7092caa15ef557fce936719fad331111cfcca37e 100644 (file)
@@ -1,9 +1,11 @@
 /*
- * Unix SMB/Netbios implementation. Version 1.9. SMB parameters and setup
+ * Unix SMB/CIFS implementation. 
+ * SMB parameters and setup
  * Copyright (C) Andrew Tridgell 1992-1998
  * Copyright (C) Simo Sorce 2000
  * Copyright (C) Gerald Carter 2000
  * Copyright (C) Jeremy Allison 2001
+ * Copyright (C) Andrew Bartlett 2002
  * 
  * This program is free software; you can redistribute it and/or modify it under
  * the terms of the GNU General Public License as published by the Free
 #ifdef WITH_TDB_SAM
 
 #define PDB_VERSION            "20010830"
-#define PASSDB_FILE_NAME       "/passdb.tdb"
+#define PASSDB_FILE_NAME       "passdb.tdb"
 #define TDB_FORMAT_STRING      "ddddddBBBBBBBBBBBBddBBwdwdBdd"
 #define USERPREFIX             "USER_"
 #define RIDPREFIX              "RID_"
 
-extern int             DEBUGLEVEL;
-extern pstring                 samlogon_user;
-extern BOOL            sam_logon_in_ssb;
-
-struct tdb_enum_info {
+struct tdbsam_privates {
        TDB_CONTEXT     *passwd_tdb;
        TDB_DATA        key;
-};
 
-static struct tdb_enum_info    global_tdb_ent;
-/*static SAM_ACCOUNT           global_sam_pass;*/
+       /* retrive-once info */
+       const char *tdbsam_location;
+
+       BOOL permit_non_unix_accounts;
+
+/*     uint32 low_nua_rid; 
+       uint32 high_nua_rid; */
+};
 
 /**********************************************************************
  Intialize a SAM_ACCOUNT struct from a BYTE buffer of size len
  *********************************************************************/
 
-static BOOL init_sam_from_buffer (SAM_ACCOUNT *sampass, uint8 *buf, uint32 buflen)
+static BOOL init_sam_from_buffer (struct tdbsam_privates *tdb_state,
+                                 SAM_ACCOUNT *sampass, uint8 *buf, uint32 buflen)
 {
 
        /* times are stored as 32bit integer
@@ -75,13 +79,23 @@ static BOOL init_sam_from_buffer (SAM_ACCOUNT *sampass, uint8 *buf, uint32 bufle
                fullname_len, homedir_len, logon_script_len,
                profile_path_len, acct_desc_len, workstations_len;
                
-       uint32  /* uid, gid,*/ user_rid, group_rid, unknown_3, hours_len, unknown_5, unknown_6;
+       uint32  user_rid, group_rid, unknown_3, hours_len, unknown_5, unknown_6;
        uint16  acct_ctrl, logon_divs;
        uint8   *hours;
        static uint8    *lm_pw_ptr, *nt_pw_ptr;
        uint32          len = 0;
        uint32          lmpwlen, ntpwlen, hourslen;
        BOOL ret = True;
+       BOOL setflag;
+       pstring sub_buffer;
+       struct passwd *pw;
+       uid_t uid;
+       gid_t gid = -1; /* This is what standard sub advanced expects if no gid is known */
+       
+       if(sampass == NULL || buf == NULL) {
+               DEBUG(0, ("init_sam_from_buffer: NULL parameters found!\n"));
+               return False;
+       }
                                                                        
        /* unpack the buffer into variables */
        len = tdb_unpack (buf, buflen, TDB_FORMAT_STRING,
@@ -120,29 +134,97 @@ static BOOL init_sam_from_buffer (SAM_ACCOUNT *sampass, uint8 *buf, uint32 bufle
                goto done;
        }
 
-       pdb_set_logon_time(sampass, logon_time);
-       pdb_set_logoff_time(sampass, logoff_time);
-       pdb_set_kickoff_time(sampass, kickoff_time);
-       pdb_set_pass_can_change_time(sampass, pass_can_change_time);
-       pdb_set_pass_must_change_time(sampass, pass_must_change_time);
+       /* validate the account and fill in UNIX uid and gid. Standard
+        * getpwnam() is used instead of Get_Pwnam() as we do not need
+        * to try case permutations
+        */
+       if (!username || !(pw = getpwnam_alloc(username))) {
+               if (!(tdb_state->permit_non_unix_accounts)) {
+                       DEBUG(0,("tdbsam: getpwnam_alloc(%s) return NULL.  User does not exist!\n", username));
+                       ret = False;
+                       goto done;
+               }
+       }
+               
+       if (pw) {
+               uid = pw->pw_uid;
+               gid = pw->pw_gid;
+               
+               passwd_free(&pw);
+
+               pdb_set_uid(sampass, uid);
+               pdb_set_gid(sampass, gid);
+       }
+
+       pdb_set_logon_time(sampass, logon_time, True);
+       pdb_set_logoff_time(sampass, logoff_time, True);
+       pdb_set_kickoff_time(sampass, kickoff_time, True);
+       pdb_set_pass_can_change_time(sampass, pass_can_change_time, True);
+       pdb_set_pass_must_change_time(sampass, pass_must_change_time, True);
        pdb_set_pass_last_set_time(sampass, pass_last_set_time);
 
-       pdb_set_username     (sampass, username_len?username:NULL);
-       pdb_set_domain       (sampass, domain_len?domain:NULL);
-       pdb_set_nt_username  (sampass, nt_username_len?nt_username:NULL);
-       pdb_set_fullname     (sampass, fullname_len?fullname:NULL);
-       pdb_set_homedir      (sampass, homedir_len?homedir:NULL);
-       pdb_set_dir_drive    (sampass, dir_drive_len?dir_drive:NULL);
-       pdb_set_logon_script (sampass, logon_script_len?logon_script:NULL);
-       pdb_set_profile_path (sampass, profile_path_len?profile_path:NULL);
-       pdb_set_acct_desc    (sampass, acct_desc_len?acct_desc:NULL);
-       pdb_set_workstations (sampass, workstations_len?workstations:NULL);
-       pdb_set_munged_dial  (sampass, munged_dial_len?munged_dial:NULL);
-       pdb_set_lanman_passwd(sampass, lmpwlen?lm_pw_ptr:NULL);
-       pdb_set_nt_passwd    (sampass, ntpwlen?nt_pw_ptr:NULL);
-
-       /*pdb_set_uid(sampass, uid);
-       pdb_set_gid(sampass, gid);*/
+       pdb_set_username     (sampass, username);
+       pdb_set_domain       (sampass, domain);
+       pdb_set_nt_username  (sampass, nt_username);
+       pdb_set_fullname     (sampass, fullname);
+
+       if (homedir) setflag = True;
+       else {
+               setflag = False;
+               pstrcpy(sub_buffer, lp_logon_home());
+               /* standard_sub_advanced() assumes pstring is passed!! */
+               standard_sub_advanced(-1, username, "", gid, username, sub_buffer);
+               homedir = strdup(sub_buffer);
+               if(!homedir) { ret = False; goto done; }
+               DEBUG(5,("Home directory set back to %s\n", homedir));
+       }
+       pdb_set_homedir(sampass, homedir, setflag);
+
+       if (dir_drive) setflag = True;
+       else {
+               setflag = False;
+               pstrcpy(sub_buffer, lp_logon_drive());
+               standard_sub_advanced(-1, username, "", gid, username, sub_buffer);
+               dir_drive = strdup(sub_buffer);
+               if(!dir_drive) { ret = False; goto done; }
+               DEBUG(5,("Drive set back to %s\n", dir_drive));
+       }
+       pdb_set_dir_drive(sampass, dir_drive, setflag);
+
+       if (logon_script) setflag = True;
+       else {
+               setflag = False;
+               pstrcpy(sub_buffer, lp_logon_script());
+               standard_sub_advanced(-1, username, "", gid, username, sub_buffer);
+               logon_script = strdup(sub_buffer);
+               if(!logon_script) { ret = False; goto done; }
+               DEBUG(5,("Logon script set back to %s\n", logon_script));
+       }
+       pdb_set_logon_script(sampass, logon_script, setflag);
+
+       if (profile_path) setflag = True;
+       else {
+               setflag = False;
+               pstrcpy(sub_buffer, lp_logon_path());
+               standard_sub_advanced(-1, username, "", gid, username, sub_buffer);
+               profile_path = strdup(sub_buffer);
+               if(!profile_path) { ret = False; goto done; }
+               DEBUG(5,("Profile path set back to %s\n", profile_path));
+       }
+       pdb_set_profile_path(sampass, profile_path, setflag);
+
+       pdb_set_acct_desc    (sampass, acct_desc);
+       pdb_set_workstations (sampass, workstations);
+       pdb_set_munged_dial  (sampass, munged_dial);
+       if (!pdb_set_lanman_passwd(sampass, lm_pw_ptr)) {
+               ret = False;
+               goto done;
+       }
+       if (!pdb_set_nt_passwd(sampass, nt_pw_ptr)) {
+               ret = False;
+               goto done;
+       }
+
        pdb_set_user_rid(sampass, user_rid);
        pdb_set_group_rid(sampass, group_rid);
        pdb_set_unknown_3(sampass, unknown_3);
@@ -150,7 +232,7 @@ static BOOL init_sam_from_buffer (SAM_ACCOUNT *sampass, uint8 *buf, uint32 bufle
        pdb_set_unknown_5(sampass, unknown_5);
        pdb_set_unknown_6(sampass, unknown_6);
        pdb_set_acct_ctrl(sampass, acct_ctrl);
-       pdb_set_logons_divs(sampass, logon_divs);
+       pdb_set_logon_divs(sampass, logon_divs);
        pdb_set_hours(sampass, hours);
 
 done:
@@ -173,7 +255,8 @@ done:
 /**********************************************************************
  Intialize a BYTE buffer from a SAM_ACCOUNT struct
  *********************************************************************/
-static uint32 init_buffer_from_sam (uint8 **buf, const SAM_ACCOUNT *sampass)
+static uint32 init_buffer_from_sam (struct tdbsam_privates *tdb_state,
+                                   uint8 **buf, const SAM_ACCOUNT *sampass)
 {
        size_t          len, buflen;
 
@@ -186,6 +269,9 @@ static uint32 init_buffer_from_sam (uint8 **buf, const SAM_ACCOUNT *sampass)
                pass_last_set_time,
                pass_can_change_time,
                pass_must_change_time;
+
+       uint32  user_rid, group_rid;
+
        const char *username;
        const char *domain;
        const char *nt_username;
@@ -209,9 +295,11 @@ static uint32 init_buffer_from_sam (uint8 **buf, const SAM_ACCOUNT *sampass)
        uint32  nt_pw_len = 16;
 
        /* do we have a valid SAM_ACCOUNT pointer? */
-       if (sampass == NULL)
+       if (sampass == NULL) {
+               DEBUG(0, ("init_buffer_from_sam: SAM_ACCOUNT is NULL!\n"));
                return -1;
-               
+       }
+       
        *buf = NULL;
        buflen = 0;
 
@@ -222,73 +310,69 @@ static uint32 init_buffer_from_sam (uint8 **buf, const SAM_ACCOUNT *sampass)
        pass_must_change_time = (uint32)pdb_get_pass_must_change_time(sampass);
        pass_last_set_time = (uint32)pdb_get_pass_last_set_time(sampass);
 
+       user_rid = pdb_get_user_rid(sampass);
+       group_rid = pdb_get_group_rid(sampass);
 
        username = pdb_get_username(sampass);
-       if (username)
-               username_len = strlen(username) +1;
-       else
-               username_len = 0;
+       if (username) username_len = strlen(username) +1;
+       else username_len = 0;
+
        domain = pdb_get_domain(sampass);
-       if (domain)
-               domain_len = strlen(domain) +1;
-       else
-               domain_len = 0;
+       if (domain) domain_len = strlen(domain) +1;
+       else domain_len = 0;
+
        nt_username = pdb_get_nt_username(sampass);
-       if (nt_username)
-               nt_username_len = strlen(nt_username) +1;
-       else
-               nt_username_len = 0;
-       dir_drive = pdb_get_dirdrive(sampass);
-       if (dir_drive)
-               dir_drive_len = strlen(dir_drive) +1;
-       else
-               dir_drive_len = 0;
-       unknown_str = NULL;
-       unknown_str_len = 0;
-       munged_dial = pdb_get_munged_dial(sampass);
-       if (munged_dial)
-               munged_dial_len = strlen(munged_dial) +1;
-       else
-               munged_dial_len = 0;
-               
+       if (nt_username) nt_username_len = strlen(nt_username) +1;
+       else nt_username_len = 0;
+
        fullname = pdb_get_fullname(sampass);
-       if (fullname)
-               fullname_len = strlen(fullname) +1;
-       else
-               fullname_len = 0;
-       homedir = pdb_get_homedir(sampass);
-       if (homedir)
-               homedir_len = strlen(homedir) +1;
-       else
-               homedir_len = 0;
-       logon_script = pdb_get_logon_script(sampass);
-       if (logon_script)
-               logon_script_len = strlen(logon_script) +1;
-       else
-               logon_script_len = 0;
-       profile_path = pdb_get_profile_path(sampass);
-       if (profile_path)
-               profile_path_len = strlen(profile_path) +1;
-       else
-               profile_path_len = 0;
-       acct_desc = pdb_get_acct_desc(sampass);
-       if (acct_desc)
-               acct_desc_len = strlen(acct_desc) +1;
-       else
-               acct_desc_len = 0;
-       workstations = pdb_get_workstations(sampass);
-       if (workstations)
-               workstations_len = strlen(workstations) +1;
-       else
-               workstations_len = 0;
+       if (fullname) fullname_len = strlen(fullname) +1;
+       else fullname_len = 0;
+
+       /*
+        * Only updates fields which have been set (not defaults from smb.conf)
+        */
+
+       if (IS_SAM_SET(sampass, FLAG_SAM_DRIVE)) dir_drive = pdb_get_dirdrive(sampass);
+       else dir_drive = NULL;
+       if (dir_drive) dir_drive_len = strlen(dir_drive) +1;
+       else dir_drive_len = 0;
+
+       if (IS_SAM_SET(sampass, FLAG_SAM_SMBHOME)) homedir = pdb_get_homedir(sampass);
+       else homedir = NULL;
+       if (homedir) homedir_len = strlen(homedir) +1;
+       else homedir_len = 0;
+
+       if (IS_SAM_SET(sampass, FLAG_SAM_LOGONSCRIPT)) logon_script = pdb_get_logon_script(sampass);
+       else logon_script = NULL;
+       if (logon_script) logon_script_len = strlen(logon_script) +1;
+       else logon_script_len = 0;
+
+       if (IS_SAM_SET(sampass, FLAG_SAM_PROFILE)) profile_path = pdb_get_profile_path(sampass);
+       else profile_path = NULL;
+       if (profile_path) profile_path_len = strlen(profile_path) +1;
+       else profile_path_len = 0;
        
        lm_pw = pdb_get_lanman_passwd(sampass);
-       if (!lm_pw)
-               lm_pw_len = 0;
+       if (!lm_pw) lm_pw_len = 0;
        
        nt_pw = pdb_get_nt_passwd(sampass);
-       if (!nt_pw)
-               nt_pw_len = 0;
+       if (!nt_pw) nt_pw_len = 0;
+               
+       acct_desc = pdb_get_acct_desc(sampass);
+       if (acct_desc) acct_desc_len = strlen(acct_desc) +1;
+       else acct_desc_len = 0;
+
+       workstations = pdb_get_workstations(sampass);
+       if (workstations) workstations_len = strlen(workstations) +1;
+       else workstations_len = 0;
+
+       unknown_str = NULL;
+       unknown_str_len = 0;
+
+       munged_dial = pdb_get_munged_dial(sampass);
+       if (munged_dial) munged_dial_len = strlen(munged_dial) +1;
+       else munged_dial_len = 0;       
                
        /* one time to get the size needed */
        len = tdb_pack(NULL, 0,  TDB_FORMAT_STRING,
@@ -310,8 +394,8 @@ static uint32 init_buffer_from_sam (uint8 **buf, const SAM_ACCOUNT *sampass)
                workstations_len, workstations,
                unknown_str_len, unknown_str,
                munged_dial_len, munged_dial,
-               pdb_get_user_rid(sampass),
-               pdb_get_group_rid(sampass),
+               user_rid,
+               group_rid,
                lm_pw_len, lm_pw,
                nt_pw_len, nt_pw,
                pdb_get_acct_ctrl(sampass),
@@ -349,8 +433,8 @@ static uint32 init_buffer_from_sam (uint8 **buf, const SAM_ACCOUNT *sampass)
                workstations_len, workstations,
                unknown_str_len, unknown_str,
                munged_dial_len, munged_dial,
-               pdb_get_user_rid(sampass),
-               pdb_get_group_rid(sampass),
+               user_rid,
+               group_rid,
                lm_pw_len, lm_pw,
                nt_pw_len, nt_pw,
                pdb_get_acct_ctrl(sampass),
@@ -364,6 +448,8 @@ static uint32 init_buffer_from_sam (uint8 **buf, const SAM_ACCOUNT *sampass)
        
        /* check to make sure we got it correct */
        if (buflen != len) {
+               DEBUG(0, ("init_buffer_from_sam: somthing odd is going on here: bufflen (%d) != len (%d) in tdb_pack operations!\n", 
+                         buflen, len));  
                /* error */
                SAFE_FREE (*buf);
                return (-1);
@@ -376,98 +462,86 @@ static uint32 init_buffer_from_sam (uint8 **buf, const SAM_ACCOUNT *sampass)
  Open the TDB passwd database for SAM account enumeration.
 ****************************************************************/
 
-BOOL pdb_setsampwent(BOOL update)
+static BOOL tdbsam_setsampwent(struct pdb_methods *my_methods, BOOL update)
 {
-       pstring         tdbfile;
-       
-       get_private_directory(tdbfile);
-       pstrcat (tdbfile, PASSDB_FILE_NAME);
+       struct tdbsam_privates *tdb_state = (struct tdbsam_privates *)my_methods->private_data;
        
        /* Open tdb passwd */
-       if (!(global_tdb_ent.passwd_tdb = tdb_open_log(tdbfile, 0, TDB_DEFAULT, update?(O_RDWR|O_CREAT):O_RDONLY, 0600)))
+       if (!(tdb_state->passwd_tdb = tdb_open_log(tdb_state->tdbsam_location, 0, TDB_DEFAULT, update?(O_RDWR|O_CREAT):O_RDONLY, 0600)))
        {
                DEBUG(0, ("Unable to open/create TDB passwd\n"));
                return False;
        }
        
-       global_tdb_ent.key = tdb_firstkey(global_tdb_ent.passwd_tdb);
+       tdb_state->key = tdb_firstkey(tdb_state->passwd_tdb);
 
        return True;
 }
 
+static void close_tdb(struct tdbsam_privates *tdb_state) 
+{
+       if (tdb_state->passwd_tdb) {
+               tdb_close(tdb_state->passwd_tdb);
+               tdb_state->passwd_tdb = NULL;
+       }
+}
+
 /***************************************************************
  End enumeration of the TDB passwd list.
 ****************************************************************/
 
-void pdb_endsampwent(void)
+static void tdbsam_endsampwent(struct pdb_methods *my_methods)
 {
-       if (global_tdb_ent.passwd_tdb) {
-               tdb_close(global_tdb_ent.passwd_tdb);
-               global_tdb_ent.passwd_tdb = NULL;
-       }
+       struct tdbsam_privates *tdb_state = (struct tdbsam_privates *)my_methods->private_data;
+       close_tdb(tdb_state);
        
-       DEBUG(7, ("endtdbpwent: closed password file.\n"));
+       DEBUG(7, ("endtdbpwent: closed sam database.\n"));
 }
 
 /*****************************************************************
  Get one SAM_ACCOUNT from the TDB (next in line)
 *****************************************************************/
 
-BOOL pdb_getsampwent(SAM_ACCOUNT *user)
+static BOOL tdbsam_getsampwent(struct pdb_methods *my_methods, SAM_ACCOUNT *user)
 {
+       struct tdbsam_privates *tdb_state = (struct tdbsam_privates *)my_methods->private_data;
        TDB_DATA        data;
-       struct passwd   *pw;
-       uid_t           uid;
-       gid_t           gid;
        char *prefix = USERPREFIX;
        int  prefixlen = strlen (prefix);
 
+
        if (user==NULL) {
                DEBUG(0,("pdb_get_sampwent: SAM_ACCOUNT is NULL.\n"));
                return False;
        }
 
-       /* skip all RID entries */
-       while ((global_tdb_ent.key.dsize != 0) && (strncmp (global_tdb_ent.key.dptr, prefix, prefixlen)))
+       /* skip all non-USER entries (eg. RIDs) */
+       while ((tdb_state->key.dsize != 0) && (strncmp(tdb_state->key.dptr, prefix, prefixlen)))
                /* increment to next in line */
-               global_tdb_ent.key = tdb_nextkey (global_tdb_ent.passwd_tdb, global_tdb_ent.key);
+               tdb_state->key = tdb_nextkey(tdb_state->passwd_tdb, tdb_state->key);
 
        /* do we have an valid interation pointer? */
-       if(global_tdb_ent.passwd_tdb == NULL) {
+       if(tdb_state->passwd_tdb == NULL) {
                DEBUG(0,("pdb_get_sampwent: Bad TDB Context pointer.\n"));
                return False;
        }
 
-       data = tdb_fetch (global_tdb_ent.passwd_tdb, global_tdb_ent.key);
+       data = tdb_fetch(tdb_state->passwd_tdb, tdb_state->key);
        if (!data.dptr) {
                DEBUG(5,("pdb_getsampwent: database entry not found.\n"));
                return False;
        }
   
        /* unpack the buffer */
-       if (!init_sam_from_buffer (user, data.dptr, data.dsize)) {
+       if (!init_sam_from_buffer(tdb_state, user, data.dptr, data.dsize)) {
                DEBUG(0,("pdb_getsampwent: Bad SAM_ACCOUNT entry returned from TDB!\n"));
                SAFE_FREE(data.dptr);
                return False;
        }
        SAFE_FREE(data.dptr);
        
-       /* validate the account and fill in UNIX uid and gid.  sys_getpwnam()
-          is used instaed of Get_Pwnam() as we do not need to try case
-          permutations */
-       if ((pw=sys_getpwnam(pdb_get_username(user))) == NULL) {
-               DEBUG(0,("pdb_getsampwent: getpwnam(%s) return NULL.  User does not exist!\n", 
-                         pdb_get_username(user)));
-               return False;
-       }
-
-       uid = pw->pw_uid;
-       gid = pw->pw_gid;
-       pdb_set_uid (user, &uid);
-       pdb_set_gid (user, &gid);
-
        /* increment to next in line */
-       global_tdb_ent.key = tdb_nextkey (global_tdb_ent.passwd_tdb, global_tdb_ent.key);
+       tdb_state->key = tdb_nextkey(tdb_state->passwd_tdb, tdb_state->key);
 
        return True;
 }
@@ -476,17 +550,13 @@ BOOL pdb_getsampwent(SAM_ACCOUNT *user)
  Lookup a name in the SAM TDB
 ******************************************************************/
 
-BOOL pdb_getsampwnam (SAM_ACCOUNT *user, const char *sname)
+static BOOL tdbsam_getsampwnam (struct pdb_methods *my_methods, SAM_ACCOUNT *user, const char *sname)
 {
+       struct tdbsam_privates *tdb_state = (struct tdbsam_privates *)my_methods->private_data;
        TDB_CONTEXT     *pwd_tdb;
        TDB_DATA        data, key;
        fstring         keystr;
-       struct passwd   *pw;
-       pstring         tdbfile;
        fstring         name;
-       uid_t           uid;
-       gid_t           gid;
-
 
        if (user==NULL) {
                DEBUG(0,("pdb_getsampwnam: SAM_ACCOUNT is NULL.\n"));
@@ -496,92 +566,52 @@ BOOL pdb_getsampwnam (SAM_ACCOUNT *user, const char *sname)
        /* Data is stored in all lower-case */
        unix_strlower(sname, -1, name, sizeof(name));
 
-       get_private_directory(tdbfile);
-       pstrcat (tdbfile, PASSDB_FILE_NAME);
-       
        /* set search key */
        slprintf(keystr, sizeof(keystr)-1, "%s%s", USERPREFIX, name);
        key.dptr = keystr;
-       key.dsize = strlen (keystr) + 1;
+       key.dsize = strlen(keystr) + 1;
 
        /* open the accounts TDB */
-       if (!(pwd_tdb = tdb_open_log(tdbfile, 0, TDB_DEFAULT, O_RDONLY, 0600))) {
-               DEBUG(0, ("pdb_getsampwnam: Unable to open TDB passwd!\n"));
+       if (!(pwd_tdb = tdb_open_log(tdb_state->tdbsam_location, 0, TDB_DEFAULT, O_RDONLY, 0600))) {
+               DEBUG(0, ("pdb_getsampwnam: Unable to open TDB passwd (%s)!\n", tdb_state->tdbsam_location));
                return False;
        }
 
        /* get the record */
-       data = tdb_fetch (pwd_tdb, key);
+       data = tdb_fetch(pwd_tdb, key);
        if (!data.dptr) {
                DEBUG(5,("pdb_getsampwnam (TDB): error fetching database.\n"));
                DEBUGADD(5, (" Error: %s\n", tdb_errorstr(pwd_tdb)));
-               tdb_close (pwd_tdb);
+               DEBUGADD(5, (" Key: %s\n", keystr));
+               tdb_close(pwd_tdb);
                return False;
        }
   
        /* unpack the buffer */
-       if (!init_sam_from_buffer (user, data.dptr, data.dsize)) {
+       if (!init_sam_from_buffer(tdb_state, user, data.dptr, data.dsize)) {
                DEBUG(0,("pdb_getsampwent: Bad SAM_ACCOUNT entry returned from TDB!\n"));
                SAFE_FREE(data.dptr);
+               tdb_close(pwd_tdb);
                return False;
        }
        SAFE_FREE(data.dptr);
-       
-       /* validate the account and fill in UNIX uid and gid.  sys_getpwnam()
-          is used instaed of Get_Pwnam() as we do not need to try case
-          permutations */
-       if ((pw=sys_getpwnam(pdb_get_username(user))) == NULL) {
-               DEBUG(0,("pdb_getsampwent: getpwnam(%s) return NULL.  User does not exist!\n", 
-                         pdb_get_username(user)));
-               return False;
-       }
-       
-       uid = pw->pw_uid;
-       gid = pw->pw_gid;
-       pdb_set_uid (user, &uid);
-       pdb_set_gid (user, &gid);
-       
-       /* cleanup */
-       tdb_close (pwd_tdb);
 
+       /* no further use for database, close it now */
+       tdb_close(pwd_tdb);
+       
        return True;
 }
 
-/***************************************************************************
- Search by uid
- **************************************************************************/
-
-BOOL pdb_getsampwuid (SAM_ACCOUNT* user, uid_t uid)
-{
-       struct passwd   *pw;
-       fstring         name;
-
-       if (user==NULL) {
-               DEBUG(0,("pdb_getsampwuid: SAM_ACCOUNT is NULL.\n"));
-               return False;
-       }
-
-       pw = sys_getpwuid(uid);
-       if (pw == NULL) {
-               DEBUG(0,("pdb_getsampwuid: getpwuid(%d) return NULL. User does not exist!\n", uid));
-               return False;
-       }
-       fstrcpy (name, pw->pw_name);
-
-       return pdb_getsampwnam (user, name);
-
-}
-
 /***************************************************************************
  Search by rid
  **************************************************************************/
 
-BOOL pdb_getsampwrid (SAM_ACCOUNT *user, uint32 rid)
+static BOOL tdbsam_getsampwrid (struct pdb_methods *my_methods, SAM_ACCOUNT *user, uint32 rid)
 {
+       struct tdbsam_privates *tdb_state = (struct tdbsam_privates *)my_methods->private_data;
        TDB_CONTEXT             *pwd_tdb;
        TDB_DATA                data, key;
        fstring                 keystr;
-       pstring                 tdbfile;
        fstring                 name;
        
        if (user==NULL) {
@@ -589,16 +619,13 @@ BOOL pdb_getsampwrid (SAM_ACCOUNT *user, uint32 rid)
                return False;
        }
 
-       get_private_directory(tdbfile);
-       pstrcat (tdbfile, PASSDB_FILE_NAME);
-       
        /* set search key */
        slprintf(keystr, sizeof(keystr)-1, "%s%.8x", RIDPREFIX, rid);
        key.dptr = keystr;
        key.dsize = strlen (keystr) + 1;
 
        /* open the accounts TDB */
-       if (!(pwd_tdb = tdb_open_log(tdbfile, 0, TDB_DEFAULT, O_RDONLY, 0600))) {
+       if (!(pwd_tdb = tdb_open_log(tdb_state->tdbsam_location, 0, TDB_DEFAULT, O_RDONLY, 0600))) {
                DEBUG(0, ("pdb_getsampwrid: Unable to open TDB rid database!\n"));
                return False;
        }
@@ -606,7 +633,7 @@ BOOL pdb_getsampwrid (SAM_ACCOUNT *user, uint32 rid)
        /* get the record */
        data = tdb_fetch (pwd_tdb, key);
        if (!data.dptr) {
-               DEBUG(5,("pdb_getsampwrid (TDB): error fetching database.\n"));
+               DEBUG(5,("pdb_getsampwrid (TDB): error looking up RID %d by key %s.\n", rid, keystr));
                DEBUGADD(5, (" Error: %s\n", tdb_errorstr(pwd_tdb)));
                tdb_close (pwd_tdb);
                return False;
@@ -617,30 +644,26 @@ BOOL pdb_getsampwrid (SAM_ACCOUNT *user, uint32 rid)
        
        tdb_close (pwd_tdb);
        
-       return pdb_getsampwnam (user, name);
+       return tdbsam_getsampwnam (my_methods, user, name);
 }
 
 /***************************************************************************
  Delete a SAM_ACCOUNT
 ****************************************************************************/
 
-BOOL pdb_delete_sam_account(const char *sname)
+static BOOL tdbsam_delete_sam_account(struct pdb_methods *my_methods, const SAM_ACCOUNT *sam_pass)
 {
-       SAM_ACCOUNT     *sam_pass = NULL;
+       struct tdbsam_privates *tdb_state = (struct tdbsam_privates *)my_methods->private_data;
        TDB_CONTEXT     *pwd_tdb;
-       TDB_DATA        key, data;
+       TDB_DATA        key;
        fstring         keystr;
-       pstring         tdbfile;
        uint32          rid;
        fstring         name;
        
-       unix_strlower(sname, -1, name, sizeof(name));
+       unix_strlower(pdb_get_username(sam_pass), -1, name, sizeof(name));
        
-       get_private_directory(tdbfile);
-       pstrcat (tdbfile, PASSDB_FILE_NAME);
-
        /* open the TDB */
-       if (!(pwd_tdb = tdb_open_log(tdbfile, 0, TDB_DEFAULT, O_RDWR, 0600))) {
+       if (!(pwd_tdb = tdb_open_log(tdb_state->tdbsam_location, 0, TDB_DEFAULT, O_RDWR, 0600))) {
                DEBUG(0, ("Unable to open TDB passwd!"));
                return False;
        }
@@ -650,33 +673,8 @@ BOOL pdb_delete_sam_account(const char *sname)
        key.dptr = keystr;
        key.dsize = strlen (keystr) + 1;
        
-       /* get the record */
-       data = tdb_fetch (pwd_tdb, key);
-       if (!data.dptr) {
-               DEBUG(5,("pdb_delete_sam_account (TDB): error fetching database.\n"));
-               DEBUGADD(5, (" Error: %s\n", tdb_errorstr(pwd_tdb)));
-               tdb_close (pwd_tdb);
-               return False;
-       }
-  
-       /* unpack the buffer */
-       if (!pdb_init_sam (&sam_pass)) {
-               tdb_close (pwd_tdb);
-               return False;
-       }
-       
-       if (!init_sam_from_buffer (sam_pass, data.dptr, data.dsize)) {
-               DEBUG(0,("pdb_getsampwent: Bad SAM_ACCOUNT entry returned from TDB!\n"));
-               tdb_close (pwd_tdb);
-               SAFE_FREE(data.dptr);
-               return False;
-       }
-       SAFE_FREE(data.dptr);
-
        rid = pdb_get_user_rid(sam_pass);
 
-       pdb_free_sam (&sam_pass);
-       
        /* it's outaa here!  8^) */
        if (tdb_delete(pwd_tdb, key) != TDB_SUCCESS) {
                DEBUG(5, ("Error deleting entry from tdb passwd database!\n"));
@@ -709,27 +707,68 @@ BOOL pdb_delete_sam_account(const char *sname)
  Update the TDB SAM
 ****************************************************************************/
 
-static BOOL tdb_update_sam(const SAM_ACCOUNT* newpwd, BOOL override, int flag)
+static BOOL tdb_update_sam(struct pdb_methods *my_methods, const SAM_ACCOUNT* newpwd, int flag)
 {
+       struct tdbsam_privates *tdb_state = (struct tdbsam_privates *)my_methods->private_data;
        TDB_CONTEXT     *pwd_tdb = NULL;
        TDB_DATA        key, data;
        uint8           *buf = NULL;
        fstring         keystr;
-       pstring         tdbfile;
        fstring         name;
        BOOL            ret = True;
-       
-       get_private_directory(tdbfile);
-       pstrcat (tdbfile, PASSDB_FILE_NAME);
-       
-       /* if we don't have a RID, then FAIL */
-       if (!pdb_get_user_rid(newpwd))
-               DEBUG (0,("tdb_update_sam: Failing to store a SAM_ACCOUNT for [%s] without a RID\n",pdb_get_username(newpwd)));
-       if (!pdb_get_group_rid(newpwd))
-               DEBUG (0,("tdb_update_sam: Failing to store a SAM_ACCOUNT for [%s] without a primary group RID\n",pdb_get_username(newpwd)));
+       uint32          user_rid;
+       int32           tdb_ret;
+
+       /* invalidate the existing TDB iterator if it is open */
+       if (tdb_state->passwd_tdb) {
+               tdb_close(tdb_state->passwd_tdb);
+               tdb_state->passwd_tdb = NULL;
+       }
+
+       /* open the account TDB passwd*/
+       pwd_tdb = tdb_open_log(tdb_state->tdbsam_location, 0, TDB_DEFAULT, O_RDWR | O_CREAT, 0600);
+       if (!pwd_tdb)
+       {
+               DEBUG(0, ("tdb_update_sam: Unable to open TDB passwd (%s)!\n", tdb_state->tdbsam_location));
+               return False;
+       }
+
+       /* if flag == TDB_INSERT then make up a new RID else throw an error. */
+       if (!(user_rid = pdb_get_user_rid(newpwd))) {
+               if (flag & TDB_INSERT) {
+                       user_rid = BASE_RID;
+                       tdb_ret = tdb_change_int32_atomic(pwd_tdb, "RID_COUNTER", &user_rid, RID_MULTIPLIER);
+                       if (tdb_ret == -1) {
+                               ret = False;
+                               goto done;
+                       }
+                       pdb_set_user_rid(newpwd, user_rid);
+               } else {
+                       DEBUG (0,("tdb_update_sam: Failing to store a SAM_ACCOUNT for [%s] without a RID\n",pdb_get_username(newpwd)));
+                       ret = False;
+                       goto done;
+               }
+       }
+
+       if (!pdb_get_group_rid(newpwd)) {
+               if (flag & TDB_INSERT) {
+                       if (!tdb_state->permit_non_unix_accounts) {
+                               DEBUG (0,("tdb_update_sam: Failing to store a SAM_ACCOUNT for [%s] without a primary group RID\n",pdb_get_username(newpwd)));
+                               ret = False;
+                               goto done;
+                       } else {
+                               /* This seems like a good default choice for non-unix users */
+                               pdb_set_group_rid(newpwd, DOMAIN_GROUP_RID_USERS);
+                       }
+               } else {
+                       DEBUG (0,("tdb_update_sam: Failing to store a SAM_ACCOUNT for [%s] without a primary group RID\n",pdb_get_username(newpwd)));
+                       ret = False;
+                       goto done;
+               }
+       }
 
        /* copy the SAM_ACCOUNT struct into a BYTE buffer for storage */
-       if ((data.dsize=init_buffer_from_sam (&buf, newpwd)) == -1) {
+       if ((data.dsize=init_buffer_from_sam (tdb_state, &buf, newpwd)) == -1) {
                DEBUG(0,("tdb_update_sam: ERROR - Unable to copy SAM_ACCOUNT info BYTE buffer!\n"));
                ret = False;
                goto done;
@@ -738,30 +777,18 @@ static BOOL tdb_update_sam(const SAM_ACCOUNT* newpwd, BOOL override, int flag)
 
        unix_strlower(pdb_get_username(newpwd), -1, name, sizeof(name));
        
+       DEBUG(5, ("Storing %saccount %s with RID %d\n", flag == TDB_INSERT ? "(new) " : "", name, user_rid));
+
        /* setup the USER index key */
        slprintf(keystr, sizeof(keystr)-1, "%s%s", USERPREFIX, name);
        key.dptr = keystr;
        key.dsize = strlen (keystr) + 1;
 
-       /* invalidate the existing TDB iterator if it is open */
-       if (global_tdb_ent.passwd_tdb) {
-               tdb_close(global_tdb_ent.passwd_tdb);
-               global_tdb_ent.passwd_tdb = NULL;
-       }
-
-       /* open the account TDB passwd*/
-       pwd_tdb = tdb_open_log(tdbfile, 0, TDB_DEFAULT, O_RDWR | O_CREAT, 0600);
-       if (!pwd_tdb)
-       {
-               DEBUG(0, ("tdb_update_sam: Unable to open TDB passwd!\n"));
-               return False;
-       }
-
        /* add the account */
        if (tdb_store(pwd_tdb, key, data, flag) != TDB_SUCCESS) {
                DEBUG(0, ("Unable to modify passwd TDB!"));
-               DEBUGADD(0, (" Error: %s\n", tdb_errorstr(pwd_tdb)));
-               tdb_close (pwd_tdb);
+               DEBUGADD(0, (" Error: %s", tdb_errorstr(pwd_tdb)));
+               DEBUGADD(0, (" occured while storing the main record (%s)\n", keystr));
                ret = False;
                goto done;
        }
@@ -771,7 +798,7 @@ static BOOL tdb_update_sam(const SAM_ACCOUNT* newpwd, BOOL override, int flag)
        data.dptr = name;
 
        /* setup the RID index key */
-       slprintf(keystr, sizeof(keystr)-1, "%s%.8x", RIDPREFIX, pdb_get_user_rid(newpwd));
+       slprintf(keystr, sizeof(keystr)-1, "%s%.8x", RIDPREFIX, user_rid);
        key.dptr = keystr;
        key.dsize = strlen (keystr) + 1;
        
@@ -779,6 +806,7 @@ static BOOL tdb_update_sam(const SAM_ACCOUNT* newpwd, BOOL override, int flag)
        if (tdb_store(pwd_tdb, key, data, flag) != TDB_SUCCESS) {
                DEBUG(0, ("Unable to modify TDB passwd !"));
                DEBUGADD(0, (" Error: %s\n", tdb_errorstr(pwd_tdb)));
+               DEBUGADD(0, (" occured while storing the RID index (%s)\n", keystr));
                ret = False;
                goto done;
        }
@@ -795,21 +823,116 @@ done:
  Modifies an existing SAM_ACCOUNT
 ****************************************************************************/
 
-BOOL pdb_update_sam_account (const SAM_ACCOUNT *newpwd, BOOL override)
+static BOOL tdbsam_update_sam_account (struct pdb_methods *my_methods, const SAM_ACCOUNT *newpwd)
 {
-       return (tdb_update_sam(newpwd, override, TDB_MODIFY));
+       return (tdb_update_sam(my_methods, newpwd, TDB_MODIFY));
 }
 
 /***************************************************************************
  Adds an existing SAM_ACCOUNT
 ****************************************************************************/
 
-BOOL pdb_add_sam_account (const SAM_ACCOUNT *newpwd)
+static BOOL tdbsam_add_sam_account (struct pdb_methods *my_methods, const SAM_ACCOUNT *newpwd)
+{
+       return (tdb_update_sam(my_methods, newpwd, TDB_INSERT));
+}
+
+static void free_private_data(void **vp) 
+{
+       struct tdbsam_privates **tdb_state = (struct tdbsam_privates **)vp;
+       close_tdb(*tdb_state);
+       *tdb_state = NULL;
+
+       /* No need to free any further, as it is talloc()ed */
+}
+
+
+NTSTATUS pdb_init_tdbsam(PDB_CONTEXT *pdb_context, PDB_METHODS **pdb_method, const char *location)
+{
+       NTSTATUS nt_status;
+       struct tdbsam_privates *tdb_state;
+
+       if (!NT_STATUS_IS_OK(nt_status = make_pdb_methods(pdb_context->mem_ctx, pdb_method))) {
+               return nt_status;
+       }
+
+       (*pdb_method)->name = "tdbsam";
+
+       (*pdb_method)->setsampwent = tdbsam_setsampwent;
+       (*pdb_method)->endsampwent = tdbsam_endsampwent;
+       (*pdb_method)->getsampwent = tdbsam_getsampwent;
+       (*pdb_method)->getsampwnam = tdbsam_getsampwnam;
+       (*pdb_method)->getsampwrid = tdbsam_getsampwrid;
+       (*pdb_method)->add_sam_account = tdbsam_add_sam_account;
+       (*pdb_method)->update_sam_account = tdbsam_update_sam_account;
+       (*pdb_method)->delete_sam_account = tdbsam_delete_sam_account;
+
+       tdb_state = talloc_zero(pdb_context->mem_ctx, sizeof(struct tdbsam_privates));
+
+       if (!tdb_state) {
+               DEBUG(0, ("talloc() failed for tdbsam private_data!\n"));
+               return NT_STATUS_NO_MEMORY;
+       }
+
+       if (location) {
+               tdb_state->tdbsam_location = talloc_strdup(pdb_context->mem_ctx, location);
+       } else {
+               pstring tdbfile;
+               get_private_directory(tdbfile);
+               pstrcat(tdbfile, "/");
+               pstrcat(tdbfile, PASSDB_FILE_NAME);
+               tdb_state->tdbsam_location = talloc_strdup(pdb_context->mem_ctx, tdbfile);
+       }
+
+       (*pdb_method)->private_data = tdb_state;
+
+       (*pdb_method)->free_private_data = free_private_data;
+
+       return NT_STATUS_OK;
+}
+
+NTSTATUS pdb_init_tdbsam_nua(PDB_CONTEXT *pdb_context, PDB_METHODS **pdb_method, const char *location)
 {
-       return (tdb_update_sam(newpwd, True, TDB_INSERT));
+       NTSTATUS nt_status;
+       struct tdbsam_privates *tdb_state;
+       uint32 low_nua_uid, high_nua_uid;
+
+       if (!NT_STATUS_IS_OK(nt_status = pdb_init_tdbsam(pdb_context, pdb_method, location))) {
+               return nt_status;
+       }
+
+       (*pdb_method)->name = "tdbsam_nua";
+
+       tdb_state = (*pdb_method)->private_data;
+       
+       tdb_state->permit_non_unix_accounts = True;
+
+       if (!lp_non_unix_account_range(&low_nua_uid, &high_nua_uid)) {
+               DEBUG(0, ("cannot use tdbsam_nua without 'non unix account range' in smb.conf!\n"));
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
+/*     tdb_state->low_nua_rid=fallback_pdb_uid_to_user_rid(low_nua_uid);
+
+       tdb_state->high_nua_rid=fallback_pdb_uid_to_user_rid(high_nua_uid);
+*/
+       return NT_STATUS_OK;
 }
 
+
 #else
-       /* Do *NOT* make this function static. It breaks the compile on gcc. JRA */
-       void samtdb_dummy_function(void) { } /* stop some compilers complaining */
-#endif /* WITH_TDB_SAM */
+
+NTSTATUS pdb_init_tdbsam(PDB_CONTEXT *pdb_context, PDB_METHODS **pdb_method, const char *location)
+{
+       DEBUG(0, ("tdbsam not compiled in!\n"));
+       return NT_STATUS_UNSUCCESSFUL;
+}
+
+NTSTATUS pdb_init_tdbsam_nua(PDB_CONTEXT *pdb_context, PDB_METHODS **pdb_method, const char *location)
+{
+       DEBUG(0, ("tdbsam_nua not compiled in!\n"));
+       return NT_STATUS_UNSUCCESSFUL;
+}
+
+
+#endif