Re: [coreutils]/lib/getgroups.c, xmalloc usage...

[Jim Meyering]

"Alfred M. Szmidt" <address@hidden> wrote:
> While having a hard time to sleep, I started to read obscure bits of
> code in coreutils.  And found this funny compiler error in
> [kernel]/lib/getgroups.c at line 51:
>
>       /* 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);
>
> I suppose that nobody has ever used that code, since compiling would
> catch the fact that xmalloc only takes one argument! :-)

Thank you.
I've applied Paul's patch for the coreutils, too.