Makefile.in: Fixed bug with continuation line causing proto to fail.
[samba.git] / source / passdb / passdb.c
index 5bb20fce982709f0f0029252808811168341efca..15b2e8ed50803152d2dccac4d50a422918bfcc27 100644 (file)
@@ -88,17 +88,13 @@ BOOL initialize_password_db(void)
     return True;
   }
 
-#ifdef USE_NISPLUS_DB
+#ifdef WITH_NISPLUS
   pdb_ops =  nisplus_initialize_password_db();
-#endif /* USE_NISPLUS_DB */
-
-#ifdef USE_LDAP_DB
+#elif defined(WITH_LDAP)
   pdb_ops = ldap_initialize_password_db();
-#endif /* USE_LDAP_DB */
-
-#ifdef USE_SMBPASS_DB
+#else 
   pdb_ops = file_initialize_password_db();
-#endif /* USE_SMBPASS_DB */
+#endif 
 
   return (pdb_ops != NULL);
 }
@@ -117,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;
        }
 
@@ -133,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);
@@ -150,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;
        }
 
@@ -166,7 +162,7 @@ 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);
@@ -213,38 +209,6 @@ 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);
-}
-
 /************************************************************************
  Routine to add an entry to the smb passwd file.
 *************************************************************************/
@@ -300,23 +264,25 @@ 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);
@@ -337,23 +303,25 @@ struct sam_passwd *iterate_getsam21pwrid(uint32 rid)
        struct sam_passwd *pwd = NULL;
        void *fp = NULL;
 
-       DEBUG(10, ("iterate_getsam21pwrid: search by rid: %x\n", rid));
+       DEBUG(10, ("search by rid: %x\n", rid));
 
        /* Open the smb password file - not for update. */
        fp = startsmbpwent(False);
 
        if (fp == NULL)
        {
-               DEBUG(0, ("iterate_getsam21pwrid: unable to open sam password database.\n"));
+               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, ("iterate_getsam21pwrid: found by user_rid: %x\n", rid));
+               DEBUG(10, ("found by user_rid: %x\n", rid));
        }
 
        endsmbpwent(fp);
@@ -374,14 +342,14 @@ 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;
        }
 
@@ -390,21 +358,13 @@ struct sam_passwd *iterate_getsam21pwuid(uid_t 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);
        return pwd;
 }
 
-/*************************************************************************
- Routine to return a display info structure, by name
- *************************************************************************/
-struct sam_disp_info *getsamdispnam(char *name)
-{
-       return pdb_ops->getsamdispnam(name);
-}
-
 /*************************************************************************
  Routine to return a display info structure, by rid
  *************************************************************************/
@@ -413,14 +373,6 @@ struct sam_disp_info *getsamdisprid(uint32 rid)
        return pdb_ops->getsamdisprid(rid);
 }
 
-/*************************************************************************
- Routine to return the next entry in the sam passwd list.
- *************************************************************************/
-struct sam_disp_info *getsamdispent(void *vp)
-{
-       return pdb_ops->getsamdispent(vp);
-}
-
 /*************************************************************************
  Routine to return the next entry in the sam passwd list.
  *************************************************************************/
@@ -430,29 +382,6 @@ 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);
-}
-
 
 /************************************************************************
  Routine to search sam passwd by name.
@@ -472,17 +401,6 @@ struct sam_passwd *getsam21pwrid(uint32 rid)
        return pdb_ops->getsam21pwrid(rid);
 }
 
-/************************************************************************
- Routine to search sam passwd by uid.  
-*************************************************************************/
-
-struct sam_passwd *getsam21pwuid(uid_t uid)
-{
-       return pdb_ops->getsam21pwuid(uid);
-}
-
-
-
 
 /**********************************************************
  **********************************************************
@@ -497,7 +415,7 @@ struct sam_passwd *getsam21pwuid(uid_t 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));
@@ -570,147 +488,38 @@ 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;
-}
-
-
-/*******************************************************************
- 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 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);
-}
+/**********************************************************
+ Encode the account control bits into a string.
+ length = length of string to encode into (including terminating
+ null). length *MUST BE MORE THAN 2* !
+ **********************************************************/
 
-/*******************************************************************
- sets kickoff time
- ********************************************************************/
-void pdb_set_kickoff_time(char *p, int max_len, time_t t)
+char *pdb_encode_acct_ctrl(uint16 acct_ctrl, size_t length)
 {
-       set_time_in_string(p, max_len, "KOT", t);
-}
+  static fstring acct_str;
+  size_t i = 0;
 
-/*******************************************************************
- 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);
-}
+  acct_str[i++] = '[';
 
-/*******************************************************************
- 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);
-}
+  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';
 
-/*******************************************************************
- 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);
-}
+  for ( ; i < length - 2 ; i++ ) { acct_str[i] = ' '; }
 
-/**********************************************************
- Encode the account control bits into a string.
- **********************************************************/
+  i = length - 2;
+  acct_str[i++] = ']';
+  acct_str[i++] = '\0';
 
-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;
 }     
 
@@ -738,15 +547,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). */ } 
@@ -756,7 +558,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': 
@@ -800,32 +602,6 @@ BOOL pdb_gethexpwd(char *p, char *pwd)
        return (True);
 }
 
-/*************************************************************
- Routine to set 32 hex password characters from a 16 byte array.
-**************************************************************/
-void pdb_sethexpwd(char *p, char *pwd, uint16 acct_ctrl)
-{
-       if (pwd != NULL)
-       {
-               int i;
-               for (i = 0; i < 16; i++)
-               {
-                       slprintf(&p[i*2], 33, "%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);
-               }
-       }
-}
-
 /*******************************************************************
  Group and User RID username mapping function
  ********************************************************************/
@@ -839,11 +615,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()))
        {
@@ -874,8 +650,10 @@ static BOOL read_sid_from_file(int fd, char *sid_file)
 {   
   fstring fline;
     
+  memset(fline, '\0', sizeof(fline));
+
   if(read(fd, fline, sizeof(fline) -1 ) < 0) {
-    DEBUG(0,("read_sid_from_file: unable to read file %s. Error was %s\n",
+    DEBUG(0,("unable to read file %s. Error was %s\n",
            sid_file, strerror(errno) ));
     return False;
   }
@@ -886,7 +664,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;
   }
 
@@ -897,195 +675,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.
  ********************************************************************/
 
-uid_t pdb_user_rid_to_uid(uint32 u_rid)
+uint32 pdb_uid_to_user_rid(uid_t uid)
 {
-       return (uid_t)(u_rid - 1000);
+       return (((((uint32)uid)*RID_MULTIPLIER) + 1000) | USER_RID_TYPE);
 }
 
 /*******************************************************************
  converts NT Group RID to a UNIX uid.
  ********************************************************************/
 
-gid_t pdb_group_rid_to_gid(uint32 g_rid)
-{
-       return (gid_t)(g_rid - 1000);
-}
-
-/*******************************************************************
- converts UNIX uid to an NT User RID.
- ********************************************************************/
-
-uint32 pdb_uid_to_user_rid(uid_t uid)
+uint32 pdb_gid_to_group_rid(gid_t gid)
 {
-       return (uint32)(uid + 1000);
+  return (((((uint32)gid)*RID_MULTIPLIER) + 1000) | GROUP_RID_TYPE);
 }
 
 /*******************************************************************
converts NT Group RID to a UNIX uid.
Decides if a RID is a well known RID.
  ********************************************************************/
 
-uint32 pdb_gid_to_group_rid(gid_t gid)
+static BOOL pdb_rid_is_well_known(uint32 rid)
 {
-       return (uint32)(gid + 1000);
+  return (rid < 1000);
 }
 
 /*******************************************************************
@@ -1094,10 +871,19 @@ uint32 pdb_gid_to_group_rid(gid_t gid)
   
 BOOL pdb_rid_is_user(uint32 rid)
 {
-  /* Punt for now - we need to look at the encoding here. JRA. */
   /* 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.
    */
-  return True;
+   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;
 }