From: Paul Eggert <eggert@cs.ucla.edu>
Date: Wed, 29 Oct 2003 17:27:58 +0000 (+0000)
Subject: (getgroups): Don't use xrealloc, since we don't need the buffer
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=d67c6629b22d9d870bc2c8af9e1655e9a0e7e6ac;p=pspp

(getgroups): Don't use xrealloc, since we don't need the buffer
preserved.  Use free and xmalloc instead.
---

diff --git a/lib/getgroups.c b/lib/getgroups.c
index e27cadc3e9..4ffa4e08fc 100644
--- a/lib/getgroups.c
+++ b/lib/getgroups.c
@@ -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;
     }