(getgroups): Don't use xrealloc, since we don't need the buffer
authorPaul Eggert <eggert@cs.ucla.edu>
Wed, 29 Oct 2003 17:27:58 +0000 (17:27 +0000)
committerPaul Eggert <eggert@cs.ucla.edu>
Wed, 29 Oct 2003 17:27:58 +0000 (17:27 +0000)
preserved.  Use free and xmalloc instead.

lib/getgroups.c

index e27cadc3e93289c7e56b597300fa491730cb96ec..4ffa4e08fc0ea647a7bd1627f518445c743dded4 100644 (file)
@@ -43,16 +43,16 @@ getgroups (int n, GETGROUPS_T *group)
     return getgroups (n, group);
 
   n = 20;
-  gbuf = NULL;
   while (1)
     {
       /* No need to worry about address arithmetic overflow here,
         since the ancient systems that we're running on have low
         limits on the number of secondary groups.  */
-      gbuf = xrealloc (gbuf, n * sizeof (GETGROUPS_T));
+      gbuf = xmalloc (gbuf, n * sizeof *gbuf);
       n_groups = getgroups (n, gbuf);
       if (n_groups < n)
        break;
+      free (gbuf);
       n += 10;
     }