Greg Dickie spotted some wierd memory corruption problem with group
[samba.git] / source3 / groupdb / aliasunix.c
index f9b93bbce439ee17e2d761e478817520e4fdcd40..c53692f32825df470c2854434312ac253cc8f594 100644 (file)
@@ -27,6 +27,13 @@ extern int DEBUGLEVEL;
 extern DOM_SID global_sam_sid;
 extern fstring global_sam_name;
 
+struct unix_entries
+{
+       struct group *grps;
+       int num_grps;
+       int grp_idx;
+};
+
 /***************************************************************
  Start to enumerate the alspasswd list. Returns a void pointer
  to ensure no modification outside this module.
@@ -34,8 +41,23 @@ extern fstring global_sam_name;
 
 static void *startalsunixpwent(BOOL update)
 {
-       setgrent();
-       return (void*)(-1);
+       struct unix_entries *grps;
+       grps = (struct unix_entries*)malloc(sizeof(struct unix_entries));
+
+       if (grps == NULL)
+       {
+               return NULL;
+       }
+
+       if (!get_unix_grps(&grps->num_grps, &grps->grps))
+       {
+               free(grps);
+               return NULL;
+       }
+
+       grps->grp_idx = 0;
+
+       return (void*)grps;
 }
 
 /***************************************************************
@@ -44,7 +66,13 @@ static void *startalsunixpwent(BOOL update)
 
 static void endalsunixpwent(void *vp)
 {
-       endgrent();
+       struct unix_entries *grps = (struct unix_entries *)vp;
+
+       if (grps != NULL)
+       {
+               free_unix_grps(grps->num_grps, grps->grps);
+               free(vp);
+       }
 }
 
 /*************************************************************************
@@ -142,7 +170,8 @@ static LOCAL_GRP *getalsunixpwent(void *vp, LOCAL_GRP_MEMBER **mem, int *num_mem
 {
        /* Static buffers we will return. */
        static LOCAL_GRP gp_buf;
-       struct group *unix_grp;
+       struct group unix_grp;
+       struct unix_entries *grps = (struct unix_entries *)vp;
 
        if (lp_server_role() == ROLE_DOMAIN_NONE)
        {
@@ -156,15 +185,22 @@ static LOCAL_GRP *getalsunixpwent(void *vp, LOCAL_GRP_MEMBER **mem, int *num_mem
 
        aldb_init_als(&gp_buf);
 
+       /* get array of unix names + gids.  this function does NOT
+          get a copy of the unix group members
+        */
+
        /* cycle through unix groups */
-       while ((unix_grp = getgrent()) != NULL)
+       for (; grps->grp_idx < grps->num_grps; grps->grp_idx++)
        {
                DOM_NAME_MAP gmep;
                fstring sid_str;
+
+               memcpy(&unix_grp, &grps->grps[grps->grp_idx], sizeof(unix_grp));
+
                DEBUG(10,("getgrpunixpwent: enum unix group entry %s\n",
-                          unix_grp->gr_name));
+                          unix_grp.gr_name));
                        
-               if (!lookupsmbgrpgid(unix_grp->gr_gid, &gmep))
+               if (!lookupsmbgrpgid(unix_grp.gr_gid, &gmep))
                {
                        continue;
                }
@@ -188,7 +224,7 @@ static LOCAL_GRP *getalsunixpwent(void *vp, LOCAL_GRP_MEMBER **mem, int *num_mem
                break;
        }
 
-       if (unix_grp == NULL)
+       if (grps->grp_idx >= grps->num_grps)
        {
                return NULL;
        }
@@ -200,7 +236,8 @@ static LOCAL_GRP *getalsunixpwent(void *vp, LOCAL_GRP_MEMBER **mem, int *num_mem
                (*mem) = NULL;
                (*num_mem) = 0;
 
-               get_unixalias_members(unix_grp, num_mem, mem);
+               memcpy(&unix_grp, getgrgid(unix_grp.gr_gid), sizeof(unix_grp));
+               get_unixalias_members(&unix_grp, num_mem, mem);
        }
 
        {
@@ -209,6 +246,7 @@ static LOCAL_GRP *getalsunixpwent(void *vp, LOCAL_GRP_MEMBER **mem, int *num_mem
                DEBUG(10,("line: '%s'\n", linebuf));
        }
 
+       grps->grp_idx++; /* advance so next enum gets next entry */
        return &gp_buf;
 }
 
@@ -224,11 +262,7 @@ static BOOL add_alsunixgrp_entry(LOCAL_GRP *newals)
 
 /************************************************************************
  Routine to search the alspasswd file for an entry matching the aliasname.
- and then modify its alias entry. We can't use the startalspwent()/
- getalspwent()/endalspwent() 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 alias or NO PASS
+ and then modify its alias entry. 
 ************************************************************************/
 
 static BOOL mod_alsunixgrp_entry(LOCAL_GRP* als)
@@ -237,6 +271,35 @@ static BOOL mod_alsunixgrp_entry(LOCAL_GRP* als)
        return False;
 }
 
+/************************************************************************
+ Routine to search the grppasswd file for an entry matching the rid.
+ and then delete it.
+************************************************************************/
+
+static BOOL del_alsunixgrp_entry(uint32 rid)
+{
+       DEBUG(0, ("del_alsunixgrp_entry: NOT IMPLEMENTED\n"));
+       return False;
+}
+
+/************************************************************************
+ Routine to add a member to an entry to the grppasswd file.
+*************************************************************************/
+static BOOL add_alsunixgrp_member(uint32 rid, DOM_SID *member_sid)
+{
+       DEBUG(0, ("add_alsunixgrp_member: NOT IMPLEMENTED\n"));
+       return False;
+}
+
+/************************************************************************
+ Routine to delete a member from an entry to the grppasswd file.
+*************************************************************************/
+static BOOL del_alsunixgrp_member(uint32 rid, DOM_SID *member_sid)
+{
+       DEBUG(0, ("del_alsunixgrp_member: NOT IMPLEMENTED\n"));
+       return False;
+}
+
 
 static struct aliasdb_ops unix_ops =
 {
@@ -252,6 +315,10 @@ static struct aliasdb_ops unix_ops =
 
        add_alsunixgrp_entry,
        mod_alsunixgrp_entry,
+       del_alsunixgrp_entry,
+
+       add_alsunixgrp_member,
+       del_alsunixgrp_member,
 
        iterate_getuseraliasntnam      /* in aliasdb.c */
 };