* getgroups.c (getgroups): xmalloc takes one argument, not two.

diff --git a/lib/ChangeLog b/lib/ChangeLog
index a240e70224ef4ce4791b842388773f94bedf214e..2ec3bff3a1ca72b1d42d5e265bf47d45b5bdecf5 100644 (file)
--- a/lib/ChangeLog
+++ b/lib/ChangeLog
@@ -1,3 +1,8 @@
+2003-12-03  Paul Eggert  <eggert@twinsun.com>
+
+       * getgroups.c (getgroups): xmalloc takes one argument, not two.
+       Bug reported by Alfred M. Szmidt.
+
 2003-11-29  Karl Berry  <karl@gnu.org>
 
        * argp-{help.c,parse.c,namefrob.h}, argp.h: update from libc.

diff --git a/lib/getgroups.c b/lib/getgroups.c
index 4ffa4e08fc0ea647a7bd1627f518445c743dded4..0f951b73baa61fcdfa6c71dcd5b398d838898a13 100644 (file)
--- a/lib/getgroups.c
+++ b/lib/getgroups.c
@@ -48,7 +48,7 @@ getgroups (int n, GETGROUPS_T *group)
       /* 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 = xmalloc (gbuf, n * sizeof *gbuf);
+      gbuf = xmalloc (n * sizeof *gbuf);
       n_groups = getgroups (n, gbuf);
       if (n_groups < n)
        break;