trying to get HEAD building again. If you want the code
[gd/samba-autobuild/.git] / source3 / lib / username.c
index 074e6c8992dd81691359e4d5d1dc0746ff382964..98b8f33aae30990e5b245febdec17903150b310d 100644 (file)
@@ -1,8 +1,8 @@
 /* 
-   Unix SMB/Netbios implementation.
-   Version 1.9.
+   Unix SMB/CIFS implementation.
    Username handling
    Copyright (C) Andrew Tridgell 1992-1998
+   Copyright (C) Jeremy Allison 1997-2001.
    
    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
 static struct passwd *uname_string_combinations(char *s, struct passwd * (*fn) (const char *), int N);
 static struct passwd *uname_string_combinations2(char *s, int offset, struct passwd * (*fn) (const char *), int N);
 
+/*****************************************************************
+ Check if a user or group name is local (this is a *local* name for
+ *local* people, there's nothing for you here...).
+*****************************************************************/
+
+static BOOL name_is_local(const char *name)
+{
+       return !(strchr_m(name, *lp_winbind_separator()));
+}
+
+/*****************************************************************
+ Splits passed user or group name to domain and user/group name parts
+ Returns True if name was splitted and False otherwise.
+*****************************************************************/
+
+BOOL split_domain_and_name(const char *name, char *domain, char* username)
+{
+       char *p = strchr(name,*lp_winbind_separator());
+       
+       
+       /* Parse a string of the form DOMAIN/user into a domain and a user */
+       DEBUG(10,("split_domain_and_name: checking whether name |%s| local or not\n", name));
+       
+       if (p) {
+               fstrcpy(username, p+1);
+               fstrcpy(domain, name);
+               domain[PTR_DIFF(p, name)] = 0;
+       } else if (lp_winbind_use_default_domain()) {
+               fstrcpy(username, name);
+               fstrcpy(domain, lp_workgroup());
+       } else {
+               return False;
+       }
+
+       DEBUG(10,("split_domain_and_name: all is fine, domain is |%s| and name is |%s|\n", domain, username));
+       return True;
+}
+
 /****************************************************************************
  Get a users home directory.
 ****************************************************************************/
 
 char *get_user_home_dir(const char *user)
 {
-  static struct passwd *pass;
+       static struct passwd *pass;
 
-  pass = Get_Pwnam(user);
+       /* Ensure the user exists. */
 
-  if (!pass) return(NULL);
-  return(pass->pw_dir);      
-}
+       pass = Get_Pwnam(user);
 
+       if (!pass)
+               return(NULL);
+       /* Return home directory from struct passwd. */
+
+       return(pass->pw_dir);      
+}
 
 /*******************************************************************
  Map a username from a dos name to a unix name by looking in the username
@@ -53,138 +95,119 @@ char *get_user_home_dir(const char *user)
 
 BOOL map_username(char *user)
 {
-  static BOOL initialised=False;
-  static fstring last_from,last_to;
-  XFILE *f;
-  char *mapfile = lp_username_map();
-  char *s;
-  pstring buf;
-  BOOL mapped_user = False;
-
-  if (!*user)
-    return False;
-
-  if (!*mapfile)
-    return False;
-
-  if (!initialised) {
-    *last_from = *last_to = 0;
-    initialised = True;
-  }
-
-  if (strequal(user,last_to))
-    return False;
-
-  if (strequal(user,last_from)) {
-    DEBUG(3,("Mapped user %s to %s\n",user,last_to));
-    fstrcpy(user,last_to);
-    return True;
-  }
+       static BOOL initialised=False;
+       static fstring last_from,last_to;
+       XFILE *f;
+       char *mapfile = lp_username_map();
+       char *s;
+       pstring buf;
+       BOOL mapped_user = False;
+
+       if (!*user)
+               return False;
+
+       if (!*mapfile)
+               return False;
+
+       if (!initialised) {
+               *last_from = *last_to = 0;
+               initialised = True;
+       }
+
+       if (strequal(user,last_to))
+               return False;
+
+       if (strequal(user,last_from)) {
+               DEBUG(3,("Mapped user %s to %s\n",user,last_to));
+               fstrcpy(user,last_to);
+               return True;
+       }
   
-  f = x_fopen(mapfile,O_RDONLY, 0);
-  if (!f) {
-    DEBUG(0,("can't open username map %s. Error %s\n",mapfile, strerror(errno) ));
-    return False;
-  }
-
-  DEBUG(4,("Scanning username map %s\n",mapfile));
-
-  while((s=fgets_slash(buf,sizeof(buf),f))!=NULL) {
-    char *unixname = s;
-    char *dosname = strchr_m(unixname,'=');
-    char **dosuserlist;
-    BOOL return_if_mapped = False;
-
-    if (!dosname)
-      continue;
-
-    *dosname++ = 0;
-
-    while (isspace(*unixname))
-      unixname++;
-    if ('!' == *unixname) {
-      return_if_mapped = True;
-      unixname++;
-      while (*unixname && isspace(*unixname))
-        unixname++;
-    }
-    
-    if (!*unixname || strchr_m("#;",*unixname))
-      continue;
-
-    {
-      int l = strlen(unixname);
-      while (l && isspace(unixname[l-1])) {
-        unixname[l-1] = 0;
-        l--;
-      }
-    }
-
-    dosuserlist = lp_list_make(dosname);
-    if (!dosuserlist) {
-       DEBUG(0,("Unable to build user list\n"));
-       return False;
-    }
-
-    if (strchr_m(dosname,'*') || user_in_list(user, dosuserlist)) {
-      DEBUG(3,("Mapped user %s to %s\n",user,unixname));
-      mapped_user = True;
-      fstrcpy(last_from,user);
-      sscanf(unixname,"%s",user);
-      fstrcpy(last_to,user);
-      if(return_if_mapped) {
-       lp_list_free (&dosuserlist);
-        x_fclose(f);
-        return True;
-      }
-    }
+       f = x_fopen(mapfile,O_RDONLY, 0);
+       if (!f) {
+               DEBUG(0,("can't open username map %s. Error %s\n",mapfile, strerror(errno) ));
+               return False;
+       }
+
+       DEBUG(4,("Scanning username map %s\n",mapfile));
+
+       while((s=fgets_slash(buf,sizeof(buf),f))!=NULL) {
+               char *unixname = s;
+               char *dosname = strchr_m(unixname,'=');
+               char **dosuserlist;
+               BOOL return_if_mapped = False;
+
+               if (!dosname)
+                       continue;
+
+               *dosname++ = 0;
+
+               while (isspace((int)*unixname))
+                       unixname++;
+
+               if ('!' == *unixname) {
+                       return_if_mapped = True;
+                       unixname++;
+                       while (*unixname && isspace((int)*unixname))
+                               unixname++;
+               }
     
-    lp_list_free (&dosuserlist);
-  }
+               if (!*unixname || strchr_m("#;",*unixname))
+                       continue;
+
+               {
+                       int l = strlen(unixname);
+                       while (l && isspace((int)unixname[l-1])) {
+                               unixname[l-1] = 0;
+                               l--;
+                       }
+               }
 
-  x_fclose(f);
+               dosuserlist = str_list_make(dosname, NULL);
+               if (!dosuserlist) {
+                       DEBUG(0,("Unable to build user list\n"));
+                       return False;
+               }
 
-  /*
-   * Setup the last_from and last_to as an optimization so 
-   * that we don't scan the file again for the same user.
-   */
-  fstrcpy(last_from,user);
-  fstrcpy(last_to,user);
+               if (strchr_m(dosname,'*') || user_in_list(user, (const char **)dosuserlist, NULL, 0)) {
+                       DEBUG(3,("Mapped user %s to %s\n",user,unixname));
+                       mapped_user = True;
+                       fstrcpy(last_from,user);
+                       sscanf(unixname,"%s",user);
+                       fstrcpy(last_to,user);
+                       if(return_if_mapped) {
+                               str_list_free (&dosuserlist);
+                               x_fclose(f);
+                               return True;
+                       }
+               }
+    
+               str_list_free (&dosuserlist);
+       }
 
-  return mapped_user;
-}
+       x_fclose(f);
 
-/****************************************************************************
- Get_Pwnam wrapper
-****************************************************************************/
+       /*
+        * Setup the last_from and last_to as an optimization so 
+        * that we don't scan the file again for the same user.
+        */
+       fstrcpy(last_from,user);
+       fstrcpy(last_to,user);
 
-static struct passwd *_Get_Pwnam(const char *s)
-{
-  struct passwd *ret;
-
-  ret = sys_getpwnam(s);
-  if (ret) {
-#ifdef HAVE_GETPWANAM
-    struct passwd_adjunct *pwret;
-    pwret = getpwanam(s);
-    if (pwret && pwret->pwa_passwd) {
-      pstrcpy(ret->pw_passwd,pwret->pwa_passwd);
-    }
-#endif
-  }
-
-  return(ret);
+       return mapped_user;
 }
 
-
-/*
- * A wrapper for getpwnam().  The following variations are tried:
+/****************************************************************************
+ * A wrapper for sys_getpwnam().  The following variations are tried:
  *   - as transmitted
  *   - in all lower case if this differs from transmitted
  *   - in all upper case if this differs from transmitted
  *   - using lp_usernamelevel() for permutations.
- */
-struct passwd *Get_Pwnam_internals(const char *user, char *user2)
+****************************************************************************/
+
+static struct passwd *Get_Pwnam_ret = NULL;
+
+static struct passwd *Get_Pwnam_internals(const char *user, char *user2)
 {
        struct passwd *ret = NULL;
 
@@ -196,58 +219,58 @@ struct passwd *Get_Pwnam_internals(const char *user, char *user2)
 
        /* Try in all lower case first as this is the most 
           common case on UNIX systems */
-       strlower(user2);
+       strlower_m(user2);
        DEBUG(5,("Trying _Get_Pwnam(), username as lowercase is %s\n",user2));
-       ret = _Get_Pwnam(user2);
+       ret = getpwnam_alloc(user2);
        if(ret)
                goto done;
 
        /* Try as given, if username wasn't originally lowercase */
-       if(strcmp(user,user2) != 0) {
-               DEBUG(5,("Trying _Get_Pwnam(), username as given is %s\n",user));
-               ret = _Get_Pwnam(user);
+       if(strcmp(user, user2) != 0) {
+               DEBUG(5,("Trying _Get_Pwnam(), username as given is %s\n", user));
+               ret = getpwnam_alloc(user);
                if(ret)
                        goto done;
-       }       
+       }
 
        /* Try as uppercase, if username wasn't originally uppercase */
-       strupper(user2);
-       if(strcmp(user,user2) != 0) {
-               DEBUG(5,("Trying _Get_Pwnam(), username as uppercase is %s\n",user2));
-               ret = _Get_Pwnam(user2);
+       strupper_m(user2);
+       if(strcmp(user, user2) != 0) {
+               DEBUG(5,("Trying _Get_Pwnam(), username as uppercase is %s\n", user2));
+               ret = getpwnam_alloc(user2);
                if(ret)
                        goto done;
        }
 
        /* Try all combinations up to usernamelevel */
-       strlower(user2);
-       DEBUG(5,("Checking combinations of %d uppercase letters in %s\n",lp_usernamelevel(),user2));
-       ret = uname_string_combinations(user2, _Get_Pwnam, lp_usernamelevel());
+       strlower_m(user2);
+       DEBUG(5,("Checking combinations of %d uppercase letters in %s\n", lp_usernamelevel(), user2));
+       ret = uname_string_combinations(user2, getpwnam_alloc, lp_usernamelevel());
 
 done:
-       DEBUG(5,("Get_Pwnam %s find a valid username!\n",ret ? "did":"didn't"));
-       return ret;
-}
+       DEBUG(5,("Get_Pwnam_internals %s find user [%s]!\n",ret ? "did":"didn't", user));
 
-/****************************************************************************
- Get_Pwnam wrapper for modification.
-  NOTE: This can potentially modify 'user'! 
-****************************************************************************/
+       /* This call used to just return the 'passwd' static buffer.
+          This could then have accidental reuse implications, so 
+          we now malloc a copy, and free it in the next use.
 
-struct passwd *Get_Pwnam_Modify(char *user)
-{
-       fstring user2;
-       struct passwd *ret;
+          This should cause the (ab)user to segfault if it 
+          uses an old struct. 
+          
+          This is better than useing the wrong data in security
+          critical operations.
 
-       fstrcpy(user2, user);
+          The real fix is to make the callers free the returned 
+          malloc'ed data.
+       */
 
-       ret = Get_Pwnam_internals(user, user2);
+       if (Get_Pwnam_ret) {
+               passwd_free(&Get_Pwnam_ret);
+       }
        
-       /* If caller wants the modified username, ensure they get it  */
-       fstrcpy(user,user2);
+       Get_Pwnam_ret = ret;
 
-       /* We can safely assume ret is NULL if none of the above succeed */
-       return(ret);  
+       return ret;
 }
 
 /****************************************************************************
@@ -262,10 +285,11 @@ struct passwd *Get_Pwnam(const char *user)
 
        fstrcpy(user2, user);
 
+       DEBUG(5,("Finding user %s\n", user));
+
        ret = Get_Pwnam_internals(user, user2);
        
-       /* We can safely assume ret is NULL if none of the above succeed */
-       return(ret);  
+       return ret;  
 }
 
 /****************************************************************************
@@ -275,73 +299,101 @@ struct passwd *Get_Pwnam(const char *user)
 static BOOL user_in_netgroup_list(const char *user, const char *ngname)
 {
 #ifdef HAVE_NETGROUP
-  static char *mydomain = NULL;
-  if (mydomain == NULL)
-    yp_get_default_domain(&mydomain);
-
-  if(mydomain == NULL) {
-    DEBUG(5,("Unable to get default yp domain\n"));
-  } else {
-    DEBUG(5,("looking for user %s of domain %s in netgroup %s\n",
-          user, mydomain, ngname));
-    DEBUG(5,("innetgr is %s\n",
-          innetgr(ngname, NULL, user, mydomain)
-          ? "TRUE" : "FALSE"));
-
-    if (innetgr(ngname, NULL, user, mydomain))
-      return (True);
-  }
+       static char *mydomain = NULL;
+       if (mydomain == NULL)
+               yp_get_default_domain(&mydomain);
+
+       if(mydomain == NULL) {
+               DEBUG(5,("Unable to get default yp domain\n"));
+               return False;
+       }
+
+       DEBUG(5,("looking for user %s of domain %s in netgroup %s\n",
+               user, mydomain, ngname));
+       DEBUG(5,("innetgr is %s\n", innetgr(ngname, NULL, user, mydomain)
+               ? "TRUE" : "FALSE"));
+
+       if (innetgr(ngname, NULL, user, mydomain))
+               return (True);
 #endif /* HAVE_NETGROUP */
-  return False;
+       return False;
 }
 
 /****************************************************************************
  Check if a user is in a winbind group.
 ****************************************************************************/
   
-static BOOL user_in_winbind_group_list(char *user,char *gname, BOOL *winbind_answered)
+static BOOL user_in_winbind_group_list(const char *user, const char *gname, BOOL *winbind_answered)
 {
-       int num_groups;
        int i;
-       gid_t *groups = NULL;
-       gid_t gid;
+       gid_t gid, gid_low, gid_high;
        BOOL ret = False;
+       static gid_t *groups = NULL;
+       static int num_groups = 0;
+       static fstring last_user = "";
  
        *winbind_answered = False;
  
-       /*
-        * Get the gid's that this user belongs to.
-        */
-       if ((num_groups = winbind_getgroups(user, 0, NULL)) == -1)
-               return False;
-       if (num_groups == 0) {
-               *winbind_answered = True;
-               return False;
-       }
-       if ((groups = (gid_t *)malloc(sizeof(gid_t) * num_groups )) == NULL) {
-               DEBUG(0,("user_in_winbind_group_list: malloc fail.\n"));
+       if ((gid = nametogid(gname)) == (gid_t)-1) {
+               DEBUG(0,("user_in_winbind_group_list: nametogid for group %s failed.\n",
+                       gname ));
                goto err;
        }
-       if ((num_groups = winbind_getgroups(user, num_groups, groups)) == -1) {
-               DEBUG(0,("user_in_winbind_group_list: second winbind_getgroups call \
-failed with error %s\n", strerror(errno) ));
+
+       if (!lp_idmap_gid(&gid_low, &gid_high)) {
+               DEBUG(4, ("winbind gid range not configured, therefore %s cannot be a winbind group\n", gname));
+               goto err;
+       }
+
+       if (gid < gid_low || gid > gid_high) {
+               DEBUG(4, ("group %s is not a winbind group\n", gname));
                goto err;
        }
  
-       /*
-        * Now we have the gid list for this user - convert the gname
-        * to a gid_t via either winbind or the local UNIX lookup and do the comparison.
-        */
+       /* try to user the last user we looked up */
+       /* otherwise fall back to lookups */
+       
+       if ( !strequal( last_user, user ) || !groups )
+       {
+               /* clear any cached information */
+               
+               SAFE_FREE(groups);
+               fstrcpy( last_user, "" );
+
+               /*
+                * Get the gid's that this user belongs to.
+                */
  
-       if ((gid = nametogid(gname)) == (gid_t)-1) {
-               DEBUG(0,("user_in_winbind_group_list: winbind_lookup_name for group %s failed.\n",
-                       gname ));
-               goto err;
-       }
+               if ((num_groups = winbind_getgroups(user, &groups)) == -1)
+                       return False;
+                       
+               if ( num_groups == -1 )
+                       return False;
+               if ( num_groups == 0 ) {
+                       *winbind_answered = True;
+                       return False;
+               }
+               
+               /* save the last username */
+               
+               fstrcpy( last_user, user );
+               
+       }
+       else 
+               DEBUG(10,("user_in_winbind_group_list: using cached user groups for [%s]\n", user));
+       if ( DEBUGLEVEL >= 10 ) {
+               DEBUG(10,("user_in_winbind_group_list: using groups -- "));
+               for ( i=0; i<num_groups; i++ )
+                       DEBUGADD(10,("%d ", groups[i]));
+               DEBUGADD(10,("\n"));    
+       }
+       /*
+        * Now we have the gid list for this user - convert the gname
+        * to a gid_t via either winbind or the local UNIX lookup and do the comparison.
+        */
  
        for (i = 0; i < num_groups; i++) {
                if (gid == groups[i]) {
@@ -365,11 +417,11 @@ failed with error %s\n", strerror(errno) ));
  Check if a user is in a UNIX group.
 ****************************************************************************/
 
-static BOOL user_in_unix_group_list(const char *user,const char *gname)
+BOOL user_in_unix_group_list(const char *user,const char *gname)
 {
-       struct group *gptr;
-       char **member;  
        struct passwd *pass = Get_Pwnam(user);
+       struct sys_userlist *user_list;
+       struct sys_userlist *member;
 
        DEBUG(10,("user_in_unix_group_list: checking user %s in group %s\n", user, gname));
 
@@ -379,27 +431,28 @@ static BOOL user_in_unix_group_list(const char *user,const char *gname)
         */
  
        if (pass) {
-               gptr = getgrgid(pass->pw_gid);
-               if (gptr && strequal(gptr->gr_name,gname)) {
+               if (strequal(gname,gidtoname(pass->pw_gid))) {
                        DEBUG(10,("user_in_unix_group_list: group %s is primary group.\n", gname ));
                        return True;
                }
        }
  
-       if ((gptr = (struct group *)getgrnam(gname)) == NULL) {
+       user_list = get_users_in_group(gname);
+       if (user_list == NULL) {
                DEBUG(10,("user_in_unix_group_list: no such group %s\n", gname ));
                return False;
        }
-       member = gptr->gr_mem;
-       while (member && *member) {
-               DEBUG(10,("user_in_unix_group_list: checking user %s against member %s\n", user, *member ));
-               if (strequal(*member,user)) {
+
+       for (member = user_list; member; member = member->next) {
+               DEBUG(10,("user_in_unix_group_list: checking user %s against member %s\n",
+                       user, member->unix_name ));
+               if (strequal(member->unix_name,user)) {
+                       free_userlist(user_list);
                        return(True);
                }
-               member++;
        }
 
+       free_userlist(user_list);
        return False;
 }            
 
@@ -407,15 +460,35 @@ static BOOL user_in_unix_group_list(const char *user,const char *gname)
  Check if a user is in a group list. Ask winbind first, then use UNIX.
 ****************************************************************************/
 
-BOOL user_in_group_list(char *user,char *gname)
+BOOL user_in_group_list(const char *user, const char *gname, gid_t *groups, size_t n_groups)
 {
        BOOL winbind_answered = False;
-       BOOL ret = user_in_winbind_group_list(user, gname, &winbind_answered);
+       BOOL ret;
+       gid_t gid;
+       unsigned i;
+
+       gid = nametogid(gname);
+       if (gid == (gid_t)-1) 
+               return False;
+
+       if (groups && n_groups > 0) {
+               for (i=0; i < n_groups; i++) {
+                       if (groups[i] == gid) {
+                               return True;
+                       }
+               }
+               return False;
+       }
 
-       if (winbind_answered)
-               return ret;
+       /* fallback if we don't yet have the group list */
 
-       return user_in_unix_group_list(user, gname);    
+       ret = user_in_winbind_group_list(user, gname, &winbind_answered);
+       if (!winbind_answered)
+               ret = user_in_unix_group_list(user, gname);
+
+       if (ret)
+               DEBUG(10,("user_in_group_list: user |%s| is in group |%s|\n", user, gname));
+       return ret;
 }
 
 /****************************************************************************
@@ -423,77 +496,122 @@ BOOL user_in_group_list(char *user,char *gname)
  and netgroup lists.
 ****************************************************************************/
 
-BOOL user_in_list(char *user,char **list)
+BOOL user_in_list(const char *user,const char **list, gid_t *groups, size_t n_groups)
 {
-
-  if (!list || !*list) return False;
-
-  DEBUG(10,("user_in_list: checking user %s in list\n", user));
-
-  while (*list) {
-    /*
-     * Check raw username.
-     */
-    if (strequal(user, *list))
-      return(True);
-
-    /*
-     * Now check to see if any combination
-     * of UNIX and netgroups has been specified.
-     */
-
-    if(**list == '@') {
-      /*
-       * Old behaviour. Check netgroup list
-       * followed by UNIX list.
-       */
-      if(user_in_netgroup_list(user, *list +1))
-        return True;
-      if(user_in_group_list(user, *list +1))
-        return True;
-    } else if (**list == '+') {
-
-      if((*(*list +1)) == '&') {
-        /*
-         * Search UNIX list followed by netgroup.
-         */
-        if(user_in_group_list(user, *list +2))
-          return True;
-        if(user_in_netgroup_list(user, *list +2))
-          return True;
-
-      } else {
-
-        /*
-         * Just search UNIX list.
-         */
-
-        if(user_in_group_list(user, *list +1))
-          return True;
-      }
-
-    } else if (**list == '&') {
-
-      if(*(*list +1) == '+') {
-        /*
-         * Search netgroup list followed by UNIX list.
-         */
-        if(user_in_netgroup_list(user, *list +2))
-          return True;
-        if(user_in_group_list(user, *list +2))
-          return True;
-      } else {
-        /*
-         * Just search netgroup list.
-         */
-        if(user_in_netgroup_list(user, *list +1))
-          return True;
-      }
-    }
+       if (!list || !*list)
+               return False;
+
+       DEBUG(10,("user_in_list: checking user %s in list\n", user));
+
+       while (*list) {
+
+               DEBUG(10,("user_in_list: checking user |%s| against |%s|\n", user, *list));
+
+               /*
+                * Check raw username.
+                */
+               if (strequal(user, *list))
+                       return(True);
+
+               /*
+                * Now check to see if any combination
+                * of UNIX and netgroups has been specified.
+                */
+
+               if(**list == '@') {
+                       /*
+                        * Old behaviour. Check netgroup list
+                        * followed by UNIX list.
+                        */
+                       if(user_in_netgroup_list(user, *list +1))
+                               return True;
+                       if(user_in_group_list(user, *list +1, groups, n_groups))
+                               return True;
+               } else if (**list == '+') {
+
+                       if((*(*list +1)) == '&') {
+                               /*
+                                * Search UNIX list followed by netgroup.
+                                */
+                               if(user_in_group_list(user, *list +2, groups, n_groups))
+                                       return True;
+                               if(user_in_netgroup_list(user, *list +2))
+                                       return True;
+
+                       } else {
+
+                               /*
+                                * Just search UNIX list.
+                                */
+
+                               if(user_in_group_list(user, *list +1, groups, n_groups))
+                                       return True;
+                       }
+
+               } else if (**list == '&') {
+
+                       if(*(*list +1) == '+') {
+                               /*
+                                * Search netgroup list followed by UNIX list.
+                                */
+                               if(user_in_netgroup_list(user, *list +2))
+                                       return True;
+                               if(user_in_group_list(user, *list +2, groups, n_groups))
+                                       return True;
+                       } else {
+                               /*
+                                * Just search netgroup list.
+                                */
+                               if(user_in_netgroup_list(user, *list +1))
+                                       return True;
+                       }
+               } else if (!name_is_local(*list)) {
+                       /*
+                        * If user name did not match and token is not
+                        * a unix group and the token has a winbind separator in the
+                        * name then see if it is a Windows group.
+                        */
+
+                       DOM_SID g_sid;
+                       enum SID_NAME_USE name_type;
+                       BOOL winbind_answered = False;
+                       BOOL ret;
+                       fstring groupname, domain;
+                       
+                       /* Parse a string of the form DOMAIN/user into a domain and a user */
+
+                       char *p = strchr(*list,*lp_winbind_separator());
+                       
+                       DEBUG(10,("user_in_list: checking if user |%s| is in winbind group |%s|\n", user, *list));
+
+                       if (p) {
+                               fstrcpy(groupname, p+1);
+                               fstrcpy(domain, *list);
+                               domain[PTR_DIFF(p, *list)] = 0;
+
+                               /* Check to see if name is a Windows group;  Win2k native mode DCs
+                                  will return domain local groups; while NT4 or mixed mode 2k DCs
+                                  will not */
+                       
+                               if ( winbind_lookup_name(NULL, *list, &g_sid, &name_type) 
+                                       && ( name_type==SID_NAME_DOM_GRP || 
+                                          (strequal(lp_workgroup(), domain) && name_type==SID_NAME_ALIAS) ) )
+                               {
+                                       
+                                       /* Check if user name is in the Windows group */
+                                       ret = user_in_winbind_group_list(user, *list, &winbind_answered);
+                                       
+                                       if (winbind_answered && ret == True) {
+                                               DEBUG(10,("user_in_list: user |%s| is in winbind group |%s|\n", user, *list));
+                                               return ret;
+                                       }
+                               }
+                       }
+               }
     
-    list++;
-  }
-  return(False);
+               list++;
+       }
+       return(False);
 }
 
 /* The functions below have been taken from password.c and slightly modified */
@@ -505,30 +623,26 @@ BOOL user_in_list(char *user,char **list)
  it assumes the string starts lowercased
 ****************************************************************************/
 
-static struct passwd *uname_string_combinations2(char *s,int offset,struct passwd *(*fn)(char *),int N)
+static struct passwd *uname_string_combinations2(char *s,int offset,struct passwd *(*fn)(const char *),int N)
 {
-  ssize_t len = (ssize_t)strlen(s);
-  int i;
-  struct passwd *ret;
-
-#ifdef PASSWORD_LENGTH
-  len = MIN(len,PASSWORD_LENGTH);
-#endif
-
-  if (N <= 0 || offset >= len)
-    return(fn(s));
-
-  for (i=offset;i<(len-(N-1));i++) {
-    char c = s[i];
-    if (!islower(c))
-      continue;
-    s[i] = toupper(c);
-    ret = uname_string_combinations2(s,i+1,fn,N-1);
-    if(ret)
-      return(ret);
-    s[i] = c;
-  }
-  return(NULL);
+       ssize_t len = (ssize_t)strlen(s);
+       int i;
+       struct passwd *ret;
+
+       if (N <= 0 || offset >= len)
+               return(fn(s));
+
+       for (i=offset;i<(len-(N-1));i++) {
+               char c = s[i];
+               if (!islower((int)c))
+                       continue;
+               s[i] = toupper(c);
+               ret = uname_string_combinations2(s,i+1,fn,N-1);
+               if(ret)
+                       return(ret);
+               s[i] = c;
+       }
+       return(NULL);
 }
 
 /****************************************************************************
@@ -539,51 +653,16 @@ static struct passwd *uname_string_combinations2(char *s,int offset,struct passw
  it assumes the string starts lowercased
 ****************************************************************************/
 
-static struct passwd * uname_string_combinations(char *s,struct passwd * (*fn)(char *),int N)
-{
-  int n;
-  struct passwd *ret;
-
-  for (n=1;n<=N;n++) {
-    ret = uname_string_combinations2(s,0,fn,n);
-    if(ret)
-      return(ret);
-  }
-  return(NULL);
-}
-
-
-/****************************************************************************
-these wrappers allow appliance mode to work. In appliance mode the username
-takes the form DOMAIN/user
-****************************************************************************/
-struct passwd *smb_getpwnam(char *user, BOOL allow_change)
+static struct passwd * uname_string_combinations(char *s,struct passwd * (*fn)(const char *),int N)
 {
-       struct passwd *pw;
-       char *p;
-       char *sep;
-       extern pstring global_myname;
+       int n;
+       struct passwd *ret;
 
-       if (allow_change) {
-               pw = Get_Pwnam_Modify(user);
-       } else {
-               pw = Get_Pwnam(user);
-       }
-       if (pw) return pw;
-
-       /* if it is a domain qualified name and it isn't in our password
-          database but the domain portion matches our local machine name then
-          lookup just the username portion locally */
-       sep = lp_winbind_separator();
-       if (!sep || !*sep) sep = "\\";
-       p = strchr_m(user,*sep);
-       if (p && 
-           strncasecmp(global_myname, user, strlen(global_myname))==0) {
-               if (allow_change) {
-                       pw = Get_Pwnam_Modify(p+1);
-               } else {
-                       pw = Get_Pwnam(p+1);
-               }
-       }
-       return NULL;
+       for (n=1;n<=N;n++) {
+               ret = uname_string_combinations2(s,0,fn,n);
+               if(ret)
+                       return(ret);
+       }  
+       return(NULL);
 }
+