removed function names printed out in debug statements (duplicated in
[kai/samba.git] / source / passdb / passdb.c
index acc8d1c6093e838633a3a81e816fad070056f630..02f30b24c87a38331ecc6b8225f95245d7d17029 100644 (file)
@@ -35,10 +35,70 @@ DOM_SID global_machine_sid;
 /*
  * 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
+ *
  */
 
+#if 0
+static struct smb_passwd    *getPDBpwent        (void *vp)                              { return pdb_sam_to_smb(getPDB21pwent(vp)); } 
+static BOOL                  add_PDBpwd_entry   (struct smb_passwd *newpwd)             { return add_PDB21pwd_entry(pdb_smb_to_sam(newpwd)); } 
+static BOOL                  mod_PDBpwd_entry   (struct smb_passwd* pwd, BOOL override) { return mod_PDB21pwd_entry(pdb_smb_to_sam(pwd), override); } 
+static struct smb_passwd    *getPDBpwnam        (char *name)                            { return pdb_sam_to_smb(getPDB21pwnam(name)); } 
+static struct smb_passwd    *getPDBpwuid        (uid_t smb_userid)                      { return pdb_sam_to_smb(getPDB21pwuid(pdb_uid_to_user_rid(smb_userid))); } 
+
+static struct sam_passwd    *getPDB21pwent      (void *vp)                              { return pdb_smb_to_sam(getPDBpwent(vp)); } 
+static BOOL                  add_PDB21pwd_entry (struct sam_passwd *newpwd)             { return add_PDBpwd_entry(pdb_sam_to_smb(newpwd)); } 
+static BOOL                  mod_PDB21pwd_entry (struct sam_passwd* pwd, BOOL override) { return mod_PDBpwd_entry(pdb_sam_to_smb(pwd), override); } 
+static struct sam_passwd    *getPDB21pwnam      (char *name)                            { return pdb_smb_to_sam(getPDBpwnam(name)); } 
+static struct sam_passwd    *getPDB21pwrid      (uint32 rid)                            { return pdb_smb_to_sam(getPDBpwuid(pdb_user_rid_to_uid(rid))); } 
+static struct sam_passwd    *getPDB21pwuid      (uid_t uid)                             { return pdb_smb_to_sam(getPDBpwuid(uid)); } 
+
+static struct sam_disp_info *getPDBdispnam      (char *name)                            { return pdb_sam_to_dispinfo(getPDB21pwnam(name)); } 
+static struct sam_disp_info *getPDBdisprid      (uint32 rid)                            { return pdb_sam_to_dispinfo(getPDB21pwrid(rid)); } 
+static struct sam_disp_info *getPDBdispent      (void *vp)                              { return pdb_sam_to_dispinfo(getPDB21pwent(vp)); } 
+#endif /* 0 */
+
 static struct passdb_ops *pdb_ops;
 
+/***************************************************************
+ Initialize the password db operations.
+***************************************************************/
+
+BOOL initialize_password_db(void)
+{
+  if (pdb_ops)
+  {
+    return True;
+  }
+
+#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 
+
+  return (pdb_ops != NULL);
+}
+
 /*
  * Functions that return/manipulate a struct smb_passwd.
  */
@@ -53,14 +113,14 @@ struct smb_passwd *iterate_getsmbpwuid(uid_t smb_userid)
        struct smb_passwd *pwd = NULL;
        void *fp = NULL;
 
-       DEBUG(10, ("iterate_getsmbpwuid: search by smb_userid: %x\n", smb_userid));
+       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, ("iterate_getsmbpwuid: unable to open smb password database.\n"));
+               DEBUG(0, ("unable to open smb password database.\n"));
                return NULL;
        }
 
@@ -69,7 +129,7 @@ struct smb_passwd *iterate_getsmbpwuid(uid_t smb_userid)
 
        if (pwd != NULL)
        {
-               DEBUG(10, ("iterate_getsmbpwuid: found by smb_userid: %x\n", smb_userid));
+               DEBUG(10, ("found by smb_userid: %x\n", (int)smb_userid));
        }
 
        endsmbpwent(fp);
@@ -86,14 +146,14 @@ struct smb_passwd *iterate_getsmbpwnam(char *name)
        struct smb_passwd *pwd = NULL;
        void *fp = NULL;
 
-       DEBUG(10, ("iterate_getsmbpwnam: search by name: %s\n", name));
+       DEBUG(10, ("search by name: %s\n", name));
 
        /* Open the sam password file - not for update. */
        fp = startsmbpwent(False);
 
        if (fp == NULL)
        {
-               DEBUG(0, ("iterate_getsmbpwnam: unable to open smb password database.\n"));
+               DEBUG(0, ("unable to open smb password database.\n"));
                return NULL;
        }
 
@@ -102,37 +162,13 @@ struct smb_passwd *iterate_getsmbpwnam(char *name)
 
        if (pwd != NULL)
        {
-               DEBUG(10, ("iterate_getsmbpwnam: found by name: %s\n", name));
+               DEBUG(10, ("found by name: %s\n", name));
        }
 
        endsmbpwent(fp);
        return pwd;
 }
 
-/***************************************************************
- Initialize the password db operations.
-***************************************************************/
-
-BOOL initialize_password_db(void)
-{
-  if(pdb_ops)
-    return True;
-
-#ifdef USE_NISPLUS_DB
-  pdb_ops =  nisplus_initialize_password_db();
-#endif /* USE_NISPLUS_DB */
-
-#ifdef USE_LDAP_DB
-  pdb_ops = ldap_initialize_password_db();
-#endif /* USE_LDAP_DB */
-
-#ifdef USE_SMBPASS_DB
-  pdb_ops = file_initialize_password_db();
-#endif /* USE_SMBPASS_DB */
-
-  return (pdb_ops != NULL);
-}
-
 /***************************************************************
  Start to enumerate the smb or sam passwd list. Returns a void pointer
  to ensure no modification outside this module.
@@ -141,6 +177,7 @@ BOOL initialize_password_db(void)
  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.
+
 ****************************************************************/
 
 void *startsmbpwent(BOOL update)
@@ -169,39 +206,7 @@ void endsmbpwent(void *vp)
 
 struct smb_passwd *getsmbpwent(void *vp)
 {
-  return pdb_ops->getsmbpwent(vp);
-}
-
-/*************************************************************************
- Return the current position in the smb passwd list as an unsigned long.
- This must be treated as an opaque token.
-
- 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. 
-
- *************************************************************************/
-
-unsigned long getsmbpwpos(void *vp)
-{
-  return pdb_ops->getsmbpwpos(vp);
-}
-
-/*************************************************************************
- Set the current position in the smb passwd list from unsigned long.
- This must be treated as an opaque token.
-
- 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. 
-
- *************************************************************************/
-
-BOOL setsmbpwpos(void *vp, unsigned long tok)
-{
-  return pdb_ops->setsmbpwpos(vp, tok);
+       return pdb_ops->getsmbpwent(vp);
 }
 
 /************************************************************************
@@ -210,7 +215,7 @@ BOOL setsmbpwpos(void *vp, unsigned long tok)
 
 BOOL add_smbpwd_entry(struct smb_passwd *newpwd)
 {
-  return pdb_ops->add_smbpwd_entry(newpwd);
+       return pdb_ops->add_smbpwd_entry(newpwd);
 }
 
 /************************************************************************
@@ -224,7 +229,7 @@ BOOL add_smbpwd_entry(struct smb_passwd *newpwd)
 
 BOOL mod_smbpwd_entry(struct smb_passwd* pwd, BOOL override)
 {
-  return pdb_ops->mod_smbpwd_entry(pwd, override);
+       return pdb_ops->mod_smbpwd_entry(pwd, override);
 }
 
 /************************************************************************
@@ -233,7 +238,7 @@ BOOL mod_smbpwd_entry(struct smb_passwd* pwd, BOOL override)
 
 struct smb_passwd *getsmbpwnam(char *name)
 {
-  return pdb_ops->getsmbpwnam(name);
+       return pdb_ops->getsmbpwnam(name);
 }
 
 /************************************************************************
@@ -242,7 +247,7 @@ struct smb_passwd *getsmbpwnam(char *name)
 
 struct smb_passwd *getsmbpwuid(uid_t smb_userid)
 {
-  return pdb_ops->getsmbpwuid(smb_userid);
+       return pdb_ops->getsmbpwuid(smb_userid);
 }
 
 /*
@@ -259,23 +264,64 @@ struct sam_passwd *iterate_getsam21pwnam(char *name)
        struct sam_passwd *pwd = NULL;
        void *fp = NULL;
 
-       DEBUG(10, ("iterate_getsam21pwnam: search by name: %s\n", name));
+       DEBUG(10, ("search by name: %s\n", name));
 
        /* Open the smb password database - not for update. */
        fp = startsmbpwent(False);
 
        if (fp == NULL)
        {
-               DEBUG(0, ("iterate_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))
-      ;
+       {
+               DEBUG(10, ("iterate: %s 0x%x\n", pwd->smb_name, pwd->user_rid));
+       }
 
        if (pwd != NULL)
        {
-               DEBUG(10, ("iterate_getsam21pwnam: found by name: %s\n", name));
+               DEBUG(10, ("found by name: %s\n", name));
+       }
+
+       endsmbpwent(fp);
+       return pwd;
+}
+
+/************************************************************************
+ Utility function to search sam passwd by rid.  use this if your database
+ does not have search facilities.
+
+ search capability by both rid and uid are needed as the rid <-> uid
+ mapping may be non-monotonic.  
+
+*************************************************************************/
+
+struct sam_passwd *iterate_getsam21pwrid(uint32 rid)
+{
+       struct sam_passwd *pwd = NULL;
+       void *fp = NULL;
+
+       DEBUG(10, ("search by rid: %x\n", rid));
+
+       /* Open the smb password file - not for update. */
+       fp = startsmbpwent(False);
+
+       if (fp == NULL)
+       {
+               DEBUG(0, ("unable to open sam password database.\n"));
+               return NULL;
+       }
+
+       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, ("found by user_rid: %x\n", rid));
        }
 
        endsmbpwent(fp);
@@ -285,21 +331,25 @@ struct sam_passwd *iterate_getsam21pwnam(char *name)
 /************************************************************************
  Utility function to search sam passwd by uid.  use this if your database
  does not have search facilities.
+
+ search capability by both rid and uid are needed as the rid <-> uid
+ mapping may be non-monotonic.  
+
 *************************************************************************/
 
-struct sam_passwd *iterate_getsam21pwuid(uint32 uid)
+struct sam_passwd *iterate_getsam21pwuid(uid_t uid)
 {
        struct sam_passwd *pwd = NULL;
        void *fp = NULL;
 
-       DEBUG(10, ("iterate_getsam21pwuid: search by uid: %x\n", uid));
+       DEBUG(10, ("search by uid: %x\n", (int)uid));
 
        /* Open the smb password file - not for update. */
        fp = startsmbpwent(False);
 
        if (fp == NULL)
        {
-               DEBUG(0, ("iterate_getsam21pwuid: unable to open sam password database.\n"));
+               DEBUG(0, ("unable to open sam password database.\n"));
                return NULL;
        }
 
@@ -308,7 +358,7 @@ struct sam_passwd *iterate_getsam21pwuid(uint32 uid)
 
        if (pwd != NULL)
        {
-               DEBUG(10, ("iterate_getsam21pwuid: found by smb_userid: %x\n", uid));
+               DEBUG(10, ("found by smb_userid: %x\n", (int)uid));
        }
 
        endsmbpwent(fp);
@@ -316,12 +366,11 @@ struct sam_passwd *iterate_getsam21pwuid(uint32 uid)
 }
 
 /*************************************************************************
- Routine to return the next entry in the sam passwd list.
+ Routine to return a display info structure, by rid
  *************************************************************************/
-
-struct sam_disp_info *getsamdispent(void *vp)
+struct sam_disp_info *getsamdisprid(uint32 rid)
 {
-  return pdb_sam_to_dispinfo(pdb_ops->getsam21pwent(vp));
+       return pdb_ops->getsamdisprid(rid);
 }
 
 /*************************************************************************
@@ -330,30 +379,7 @@ struct sam_disp_info *getsamdispent(void *vp)
 
 struct sam_passwd *getsam21pwent(void *vp)
 {
-  return pdb_ops->getsam21pwent(vp);
-}
-
-/************************************************************************
- Routine to add an entry to the sam passwd file.
-*************************************************************************/
-
-BOOL add_sam21pwd_entry(struct sam_passwd *newpwd)
-{
-  return pdb_ops->add_sam21pwd_entry(newpwd);
-}
-
-/************************************************************************
- Routine to search the sam passwd database 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)
-{
-  return pdb_ops->mod_sam21pwd_entry(pwd, override);
+       return pdb_ops->getsam21pwent(vp);
 }
 
 
@@ -363,16 +389,16 @@ BOOL mod_sam21pwd_entry(struct sam_passwd* pwd, BOOL override)
 
 struct sam_passwd *getsam21pwnam(char *name)
 {
-  return pdb_ops->getsam21pwnam(name);
+       return pdb_ops->getsam21pwnam(name);
 }
 
 /************************************************************************
- Routine to search sam passwd by uid.  
+ Routine to search sam passwd by rid.  
 *************************************************************************/
 
-struct sam_passwd *getsam21pwuid(uint32 uid)
+struct sam_passwd *getsam21pwrid(uint32 rid)
 {
-  return pdb_ops->getsam21pwuid(uid);
+       return pdb_ops->getsam21pwrid(rid);
 }
 
 
@@ -389,7 +415,7 @@ struct sam_passwd *getsam21pwuid(uint32 uid)
  initialises a struct sam_disp_info.
  **************************************************************/
 
-void pdb_init_dispinfo(struct sam_disp_info *user)
+static void pdb_init_dispinfo(struct sam_disp_info *user)
 {
        if (user == NULL) return;
        bzero(user, sizeof(*user));
@@ -462,95 +488,10 @@ 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;
-
-       if (user == NULL) return NULL;
-
-       pdb_init_sam(&pw_buf);
-
-       pw_buf.smb_userid         = user->smb_userid;
-       pw_buf.smb_name           = user->smb_name;
-       pw_buf.smb_passwd         = user->smb_passwd;
-       pw_buf.smb_nt_passwd      = user->smb_nt_passwd;
-       pw_buf.acct_ctrl          = user->acct_ctrl;
-       pw_buf.pass_last_set_time = user->pass_last_set_time;
-
-       return &pw_buf;
-}
-
-#if 0
-
-  COMMENTED OUT UNTIL SOMETHING ACTUALLY USES THEM. JRA.
-
-/*******************************************************************
- gets password-database-format time from a string.
- ********************************************************************/
-
-static time_t get_time_from_string(char *p)
-{
-       int i;
-
-       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;
-}
-
-/*******************************************************************
- 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;
-}
-
-
-/*******************************************************************
- 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 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);
-}
-
-#endif /* 0 */
 
 /**********************************************************
  Encode the account control bits into a string.
  **********************************************************/
-
 char *pdb_encode_acct_ctrl(uint16 acct_ctrl)
 {
   static fstring acct_str;
@@ -628,31 +569,35 @@ uint16 pdb_decode_acct_ctrl(char *p)
 }
 
 /*************************************************************
- Routine to get the next 32 hex characters and turn them
+ 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, char *pwd)
 {
-  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);
+       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);
 }
 
 /*******************************************************************
@@ -668,11 +613,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()))
        {
@@ -703,8 +648,10 @@ 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;
   }
@@ -715,7 +662,7 @@ 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"));
+    DEBUG(0,("unable to generate machine SID.\n"));
     return False;
   }
 
@@ -726,195 +673,194 @@ 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.
-   */
+       int fd;
+       char *p;
+       pstring sid_file;
+       fstring sid_string;
+       SMB_STRUCT_STAT st;
+       uchar raw_sid_data[12];
+
+       pstrcpy(sid_file, lp_smb_passwd_file());
+       p = strrchr(sid_file, '/');
+       if(p != NULL) {
+               *++p = '\0';
+       }
+
+       if (!directory_exist(sid_file, NULL)) {
+               if (dos_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");
     
-  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((fd = 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;
+       } 
   
-  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;
-  } 
+       /*
+        * 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;
+       } 
   
-  /*
-   * 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);
-    
-    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(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;
+               }
+               close(fd);
+               return True;
+       } 
   
-  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;
-  } 
+       /*
+        * 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);
+               
+               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);
+               }
+       } 
+       
+       fstrcat(sid_string, "\n");
+       
+       /*
+        * Ensure our new SID is valid.
+        */
+       
+       if(!string_to_sid( &global_machine_sid, sid_string)) {
+               DEBUG(0,("unable to generate machine SID.\n"));
+               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;
-  } 
+       /*
+        * 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;
+       
   
-  /*
-   * 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;
-  } 
+       /*
+        * 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.
+        * Write out out SID data into the file.
+        */
+       
+       if(fchmod(fd, 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;
+               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) ));
+               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 UNIX uid to an NT User RID.
  ********************************************************************/
 
-uint32 pdb_user_rid_to_uid(uint32 u_rid)
+uint32 pdb_uid_to_user_rid(uid_t uid)
 {
-       return (u_rid - 1000);
+       return (((((uint32)uid)*RID_MULTIPLIER) + 1000) | USER_RID_TYPE);
 }
 
 /*******************************************************************
  converts NT Group RID to a UNIX uid.
  ********************************************************************/
 
-uint32 pdb_group_rid_to_gid(uint32 u_gid)
+uint32 pdb_gid_to_group_rid(gid_t gid)
 {
-       return (u_gid - 1000);
+  return (((((uint32)gid)*RID_MULTIPLIER) + 1000) | GROUP_RID_TYPE);
 }
 
 /*******************************************************************
converts UNIX uid to an NT User RID.
Decides if a RID is a well known RID.
  ********************************************************************/
 
-uint32 pdb_uid_to_user_rid(uint32 uid)
+static BOOL pdb_rid_is_well_known(uint32 rid)
 {
-       return (uint32)(uid + 1000);
-}
-
-/*******************************************************************
- converts NT Group RID to a UNIX uid.
- ********************************************************************/
-
-uint32 pdb_gid_to_group_rid(uint32 gid)
-{
-       return (uint32)(gid + 1000);
+  return (rid < 1000);
 }
 
 /*******************************************************************
@@ -923,6 +869,19 @@ uint32 pdb_gid_to_group_rid(uint32 gid)
   
 BOOL pdb_rid_is_user(uint32 rid)
 {
-  /* Punt for now - we need to look at the encoding here. JRA. */
-  return True;
+  /* 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;
 }