first pass at updating head branch to be to be the same as the SAMBA_2_0 branch
[kai/samba.git] / source3 / passdb / passdb.c
index b43333f389352748f4d4eb5eb44d2fb0fbb2a1fe..c0a20f34204c3bc7b3814bd5fa1acc2fe445f89c 100644 (file)
@@ -2,9 +2,9 @@
    Unix SMB/Netbios implementation.
    Version 1.9.
    Password and authentication handling
-   Copyright (C) Andrew Tridgell 1992-1998
+   Copyright (C) Jeremy Allison 1996-1998
    Copyright (C) Luke Kenneth Casson Leighton 1996-1998
-   
+      
    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 Software Foundation; either version 2 of the License, or
@@ -27,459 +27,380 @@ extern int DEBUGLEVEL;
 
 /*
  * This is set on startup - it defines the SID for this
- * machine.
+ * machine, and therefore the SAM database for which it is
+ * responsible.
  */
-DOM_SID global_machine_sid;
-
-/**********************************************************
- **********************************************************
-
- low-level redirection routines:
 
-       startsampwent()
-       endsampwent()
-       getsampwent()
-       getsam21pwent()
-       getsampwpos()
-       setsampwpos()
+extern DOM_SID global_sam_sid;
+extern pstring global_myname;
+extern fstring global_myworkgroup;
 
-       add_sampwd_entry()
-       mod_sampwd_entry()
-       add_sam21pwd_entry()
-       mod_sam21pwd_entry()
+/*
+ * NOTE. All these functions are abstracted into a structure
+ * that points to the correct function for the selected database. JRA.
+ *
+ * NOTE.  for the get/mod/add functions, there are two sets of functions.
+ * one supports struct sam_passwd, the other supports struct smb_passwd.
+ * for speed optimisation it is best to support both these sets.
+ * 
+ * it is, however, optional to support one set but not the other: there
+ * is conversion-capability built in to passdb.c, and run-time error
+ * detection for when neither are supported.
+ * 
+ * password database writers are recommended to implement the sam_passwd
+ * functions in a first pass, as struct sam_passwd contains more
+ * information, needed by the NT Domain support.
+ * 
+ * a full example set of derivative functions are listed below.  an API
+ * writer is expected to cut/paste these into their module, replace
+ * either one set (struct smb_passwd) or the other (struct sam_passwd)
+ * OR both, and optionally also to write display info routines
+ * (struct sam_disp_info).  lkcl
+ *
+ */
 
- **********************************************************
- **********************************************************/
+static struct passdb_ops *pdb_ops;
 
 /***************************************************************
- Start to enumerate the sam passwd list. Returns a void pointer
- to ensure no modification outside this module.
-****************************************************************/
-void *startsampwent(BOOL update)
+ Initialize the password db operations.
+***************************************************************/
+
+BOOL initialize_password_db(void)
 {
-#ifdef USE_NISPLUS_DB
-  return startnisppwent(update);
-#endif /* USE_NISPLUS_DB */
+  if (pdb_ops)
+  {
+    return True;
+  }
 
-#ifdef USE_LDAP_DB
-  return startldappwent(update);
-#endif /* USE_LDAP_DB */
+#ifdef WITH_NISPLUS
+  pdb_ops =  nisplus_initialize_password_db();
+#elif defined(WITH_LDAP)
+  pdb_ops = ldap_initialize_password_db();
+#else 
+  pdb_ops = file_initialize_password_db();
+#endif 
 
-#ifdef USE_SMBPASS_DB
-  return startsmbpwent(update);
-#endif /* USE_SMBPASS_DB */
+  return (pdb_ops != NULL);
 }
 
-/***************************************************************
- End enumeration of the sam passwd list.
-****************************************************************/
-void endsampwent(void *vp)
-{
-#ifdef USE_NISPLUS_DB
-  endnisppwent(vp);
-#endif /* USE_NISPLUS_DB */
+/*
+ * Functions that return/manipulate a struct smb_passwd.
+ */
 
-#ifdef USE_LDAP_DB
-  endldappwent(vp);
-#endif /* USE_LDAP_DB */
+/************************************************************************
+ Utility function to search smb passwd by rid.  
+*************************************************************************/
 
-#ifdef USE_SMBPASS_DB
-  endsmbpwent(vp);
-#endif /* USE_SMBPASS_DB */
+struct smb_passwd *iterate_getsmbpwrid(uint32 user_rid)
+{
+       return iterate_getsmbpwuid(pdb_user_rid_to_uid(user_rid));
 }
 
-/*************************************************************************
- Routine to return the next entry in the sam passwd list.
- *************************************************************************/
-struct smb_passwd *getsampwent(void *vp)
+/************************************************************************
+ Utility function to search smb passwd by uid.  use this if your database
+ does not have search facilities.
+*************************************************************************/
+
+struct smb_passwd *iterate_getsmbpwuid(uid_t smb_userid)
 {
-#ifdef USE_NISPLUS_DB
-  return getnisppwent(vp);
-#endif /* USE_NISPLUS_DB */
+       struct smb_passwd *pwd = NULL;
+       void *fp = NULL;
+
+       DEBUG(10, ("search by smb_userid: %x\n", (int)smb_userid));
+
+       /* Open the smb password database - not for update. */
+       fp = startsmbpwent(False);
+
+       if (fp == NULL)
+       {
+               DEBUG(0, ("unable to open smb password database.\n"));
+               return NULL;
+       }
 
-#ifdef USE_LDAP_DB
-  return getldappwent(vp);
-#endif /* USE_LDAP_DB */
+       while ((pwd = getsmbpwent(fp)) != NULL && pwd->smb_userid != smb_userid)
+      ;
 
-#ifdef USE_SMBPASS_DB
-  return getsmbpwent(vp);
-#endif /* USE_SMBPASS_DB */
+       if (pwd != NULL)
+       {
+               DEBUG(10, ("found by smb_userid: %x\n", (int)smb_userid));
+       }
+
+       endsmbpwent(fp);
+       return pwd;
 }
 
-/*************************************************************************
- Routine to return the next entry in the sam passwd list.
- *************************************************************************/
-struct sam_disp_info *getsamdispent(void *vp)
+/************************************************************************
+ Utility function to search smb passwd by name.  use this if your database
+ does not have search facilities.
+*************************************************************************/
+
+struct smb_passwd *iterate_getsmbpwnam(char *name)
 {
-       struct sam_passwd *pwd = NULL;
-       static struct sam_disp_info disp_info;
+       struct smb_passwd *pwd = NULL;
+       void *fp = NULL;
 
-#ifdef USE_NISPLUS_DB
-       pwd = getnisp21pwent(vp);
-#endif /* USE_NISPLUS_DB */
+       DEBUG(10, ("search by name: %s\n", name));
 
-#ifdef USE_LDAP_DB
-       pwd = getldap21pwent(vp);
-#endif /* USE_LDAP_DB */
+       /* Open the sam password file - not for update. */
+       fp = startsmbpwent(False);
 
-#ifdef USE_SMBPASS_DB
-       pwd = getsmb21pwent(vp);
-#endif /* USE_SMBPASS_DB */
+       if (fp == NULL)
+       {
+               DEBUG(0, ("unable to open smb password database.\n"));
+               return NULL;
+       }
 
-       if (pwd == NULL) return NULL;
+       while ((pwd = getsmbpwent(fp)) != NULL && !strequal(pwd->smb_name, name))
+      ;
 
-       disp_info.smb_name  = pwd->smb_name;
-       disp_info.full_name = pwd->full_name;
-       disp_info.user_rid  = pwd->user_rid;
+       if (pwd != NULL)
+       {
+               DEBUG(10, ("found by name: %s\n", name));
+       }
 
-       return &disp_info;
+       endsmbpwent(fp);
+       return pwd;
 }
 
-/*************************************************************************
- Routine to return the next entry in the sam passwd list.
- *************************************************************************/
-struct sam_passwd *getsam21pwent(void *vp)
-{
-#ifdef USE_NISPLUS_DB
-  return getnisp21pwent(vp);
-#endif /* USE_NISPLUS_DB */
+/***************************************************************
+ Start to enumerate the smb or sam passwd list. Returns a void pointer
+ to ensure no modification outside this module.
 
-#ifdef USE_LDAP_DB
-  return getldap21pwent(vp);
-#endif /* USE_LDAP_DB */
+ Note that currently it is being assumed that a pointer returned
+ from this function may be used to enumerate struct sam_passwd
+ entries as well as struct smb_passwd entries. This may need
+ to change. JRA.
 
-#ifdef USE_SMBPASS_DB
-  return getsmb21pwent(vp);
-#endif /* USE_SMBPASS_DB */
-}
+****************************************************************/
 
-/*************************************************************************
- Return the current position in the sam passwd list as an unsigned long.
- This must be treated as an opaque token.
- *************************************************************************/
-unsigned long getsampwpos(void *vp)
+void *startsmbpwent(BOOL update)
 {
-#ifdef USE_NISPLUS_DB
-  return getnisppwpos(vp);
-#endif /* USE_NISPLUS_DB */
-
-#ifdef USE_LDAP_DB
-  return getldappwpos(vp);
-#endif /* USE_LDAP_DB */
-
-#ifdef USE_SMBPASS_DB
-  return getsmbpwpos(vp);
-#endif /* USE_SMBPASS_DB */
+  return pdb_ops->startsmbpwent(update);
 }
 
-/*************************************************************************
- Set the current position in the sam passwd list from unsigned long.
- This must be treated as an opaque token.
- *************************************************************************/
-BOOL setsampwpos(void *vp, unsigned long tok)
-{
-#ifdef USE_NISPLUS_DB
-  return setnisppwpos(vp, tok);
-#endif /* USE_NISPLUS_DB */
+/***************************************************************
+ End enumeration of the smb or sam passwd list.
 
-#ifdef USE_LDAP_DB
-  return setldappwpos(vp, tok);
-#endif /* USE_LDAP_DB */
+ Note that currently it is being assumed that a pointer returned
+ from this function may be used to enumerate struct sam_passwd
+ entries as well as struct smb_passwd entries. This may need
+ to change. JRA.
 
-#ifdef USE_SMBPASS_DB
-  return setsmbpwpos(vp, tok);
-#endif /* USE_SMBPASS_DB */
-}
+****************************************************************/
 
-/************************************************************************
- Routine to add an entry to the sam passwd file.
-*************************************************************************/
-BOOL add_sampwd_entry(struct smb_passwd *newpwd)
+void endsmbpwent(void *vp)
 {
-#ifdef USE_NISPLUS_DB
-  return add_nisppwd_entry(newpwd);
-#endif /* USE_NISPLUS_DB */
+  pdb_ops->endsmbpwent(vp);
+}
 
-#ifdef USE_LDAP_DB
-  return add_ldappwd_entry(newpwd);
-#endif /* USE_LDAP_DB */
+/*************************************************************************
+ Routine to return the next entry in the smb passwd list.
+ *************************************************************************/
 
-#ifdef USE_SMBPASS_DB
-  return add_smbpwd_entry(newpwd);
-#endif /* USE_SMBPASS_DB */
+struct smb_passwd *getsmbpwent(void *vp)
+{
+       return pdb_ops->getsmbpwent(vp);
 }
 
 /************************************************************************
- Routine to add an entry to the sam passwd file.
+ Routine to add an entry to the smb passwd file.
 *************************************************************************/
-BOOL add_sam21pwd_entry(struct sam_passwd *newpwd)
-{
-#ifdef USE_NISPLUS_DB
-  return add_nisp21pwd_entry(newpwd);
-#endif /* USE_NISPLUS_DB */
 
-#ifdef USE_LDAP_DB
-  return add_ldap21pwd_entry(newpwd);
-#endif /* USE_LDAP_DB */
-
-#ifdef USE_SMBPASS_DB
-  return add_smb21pwd_entry(newpwd);
-#endif /* USE_SMBPASS_DB */
+BOOL add_smbpwd_entry(struct smb_passwd *newpwd)
+{
+       return pdb_ops->add_smbpwd_entry(newpwd);
 }
 
 /************************************************************************
- Routine to search the sam passwd file for an entry matching the username.
+ Routine to search the smb passwd file for an entry matching the username.
  and then modify its password entry. We can't use the startsampwent()/
  getsampwent()/endsampwent() interfaces here as we depend on looking
  in the actual file to decide how much room we have to write data.
  override = False, normal
  override = True, override XXXXXXXX'd out password or NO PASS
 ************************************************************************/
-BOOL mod_sampwd_entry(struct smb_passwd* pwd, BOOL override)
-{
-#ifdef USE_NISPLUS_DB
-  return mod_nisppwd_entry(pwd, override);
-#endif /* USE_NISPLUS_DB */
-
-#ifdef USE_LDAP_DB
-  return mod_ldappwd_entry(pwd, override);
-#endif /* USE_LDAP_DB */
 
-#ifdef USE_SMBPASS_DB
-  return mod_smbpwd_entry(pwd, override);
-#endif /* USE_SMBPASS_DB */
+BOOL mod_smbpwd_entry(struct smb_passwd* pwd, BOOL override)
+{
+       return pdb_ops->mod_smbpwd_entry(pwd, override);
 }
 
 /************************************************************************
- Routine to search the sam passwd file for an entry matching the username.
- and then modify its password entry. We can't use the startsampwent()/
- getsampwent()/endsampwent() interfaces here as we depend on looking
- in the actual file to decide how much room we have to write data.
- override = False, normal
- override = True, override XXXXXXXX'd out password or NO PASS
-************************************************************************/
-BOOL mod_sam21pwd_entry(struct sam_passwd* pwd, BOOL override)
+ Routine to search smb passwd by name.
+*************************************************************************/
+
+struct smb_passwd *getsmbpwnam(char *name)
 {
-#ifdef USE_NISPLUS_DB
-  return mod_nisp21pwd_entry(pwd, override);
-#endif /* USE_NISPLUS_DB */
+       return pdb_ops->getsmbpwnam(name);
+}
 
-#ifdef USE_LDAP_DB
-  return mod_ldap21pwd_entry(pwd, override);
-#endif /* USE_LDAP_DB */
+/************************************************************************
+ Routine to search smb passwd by user rid.
+*************************************************************************/
 
-#ifdef USE_SMBPASS_DB
-  return mod_smb21pwd_entry(pwd, override);
-#endif /* USE_SMBPASS_DB */
+struct smb_passwd *getsmbpwrid(uint32 user_rid)
+{
+       return pdb_ops->getsmbpwrid(user_rid);
 }
 
-/**********************************************************
- **********************************************************
+/************************************************************************
+ Routine to search smb passwd by uid.
+*************************************************************************/
 
- high-level database routines:
-       getsampwnam()
-       getsampwuid()
-       getsam21pwnam()
-       getsam21pwuid()
+struct smb_passwd *getsmbpwuid(uid_t smb_userid)
+{
+       return pdb_ops->getsmbpwuid(smb_userid);
+}
 
- **********************************************************
- **********************************************************/
+/*
+ * Functions that manupulate a struct sam_passwd.
+ */
 
 /************************************************************************
Routine to search sam passwd by name.  use this if your database
Utility function to search sam passwd by name.  use this if your database
  does not have search facilities.
 *************************************************************************/
-static struct smb_passwd *_getsampwnam(char *name)
+
+struct sam_passwd *iterate_getsam21pwnam(char *name)
 {
-       struct smb_passwd *pwd = NULL;
+       struct sam_passwd *pwd = NULL;
        void *fp = NULL;
 
-       DEBUG(10, ("getsampwnam: search by name: %s\n", name));
+       DEBUG(10, ("search by name: %s\n", name));
 
-       /* Open the sam password file - not for update. */
-       fp = startsampwent(False);
+       /* Open the smb password database - not for update. */
+       fp = startsmbpwent(False);
 
        if (fp == NULL)
        {
-               DEBUG(0, ("getsampwnam: unable to open sam password database.\n"));
+               DEBUG(0, ("unable to open sam password database.\n"));
                return NULL;
        }
 
-       while ((pwd = getsampwent(fp)) != NULL && !strequal(pwd->smb_name, name));
+       while ((pwd = getsam21pwent(fp)) != NULL && !strequal(pwd->smb_name, name))
+       {
+               DEBUG(10, ("iterate: %s 0x%x\n", pwd->smb_name, pwd->user_rid));
+       }
 
        if (pwd != NULL)
        {
-               DEBUG(10, ("getsampwnam: found by name: %s\n", name));
+               DEBUG(10, ("found by name: %s\n", name));
        }
 
-       endsampwent(fp);
+       endsmbpwent(fp);
        return pwd;
 }
 
 /************************************************************************
- Routine to search sam passwd by name.
-*************************************************************************/
-struct smb_passwd *getsampwnam(char *name)
-{
-#ifdef USE_NISPLUS_DB
-       return _getsampwnam(name);
-#endif /* USE_NISPLUS_DB */
-
-#ifdef USE_LDAP_DB
-       return _getsampwnam(name);
-#endif /* USE_LDAP_DB */
+ Utility function to search sam passwd by rid.  use this if your database
+ does not have search facilities.
 
-#ifdef USE_SMBPASS_DB
-       return _getsampwnam(name);
-#endif /* USE_SMBPASS_DB */
-}
+ search capability by both rid and uid are needed as the rid <-> uid
+ mapping may be non-monotonic.  
 
-/************************************************************************
- Routine to search sam passwd by name.  use this if your database
- does not have search facilities.
 *************************************************************************/
-static struct sam_passwd *_getsam21pwnam(char *name)
+
+struct sam_passwd *iterate_getsam21pwrid(uint32 rid)
 {
        struct sam_passwd *pwd = NULL;
        void *fp = NULL;
 
-       DEBUG(10, ("_getsam21pwnam: search by name: %s\n", name));
+       DEBUG(10, ("search by rid: %x\n", rid));
 
-       /* Open the sam password file - not for update. */
-       fp = startsampwent(False);
+       /* Open the smb password file - not for update. */
+       fp = startsmbpwent(False);
 
        if (fp == NULL)
        {
-               DEBUG(0, ("_getsam21pwnam: unable to open sam password database.\n"));
+               DEBUG(0, ("unable to open sam password database.\n"));
                return NULL;
        }
 
-       while ((pwd = getsam21pwent(fp)) != NULL && !strequal(pwd->smb_name, name));
+       while ((pwd = getsam21pwent(fp)) != NULL && pwd->user_rid != rid)
+       {
+               DEBUG(10, ("iterate: %s 0x%x\n", pwd->smb_name, pwd->user_rid));
+       }
 
        if (pwd != NULL)
        {
-               DEBUG(10, ("_getsam21pwnam: found by name: %s\n", name));
+               DEBUG(10, ("found by user_rid: %x\n", rid));
        }
 
-       endsampwent(fp);
+       endsmbpwent(fp);
        return pwd;
 }
 
 /************************************************************************
- Routine to search sam passwd by name.
-*************************************************************************/
-struct sam_passwd *getsam21pwnam(char *name)
-{
-#ifdef USE_NISPLUS_DB
-       return _getsam21pwnam(name);
-#endif /* USE_NISPLUS_DB */
-
-#ifdef USE_LDAP_DB
-       return _getsam21pwnam(name);
-#endif /* USE_LDAP_DB */
+ Utility function to search sam passwd by uid.  use this if your database
+ does not have search facilities.
 
-#ifdef USE_SMBPASS_DB
-       return _getsam21pwnam(name);
-#endif /* USE_SMBPASS_DB */
-}
+ search capability by both rid and uid are needed as the rid <-> uid
+ mapping may be non-monotonic.  
 
-/************************************************************************
- Routine to search sam passwd by uid.  use this if your database
- does not have search facilities.
 *************************************************************************/
-static struct smb_passwd *_getsampwuid(uid_t smb_userid)
+
+struct sam_passwd *iterate_getsam21pwuid(uid_t uid)
 {
-       struct smb_passwd *pwd = NULL;
+       struct sam_passwd *pwd = NULL;
        void *fp = NULL;
 
-       DEBUG(10, ("getsampwuid: search by smb_userid: %x\n", smb_userid));
+       DEBUG(10, ("search by uid: %x\n", (int)uid));
 
-       /* Open the sam password file - not for update. */
-       fp = startsampwent(False);
+       /* Open the smb password file - not for update. */
+       fp = startsmbpwent(False);
 
        if (fp == NULL)
        {
-               DEBUG(0, ("getsampwuid: unable to open sam password database.\n"));
+               DEBUG(0, ("unable to open sam password database.\n"));
                return NULL;
        }
 
-       while ((pwd = getsampwent(fp)) != NULL && pwd->smb_userid != smb_userid);
+       while ((pwd = getsam21pwent(fp)) != NULL && pwd->smb_userid != uid)
+      ;
 
        if (pwd != NULL)
        {
-               DEBUG(10, ("getsampwuid: found by smb_userid: %x\n", smb_userid));
+               DEBUG(10, ("found by smb_userid: %x\n", (int)uid));
        }
 
        endsmbpwent(fp);
        return pwd;
 }
 
-/************************************************************************
- Routine to search sam passwd by uid.
-*************************************************************************/
-struct smb_passwd *getsampwuid(uid_t smb_userid)
+/*************************************************************************
+ Routine to return a display info structure, by rid
+ *************************************************************************/
+struct sam_disp_info *getsamdisprid(uint32 rid)
 {
-#ifdef USE_NISPLUS_DB
-       return _getsampwuid(smb_userid);
-#endif /* USE_NISPLUS_DB */
+       return pdb_ops->getsamdisprid(rid);
+}
 
-#ifdef USE_LDAP_DB
-       return _getsampwuid(smb_userid);
-#endif /* USE_LDAP_DB */
+/*************************************************************************
+ Routine to return the next entry in the sam passwd list.
+ *************************************************************************/
 
-#ifdef USE_SMBPASS_DB
-       return _getsampwuid(smb_userid);
-#endif /* USE_SMBPASS_DB */
+struct sam_passwd *getsam21pwent(void *vp)
+{
+       return pdb_ops->getsam21pwent(vp);
 }
 
 
 /************************************************************************
- Routine to search sam passwd by rid.  use this if your database
- does not have search facilities.
+ Routine to search sam passwd by name.
 *************************************************************************/
-static struct sam_passwd *_getsam21pwrid(uint32 rid)
-{
-       struct sam_passwd *pwd = NULL;
-       void *fp = NULL;
-
-       DEBUG(10, ("_getsam21pwrid: search by rid: %x\n", rid));
-
-       /* Open the sam password file - not for update. */
-       fp = startsampwent(False);
-
-       if (fp == NULL)
-       {
-               DEBUG(0, ("_getsam21pwrid: unable to open sam password database.\n"));
-               return NULL;
-       }
-
-       while ((pwd = getsam21pwent(fp)) != NULL && pwd->user_rid != rid);
-
-       if (pwd != NULL)
-       {
-               DEBUG(10, ("_getsam21pwrid: found by smb_userid: %x\n", rid));
-       }
 
-       endsmbpwent(fp);
-       return pwd;
+struct sam_passwd *getsam21pwnam(char *name)
+{
+       return pdb_ops->getsam21pwnam(name);
 }
 
 /************************************************************************
  Routine to search sam passwd by rid.  
 *************************************************************************/
+
 struct sam_passwd *getsam21pwrid(uint32 rid)
 {
-#ifdef USE_NISPLUS_DB
-       return _getsam21pwrid(rid);
-#endif /* USE_NISPLUS_DB */
-
-#ifdef USE_LDAP_DB
-       return _getsam21pwrid(rid);
-#endif /* USE_LDAP_DB */
-
-#ifdef USE_SMBPASS_DB
-       return _getsam21pwrid(rid);
-#endif /* USE_SMBPASS_DB */
+       return pdb_ops->getsam21pwrid(rid);
 }
 
 
@@ -492,14 +413,24 @@ struct sam_passwd *getsam21pwrid(uint32 rid)
  **********************************************************
  **********************************************************/
 
+/*************************************************************
+ initialises a struct sam_disp_info.
+ **************************************************************/
+
+static void pdb_init_dispinfo(struct sam_disp_info *user)
+{
+       if (user == NULL) return;
+       memset((char *)user, '\0', sizeof(*user));
+}
+
 /*************************************************************
  initialises a struct smb_passwd.
  **************************************************************/
+
 void pdb_init_smb(struct smb_passwd *user)
 {
        if (user == NULL) return;
-
-       bzero(user, sizeof(*user));
+       memset((char *)user, '\0', sizeof(*user));
        user->pass_last_set_time    = (time_t)-1;
 }
 
@@ -509,8 +440,7 @@ void pdb_init_smb(struct smb_passwd *user)
 void pdb_init_sam(struct sam_passwd *user)
 {
        if (user == NULL) return;
-
-       bzero(user, sizeof(*user));
+       memset((char *)user, '\0', sizeof(*user));
        user->logon_time            = (time_t)-1;
        user->logoff_time           = (time_t)-1;
        user->kickoff_time          = (time_t)-1;
@@ -519,9 +449,29 @@ void pdb_init_sam(struct sam_passwd *user)
        user->pass_must_change_time = (time_t)-1;
 }
 
+/*************************************************************************
+ Routine to return the next entry in the sam passwd list.
+ *************************************************************************/
+
+struct sam_disp_info *pdb_sam_to_dispinfo(struct sam_passwd *user)
+{
+       static struct sam_disp_info disp_info;
+
+       if (user == NULL) return NULL;
+
+       pdb_init_dispinfo(&disp_info);
+
+       disp_info.smb_name  = user->smb_name;
+       disp_info.full_name = user->full_name;
+       disp_info.user_rid  = user->user_rid;
+
+       return &disp_info;
+}
+
 /*************************************************************
  converts a sam_passwd structure to a smb_passwd structure.
  **************************************************************/
+
 struct smb_passwd *pdb_sam_to_smb(struct sam_passwd *user)
 {
        static struct smb_passwd pw_buf;
@@ -540,9 +490,11 @@ struct smb_passwd *pdb_sam_to_smb(struct sam_passwd *user)
        return &pw_buf;
 }
 
+
 /*************************************************************
  converts a smb_passwd structure to a sam_passwd structure.
  **************************************************************/
+
 struct sam_passwd *pdb_smb_to_sam(struct smb_passwd *user)
 {
        static struct sam_passwd pw_buf;
@@ -561,80 +513,37 @@ struct sam_passwd *pdb_smb_to_sam(struct smb_passwd *user)
        return &pw_buf;
 }
 
-/*******************************************************************
- gets password-database-format time from a string.
- ********************************************************************/
-static time_t get_time_from_string(char *p)
+/**********************************************************
+ Encode the account control bits into a string.
+ length = length of string to encode into (including terminating
+ null). length *MUST BE MORE THAN 2* !
+ **********************************************************/
+
+char *pdb_encode_acct_ctrl(uint16 acct_ctrl, size_t length)
 {
-       int i;
+  static fstring acct_str;
+  size_t i = 0;
 
-       for (i = 0; i < 8; i++)
-       {
-               if (p[i] == '\0' || !isxdigit(p[i]))
-               break;
-       }
-       if (i == 8)
-       {
-               /*
-                * p points at 8 characters of hex digits - 
-                * read into a time_t as the seconds since
-                * 1970 that the password was last changed.
-                */
-               return (time_t)strtol((char *)p, NULL, 16);
-       }
-       return (time_t)-1;
-}
+  acct_str[i++] = '[';
 
-/*******************************************************************
- gets password last set time
- ********************************************************************/
-time_t pdb_get_last_set_time(char *p)
-{
-       if (*p && StrnCaseCmp((char *)p, "LCT-", 4))
-       {
-               return get_time_from_string(p + 4);
-       }
-       return (time_t)-1;
-}
+  if (acct_ctrl & ACB_PWNOTREQ ) acct_str[i++] = 'N';
+  if (acct_ctrl & ACB_DISABLED ) acct_str[i++] = 'D';
+  if (acct_ctrl & ACB_HOMDIRREQ) acct_str[i++] = 'H';
+  if (acct_ctrl & ACB_TEMPDUP  ) acct_str[i++] = 'T'; 
+  if (acct_ctrl & ACB_NORMAL   ) acct_str[i++] = 'U';
+  if (acct_ctrl & ACB_MNS      ) acct_str[i++] = 'M';
+  if (acct_ctrl & ACB_WSTRUST  ) acct_str[i++] = 'W';
+  if (acct_ctrl & ACB_SVRTRUST ) acct_str[i++] = 'S';
+  if (acct_ctrl & ACB_AUTOLOCK ) acct_str[i++] = 'L';
+  if (acct_ctrl & ACB_PWNOEXP  ) acct_str[i++] = 'X';
+  if (acct_ctrl & ACB_DOMTRUST ) acct_str[i++] = 'I';
 
+  for ( ; i < length - 2 ; i++ ) { acct_str[i] = ' '; }
 
-/*******************************************************************
- sets password-database-format time in a string.
- ********************************************************************/
-static void set_time_in_string(char *p, int max_len, char *type, time_t t)
-{
-       slprintf(p, max_len, ":%s-%08X:", type, (uint32)t);
-}
+  i = length - 2;
+  acct_str[i++] = ']';
+  acct_str[i++] = '\0';
 
-/*******************************************************************
- sets password last set time
- ********************************************************************/
-void pdb_set_last_set_time(char *p, int max_len, time_t t)
-{
-       set_time_in_string(p, max_len, "LCT", t);
-}
-/**********************************************************
- Encode the account control bits into a string.
- **********************************************************/
-char *pdb_encode_acct_ctrl(uint16 acct_ctrl)
-{
-  static fstring acct_str;
-  char *p = acct_str;
-  *p++ = '[';
-
-  if (acct_ctrl & ACB_HOMDIRREQ) *p++ = 'H';
-  if (acct_ctrl & ACB_TEMPDUP  ) *p++ = 'T'; 
-  if (acct_ctrl & ACB_NORMAL   ) *p++ = 'U';
-  if (acct_ctrl & ACB_MNS      ) *p++ = 'M';
-  if (acct_ctrl & ACB_WSTRUST  ) *p++ = 'W';
-  if (acct_ctrl & ACB_SVRTRUST ) *p++ = 'S';
-  if (acct_ctrl & ACB_AUTOLOCK ) *p++ = 'L';
-  if (acct_ctrl & ACB_PWNOEXP  ) *p++ = 'X';
-  if (acct_ctrl & ACB_DOMTRUST ) *p++ = 'I';
-      
-  *p++ = ']';
-  *p = '\0';
   return acct_str;
 }     
 
@@ -645,7 +554,8 @@ char *pdb_encode_acct_ctrl(uint16 acct_ctrl)
  reason: vertical line-up code clarity - all case statements fit into
  15 lines, which is more important.
  **********************************************************/
-uint16 pdb_decode_acct_ctrl(char *p)
+
+uint16 pdb_decode_acct_ctrl(const char *p)
 {
        uint16 acct_ctrl = 0;
        BOOL finished = False;
@@ -661,15 +571,8 @@ uint16 pdb_decode_acct_ctrl(char *p)
        {
                switch (*p)
                {
-#if 0
-                       /*
-                        * Hmmm. Don't allow these to be set/read independently
-                        * of the actual password fields. We don't want a mismatch.
-                        * JRA.
-                        */
                        case 'N': { acct_ctrl |= ACB_PWNOTREQ ; break; /* 'N'o password. */ }
                        case 'D': { acct_ctrl |= ACB_DISABLED ; break; /* 'D'isabled. */ }
-#endif 
                        case 'H': { acct_ctrl |= ACB_HOMDIRREQ; break; /* 'H'omedir required. */ }
                        case 'T': { acct_ctrl |= ACB_TEMPDUP  ; break; /* 'T'emp account. */ } 
                        case 'U': { acct_ctrl |= ACB_NORMAL   ; break; /* 'U'ser account (normal). */ } 
@@ -679,7 +582,7 @@ uint16 pdb_decode_acct_ctrl(char *p)
                        case 'L': { acct_ctrl |= ACB_AUTOLOCK ; break; /* 'L'ocked account. */ } 
                        case 'X': { acct_ctrl |= ACB_PWNOEXP  ; break; /* No 'X'piry on password */ } 
                        case 'I': { acct_ctrl |= ACB_DOMTRUST ; break; /* 'I'nterdomain trust account. */ }
-
+            case ' ': { break; }
                        case ':':
                        case '\n':
                        case '\0': 
@@ -691,36 +594,163 @@ uint16 pdb_decode_acct_ctrl(char *p)
        return acct_ctrl;
 }
 
+/*******************************************************************
+ gets password-database-format time from a string.
+ ********************************************************************/
+
+static time_t get_time_from_string(const char *p)
+{
+       int i;
+
+       for (i = 0; i < 8; i++)
+       {
+               if (p[i] == '\0' || !isxdigit((int)(p[i]&0xFF)))
+               break;
+       }
+       if (i == 8)
+       {
+               /*
+                * p points at 8 characters of hex digits - 
+                * read into a time_t as the seconds since
+                * 1970 that the password was last changed.
+                */
+               return (time_t)strtol(p, NULL, 16);
+       }
+       return (time_t)-1;
+}
+
+/*******************************************************************
+ gets password last set time
+ ********************************************************************/
+
+time_t pdb_get_last_set_time(const char *p)
+{
+       if (*p && StrnCaseCmp(p, "LCT-", 4))
+       {
+               return get_time_from_string(p + 4);
+       }
+       return (time_t)-1;
+}
+
+
+/*******************************************************************
+ sets password-database-format time in a string.
+ ********************************************************************/
+static void set_time_in_string(char *p, int max_len, char *type, time_t t)
+{
+       slprintf(p, max_len, ":%s-%08X:", type, (uint32)t);
+}
+
+/*******************************************************************
+ sets logon time
+ ********************************************************************/
+void pdb_set_logon_time(char *p, int max_len, time_t t)
+{
+       set_time_in_string(p, max_len, "LNT", t);
+}
+
+/*******************************************************************
+ sets logoff time
+ ********************************************************************/
+void pdb_set_logoff_time(char *p, int max_len, time_t t)
+{
+       set_time_in_string(p, max_len, "LOT", t);
+}
+
+/*******************************************************************
+ sets kickoff time
+ ********************************************************************/
+void pdb_set_kickoff_time(char *p, int max_len, time_t t)
+{
+       set_time_in_string(p, max_len, "KOT", t);
+}
+
+/*******************************************************************
+ sets password can change time
+ ********************************************************************/
+void pdb_set_can_change_time(char *p, int max_len, time_t t)
+{
+       set_time_in_string(p, max_len, "CCT", t);
+}
+
+/*******************************************************************
+ sets password last set time
+ ********************************************************************/
+void pdb_set_must_change_time(char *p, int max_len, time_t t)
+{
+       set_time_in_string(p, max_len, "MCT", t);
+}
+
+/*******************************************************************
+ sets password last set time
+ ********************************************************************/
+void pdb_set_last_set_time(char *p, int max_len, time_t t)
+{
+       set_time_in_string(p, max_len, "LCT", t);
+}
+
+
 /*************************************************************
- Routine to get the next 32 hex characters and turn them
+ Routine to set 32 hex password characters from a 16 byte array.
+**************************************************************/
+void pdb_sethexpwd(char *p, unsigned char *pwd, uint16 acct_ctrl)
+{
+       if (pwd != NULL)
+       {
+               int i;
+               for (i = 0; i < 16; i++)
+               {
+                       slprintf(&p[i*2], 3, "%02X", pwd[i]);
+               }
+       }
+       else
+       {
+               if (IS_BITS_SET_ALL(acct_ctrl, ACB_PWNOTREQ))
+               {
+                       safe_strcpy(p, "NO PASSWORDXXXXXXXXXXXXXXXXXXXXX", 33);
+               }
+               else
+               {
+                       safe_strcpy(p, "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX", 33);
+               }
+       }
+}
+/*************************************************************
+ Routine to get the 32 hex characters and turn them
  into a 16 byte array.
 **************************************************************/
-int pdb_gethexpwd(char *p, char *pwd)
+BOOL pdb_gethexpwd(char *p, unsigned char *pwd)
 {
-  int i;
-  unsigned char   lonybble, hinybble;
-  char           *hexchars = "0123456789ABCDEF";
-  char           *p1, *p2;
+       int i;
+       unsigned char   lonybble, hinybble;
+       char           *hexchars = "0123456789ABCDEF";
+       char           *p1, *p2;
 
-  for (i = 0; i < 32; i += 2) {
-    hinybble = toupper(p[i]);
-    lonybble = toupper(p[i + 1]);
-    p1 = strchr(hexchars, hinybble);
-    p2 = strchr(hexchars, lonybble);
-    if (!p1 || !p2)
-      return (False);
-    hinybble = PTR_DIFF(p1, hexchars);
-    lonybble = PTR_DIFF(p2, hexchars);
-    pwd[i / 2] = (hinybble << 4) | lonybble;
-  }
-  return (True);
+       for (i = 0; i < 32; i += 2)
+       {
+               hinybble = toupper(p[i]);
+               lonybble = toupper(p[i + 1]);
+
+               p1 = strchr(hexchars, hinybble);
+               p2 = strchr(hexchars, lonybble);
+
+               if (!p1 || !p2)
+               {
+                       return (False);
+               }
+
+               hinybble = PTR_DIFF(p1, hexchars);
+               lonybble = PTR_DIFF(p2, hexchars);
+
+               pwd[i / 2] = (hinybble << 4) | lonybble;
+       }
+       return (True);
 }
 
 /*******************************************************************
  Group and User RID username mapping function
  ********************************************************************/
+
 BOOL pdb_name_to_rid(char *user_name, uint32 *u_rid, uint32 *g_rid)
 {
     struct passwd *pw = Get_Pwnam(user_name, False);
@@ -730,11 +760,11 @@ BOOL pdb_name_to_rid(char *user_name, uint32 *u_rid, uint32 *g_rid)
                return False;
        }
 
-    if (!pw)
+       if (!pw)
        {
-      DEBUG(1,("Username %s is invalid on this system\n", user_name));
-      return False;
-    }
+               DEBUG(1,("Username %s is invalid on this system\n", user_name));
+               return False;
+       }
 
        if (user_in_list(user_name, lp_domain_guest_users()))
        {
@@ -760,12 +790,15 @@ BOOL pdb_name_to_rid(char *user_name, uint32 *u_rid, uint32 *g_rid)
 /****************************************************************************
  Read the machine SID from a file.
 ****************************************************************************/
+
 static BOOL read_sid_from_file(int fd, char *sid_file)
 {   
   fstring fline;
     
-  if(read(fd, &fline, sizeof(fline) -1 ) < 0) {
-    DEBUG(0,("read_sid_from_file: unable to read file %s. Error was %s\n",
+  memset(fline, '\0', sizeof(fline));
+
+  if(read(fd, fline, sizeof(fline) -1 ) < 0) {
+    DEBUG(0,("unable to read file %s. Error was %s\n",
            sid_file, strerror(errno) ));
     return False;
   }
@@ -775,8 +808,8 @@ static BOOL read_sid_from_file(int fd, char *sid_file)
    */
 
   fline[sizeof(fline)-1] = '\0';
-  if(!string_to_sid( &global_machine_sid, fline)) {
-    DEBUG(0,("read_sid_from_file: unable to generate machine SID.\n"));
+  if(!string_to_sid( &global_sam_sid, fline)) {
+    DEBUG(0,("unable to generate machine SID.\n"));
     return False;
   }
 
@@ -787,189 +820,387 @@ static BOOL read_sid_from_file(int fd, char *sid_file)
  Generate the global machine sid. Look for the MACHINE.SID file first, if
  not found then look in smb.conf and use it to create the MACHINE.SID file.
 ****************************************************************************/
-BOOL pdb_generate_machine_sid(void)
-{
-  int fd;
-  char *p;
-  pstring sid_file;
-  fstring sid_string;
-  struct stat st;
-  uchar raw_sid_data[12];
-
-  pstrcpy(sid_file, lp_smb_passwd_file());
-  p = strrchr(sid_file, '/');
-  if(p != NULL)
-    *++p = '\0';
-    
-  pstrcat(sid_file, "MACHINE.SID");
-    
-  if((fd = open( sid_file, O_RDWR | O_CREAT, 0644)) < 0 ) {
-    DEBUG(0,("generate_machine_sid: unable to open or create file %s. Error was %s\n",
-             sid_file, strerror(errno) ));
-    return False;
-  } 
-  
-  /*
-   * Check if the file contains data.
-   */
-    
-  if(fstat( fd, &st) < 0) {
-    DEBUG(0,("generate_machine_sid: unable to stat file %s. Error was %s\n",
-             sid_file, strerror(errno) ));
-    close(fd);
-    return False;
-  } 
-  
-  if(st.st_size > 0) {
-    /*
-     * We have a valid SID - read it.
-     */
-    if(!read_sid_from_file( fd, sid_file)) {
-      DEBUG(0,("generate_machine_sid: unable to read file %s. Error was %s\n",
-             sid_file, strerror(errno) ));
-      close(fd);
-      return False;
-    }
-    close(fd);
-    return True;
-  } 
-  
-  /*
-   * The file contains no data - we may need to generate our
-   * own sid. Try the lp_domain_sid() first.
-   */
-    
-  if(*lp_domain_sid())
-    fstrcpy( sid_string, lp_domain_sid());
-  else {
-    /*
-     * Generate the new sid data & turn it into a string.
-     */
-    int i;
-    generate_random_buffer( raw_sid_data, 12, True);
+BOOL pdb_generate_sam_sid(void)
+{
+       int fd;
+       char *p;
+       pstring sid_file;
+       fstring sid_string;
+       SMB_STRUCT_STAT st;
+       BOOL overwrite_bad_sid = False;
+
+       generate_wellknown_sids();
+
+       pstrcpy(sid_file, lp_smb_passwd_file());
+       p = strrchr(sid_file, '/');
+       if(p != NULL) {
+               *++p = '\0';
+       }
+
+       if (!directory_exist(sid_file, NULL)) {
+               if (mkdir(sid_file, 0700) != 0) {
+                       DEBUG(0,("can't create private directory %s : %s\n",
+                                sid_file, strerror(errno)));
+                       return False;
+               }
+       }
+
+       pstrcat(sid_file, "MACHINE.SID");
     
-    fstrcpy( sid_string, "S-1-5-21");
-    for( i = 0; i < 3; i++) {
-      fstring tmp_string;
-      slprintf( tmp_string, sizeof(tmp_string) - 1, "-%u", IVAL(raw_sid_data, i*4));
-      fstrcat( sid_string, tmp_string);
-    }
-  } 
+       if((fd = sys_open(sid_file, O_RDWR | O_CREAT, 0644)) == -1) {
+               DEBUG(0,("unable to open or create file %s. Error was %s\n",
+                        sid_file, strerror(errno) ));
+               return False;
+       } 
   
-  fstrcat(sid_string, "\n");
-    
-  /*
-   * Ensure our new SID is valid.
-   */
-    
-  if(!string_to_sid( &global_machine_sid, sid_string)) {
-    DEBUG(0,("generate_machine_sid: unable to generate machine SID.\n"));
-    return False;
-  } 
+       /*
+        * Check if the file contains data.
+        */
+       
+       if(sys_fstat( fd, &st) < 0) {
+               DEBUG(0,("unable to stat file %s. Error was %s\n",
+                        sid_file, strerror(errno) ));
+               close(fd);
+               return False;
+       
   
-  /*
-   * Do an exclusive blocking lock on the file.
-   */
-    
-  if(!do_file_lock( fd, 60, F_WRLCK)) {
-    DEBUG(0,("generate_machine_sid: unable to lock file %s. Error was %s\n",
-             sid_file, strerror(errno) ));
-    close(fd);
-    return False;
-  } 
+       if(st.st_size > 0) {
+               /*
+                * We have a valid SID - read it.
+                */
+               if(!read_sid_from_file( fd, sid_file)) {
+                       DEBUG(0,("unable to read file %s. Error was %s\n",
+                                sid_file, strerror(errno) ));
+                       close(fd);
+                       return False;
+               }
+
+               /*
+                * JRA. Reversed the sense of this test now that I have
+                * actually done this test *personally*. One more reason
+                * to never trust third party information you have not
+                * independently verified.... sigh. JRA.
+                */
+
+               if(global_sam_sid.num_auths > 0 && global_sam_sid.sub_auths[0] == 0x21) {
+                       /*
+                        * Fix and re-write...
+                        */
+                       overwrite_bad_sid = True;
+                       global_sam_sid.sub_auths[0] = 21;
+                       DEBUG(5,("pdb_generate_sam_sid: Old (incorrect) sid id_auth of hex 21 \
+detected - re-writing to be decimal 21 instead.\n" ));
+                       sid_to_string(sid_string, &global_sam_sid);
+                       if(sys_lseek(fd, (SMB_OFF_T)0, SEEK_SET) != 0) {
+                               DEBUG(0,("unable to seek file file %s. Error was %s\n",
+                                        sid_file, strerror(errno) ));
+                               close(fd);
+                               return False;
+                       }
+               } else {
+                       close(fd);
+                       return True;
+               }
+       } else {
+               /*
+                * The file contains no data - we need to generate our
+                * own sid.
+                * Generate the new sid data & turn it into a string.
+                */
+               int i;
+               uchar raw_sid_data[12];
+               DOM_SID mysid;
+
+               memset((char *)&mysid, '\0', sizeof(DOM_SID));
+               mysid.sid_rev_num = 1;
+               mysid.id_auth[5] = 5;
+               mysid.num_auths = 0;
+               mysid.sub_auths[mysid.num_auths++] = 21;
+
+               generate_random_buffer( raw_sid_data, 12, True);
+               for( i = 0; i < 3; i++)
+                       mysid.sub_auths[mysid.num_auths++] = IVAL(raw_sid_data, i*4);
+
+               sid_to_string(sid_string, &mysid);
+       } 
+       
+       fstrcat(sid_string, "\n");
+       
+       /*
+        * Ensure our new SID is valid.
+        */
+       
+       if(!string_to_sid( &global_sam_sid, sid_string)) {
+               DEBUG(0,("unable to generate machine SID.\n"));
+               return False;
+       } 
   
-  /*
-   * At this point we have a blocking lock on the SID
-   * file - check if in the meantime someone else wrote
-   * SID data into the file. If so - they were here first,
-   * use their data.
-   */
-    
-  if(fstat( fd, &st) < 0) {
-    DEBUG(0,("generate_machine_sid: unable to stat file %s. Error was %s\n",
-             sid_file, strerror(errno) ));
-    close(fd);
-    return False;
-  } 
+       /*
+        * Do an exclusive blocking lock on the file.
+        */
+       
+       if(!do_file_lock( fd, 60, F_WRLCK)) {
+               DEBUG(0,("unable to lock file %s. Error was %s\n",
+                        sid_file, strerror(errno) ));
+               close(fd);
+               return False;
+       } 
+       if(!overwrite_bad_sid) {
+               /*
+                * At this point we have a blocking lock on the SID
+                * file - check if in the meantime someone else wrote
+                * SID data into the file. If so - they were here first,
+                * use their data.
+                */
+       
+               if(sys_fstat( fd, &st) < 0) {
+                       DEBUG(0,("unable to stat file %s. Error was %s\n",
+                                sid_file, strerror(errno) ));
+                       close(fd);
+                       return False;
+               } 
   
-  if(st.st_size > 0) {
-    /*
-     * Unlock as soon as possible to reduce
-     * contention on the exclusive lock.
-     */ 
-    do_file_lock( fd, 60, F_UNLCK);
-    
-    /*
-     * We have a valid SID - read it.
-     */
-    
-    if(!read_sid_from_file( fd, sid_file)) {
-      DEBUG(0,("generate_machine_sid: unable to read file %s. Error was %s\n",
-             sid_file, strerror(errno) ));
-      close(fd);
-      return False;
-    }
-    close(fd);
-    return True;
-  } 
-    
-  /*
-   * The file is still empty and we have an exlusive lock on it.
-   * Write out out SID data into the file.
-   */
-    
-  if(fchmod(fd, 0644) < 0) {
-    DEBUG(0,("generate_machine_sid: unable to set correct permissions on file %s. \
+               if(st.st_size > 0) {
+                       /*
+                        * Unlock as soon as possible to reduce
+                        * contention on the exclusive lock.
+                        */ 
+                       do_file_lock( fd, 60, F_UNLCK);
+               
+                       /*
+                        * We have a valid SID - read it.
+                        */
+               
+                       if(!read_sid_from_file( fd, sid_file)) {
+                               DEBUG(0,("unable to read file %s. Error was %s\n",
+                                        sid_file, strerror(errno) ));
+                               close(fd);
+                               return False;
+                       }
+                       close(fd);
+                       return True;
+               } 
+       }
+       
+       /*
+        * The file is still empty and we have an exlusive lock on it,
+        * or we're fixing an earlier mistake.
+        * Write out out SID data into the file.
+        */
+
+       /*
+        * Use chmod here as some (strange) UNIX's don't
+        * have fchmod. JRA.
+        */     
+
+       if(chmod(sid_file, 0644) < 0) {
+               DEBUG(0,("unable to set correct permissions on file %s. \
 Error was %s\n", sid_file, strerror(errno) ));
-    close(fd);
-    return False;
-  } 
-  
-  if(write( fd, sid_string, strlen(sid_string)) != strlen(sid_string)) {
-    DEBUG(0,("generate_machine_sid: unable to write file %s. Error was %s\n",
-          sid_file, strerror(errno) ));
-    close(fd);
-    return False;
-  } 
-  
-  /*
-   * Unlock & exit.
-   */
-    
-  do_file_lock( fd, 60, F_UNLCK);
-  close(fd);
-  return True;
+               do_file_lock( fd, 60, F_UNLCK);
+               close(fd);
+               return False;
+       } 
+       
+       if(write( fd, sid_string, strlen(sid_string)) != strlen(sid_string)) {
+               DEBUG(0,("unable to write file %s. Error was %s\n",
+                        sid_file, strerror(errno) ));
+               do_file_lock( fd, 60, F_UNLCK);
+               close(fd);
+               return False;
+       } 
+       
+       /*
+        * Unlock & exit.
+        */
+       
+       do_file_lock( fd, 60, F_UNLCK);
+       close(fd);
+       return True;
 }   
 
 /*******************************************************************
converts NT User RID to a UNIX uid.
Converts NT user RID to a UNIX uid.
  ********************************************************************/
-uid_t pdb_user_rid_to_uid(uint32 u_rid)
+
+uid_t pdb_user_rid_to_uid(uint32 user_rid)
 {
-       return (uid_t)(u_rid - 1000);
+       return (uid_t)(((user_rid & (~USER_RID_TYPE))- 1000)/RID_MULTIPLIER);
 }
 
 /*******************************************************************
converts NT Group RID to a UNIX uid.
Converts NT user RID to a UNIX gid.
  ********************************************************************/
-uid_t pdb_group_rid_to_uid(uint32 u_gid)
+
+gid_t pdb_user_rid_to_gid(uint32 user_rid)
 {
-       return (uid_t)(u_gid - 1000);
+       return (uid_t)(((user_rid & (~GROUP_RID_TYPE))- 1000)/RID_MULTIPLIER);
 }
 
 /*******************************************************************
  converts UNIX uid to an NT User RID.
  ********************************************************************/
-uint32 pdb_uid_to_user_rid(uint32 uid)
+
+uint32 pdb_uid_to_user_rid(uid_t uid)
 {
-       return (uint32)(uid + 1000);
+       return (((((uint32)uid)*RID_MULTIPLIER) + 1000) | USER_RID_TYPE);
 }
 
 /*******************************************************************
  converts NT Group RID to a UNIX uid.
  ********************************************************************/
-uint32 pdb_gid_to_group_rid(uint32 gid)
+
+uint32 pdb_gid_to_group_rid(gid_t gid)
+{
+  return (((((uint32)gid)*RID_MULTIPLIER) + 1000) | GROUP_RID_TYPE);
+}
+
+/*******************************************************************
+ Decides if a RID is a well known RID.
+ ********************************************************************/
+
+static BOOL pdb_rid_is_well_known(uint32 rid)
 {
-       return (uint32)(gid + 1000);
+  return (rid < 1000);
+}
+
+/*******************************************************************
+ Decides if a RID is a user or group RID.
+ ********************************************************************/
+  
+BOOL pdb_rid_is_user(uint32 rid)
+{
+  /* lkcl i understand that NT attaches an enumeration to a RID
+   * such that it can be identified as either a user, group etc
+   * type.  there are 5 such categories, and they are documented.
+   */
+   if(pdb_rid_is_well_known(rid)) {
+      /*
+       * The only well known user RIDs are DOMAIN_USER_RID_ADMIN
+       * and DOMAIN_USER_RID_GUEST.
+       */
+     if(rid == DOMAIN_USER_RID_ADMIN || rid == DOMAIN_USER_RID_GUEST)
+       return True;
+   } else if((rid & RID_TYPE_MASK) == USER_RID_TYPE) {
+     return True;
+   }
+   return False;
+}
+
+/*******************************************************************
+ Convert a rid into a name. Used in the lookup SID rpc.
+ ********************************************************************/
+
+BOOL lookup_local_rid(uint32 rid, char *name, uint8 *psid_name_use)
+{
+
+       BOOL is_user = pdb_rid_is_user(rid);
+
+       DEBUG(5,("lookup_local_rid: looking up %s RID %u.\n", is_user ? "user" :
+                       "group", (unsigned int)rid));
+
+       if(is_user) {
+               if(rid == DOMAIN_USER_RID_ADMIN) {
+                       pstring admin_users;
+                       char *p = admin_users;
+                       pstrcpy( admin_users, lp_domain_admin_users());
+                       if(!next_token(&p, name, NULL, sizeof(fstring)))
+                               fstrcpy(name, "Administrator");
+               } else if (rid == DOMAIN_USER_RID_GUEST) {
+                       pstring guest_users;
+                       char *p = guest_users;
+                       pstrcpy( guest_users, lp_domain_guest_users());
+                       if(!next_token(&p, name, NULL, sizeof(fstring)))
+                               fstrcpy(name, "Guest");
+               } else {
+                       uid_t uid = pdb_user_rid_to_uid(rid);
+                       struct passwd *pass = sys_getpwuid(uid);
+
+                       *psid_name_use = SID_NAME_USER;
+
+                       DEBUG(5,("lookup_local_rid: looking up uid %u %s\n", (unsigned int)uid,
+                               pass ? "succeeded" : "failed" ));
+
+                       if(!pass) {
+                               slprintf(name, sizeof(fstring)-1, "unix_user.%u", (unsigned int)uid);
+                               return True;
+                       }
+
+                       fstrcpy(name, pass->pw_name);
+
+                       DEBUG(5,("lookup_local_rid: found user %s for rid %u\n", name,
+                               (unsigned int)rid ));
+               }
+
+       } else {
+               gid_t gid = pdb_user_rid_to_gid(rid);
+               struct group *gr = getgrgid(gid);
+
+               *psid_name_use = SID_NAME_ALIAS;
+
+               DEBUG(5,("lookup_local_rid: looking up gid %u %s\n", (unsigned int)gid,
+                       gr ? "succeeded" : "failed" ));
+
+               if(!gr) {
+                       slprintf(name, sizeof(fstring)-1, "unix_group.%u", (unsigned int)gid);
+                       return True;
+               }
+
+               fstrcpy( name, gr->gr_name);
+
+               DEBUG(5,("lookup_local_rid: found group %s for rid %u\n", name,
+                       (unsigned int)rid ));
+       }
+
+       return True;
 }
 
+/*******************************************************************
+ Convert a name into a SID. Used in the lookup name rpc.
+ ********************************************************************/
+
+BOOL lookup_local_name(char *domain, char *user, DOM_SID *psid, uint8 *psid_name_use)
+{
+       extern DOM_SID global_sid_World_Domain;
+       struct passwd *pass = NULL;
+       DOM_SID local_sid;
+
+       sid_copy(&local_sid, &global_sam_sid);
+
+       if(!strequal(global_myname, domain) && !strequal(global_myworkgroup, domain))
+               return False;
+
+       /*
+        * Special case for MACHINE\Everyone. Map to the world_sid.
+        */
+
+       if(strequal(user, "Everyone")) {
+               sid_copy( psid, &global_sid_World_Domain);
+               sid_append_rid(psid, 0);
+               *psid_name_use = SID_NAME_ALIAS;
+               return True;
+       }
+
+       (void)map_username(user);
+
+       if(!(pass = Get_Pwnam(user, False))) {
+               /*
+                * Maybe it was a group ?
+                */
+               struct group *grp = getgrnam(user);
+
+               if(!grp)
+                       return False;
+
+               sid_append_rid( &local_sid, pdb_gid_to_group_rid(grp->gr_gid));
+               *psid_name_use = SID_NAME_ALIAS;
+       } else {
+
+               sid_append_rid( &local_sid, pdb_uid_to_user_rid(pass->pw_uid));
+               *psid_name_use = SID_NAME_USER;
+       }
+
+       sid_copy( psid, &local_sid);
+
+       return True;
+}