请帮看一下这段代码

xz4

这段代码是gnu coreutils-5.2.1 的 id 的源代码。
注意红色部分。

static int
xgetgroups (const char *username, gid_t gid, int *n_groups,
            GETGROUPS_T **groups)
{
  int max_n_groups;
  int ng;
  GETGROUPS_T *g;
  int fail = 0;

  if (username == 0)
    max_n_groups = getgroups (0, NULL);
  else
    max_n_groups = getugroups (0, NULL, username, gid);

  /* Add 1 just in case max_n_groups is zero.  */
  g = xmalloc (max_n_groups * sizeof (GETGROUPS_T) + 1);
  if (username == 0)
    ng = getgroups (max_n_groups, g);
  else
    ng = getugroups (max_n_groups, g, username, gid);

  if (ng < 0)
    {
      error (0, errno, _("cannot get supplemental group list"));
      ++fail;
      free (groups);
    }
  if (!fail)
    {
      *n_groups = ng;
      *groups = g;
    }
  return fail;
}


free(groups)  是否应该是 free(g)

rickxbx

You're right, please have a look at the 5.3.0 version:

1. 
   
2.     252 static bool
   
3.     253 xgetgroups (const char *username, gid_t gid, int *n_groups,
   
4.     254         GETGROUPS_T **groups)
   
5.     255 {
   
6.     256   int max_n_groups;
   
7.     257   int ng;
   
8.     258   GETGROUPS_T *g = NULL;
   
9.     259
   
10.     260   if (!username)
    
11.     261     max_n_groups = getgroups (0, NULL);
    
12.     262   else
    
13.     263     max_n_groups = getugroups (0, NULL, username, gid);
    
14.     264
    
15.     265   if (max_n_groups < 0)
    
16.     266     ng = -1;
    
17.     267   else
    
18.     268     {
    
19.     269       g = xnmalloc (max_n_groups, sizeof *g);
    
20.     270       if (!username)
    
21.     271     ng = getgroups (max_n_groups, g);
    
22.     272       else
    
23.     273     ng = getugroups (max_n_groups, g, username, gid);
    
24.     274     }
    
25.     275
    
26.     276   if (ng < 0)
    
27.     277     {
    
28.     278       error (0, errno, _("cannot get supplemental group list"));
    
29.     279       free (g);
    
30.     280       return false;
    
31.     281     }
    
32.     282   else
    
33.     283     {
    
34.     284       *n_groups = ng;
    
35.     285       *groups = g;
    
36.     286       return true;
    
37.     287     }
    
38.     288 }
    

复制代码

xz4

谢谢.
马上下5.3.0来看看。