(getgrouplist): Don't allocate one additional element on NEWGROUPS.
authordrepper <drepper>
Thu, 3 Aug 2006 15:29:58 +0000 (15:29 +0000)
committerdrepper <drepper>
Thu, 3 Aug 2006 15:29:58 +0000 (15:29 +0000)
grp/initgroups.c

index 1cc07b6..883ea71 100644 (file)
@@ -144,7 +144,7 @@ getgrouplist (const char *user, gid_t group, gid_t *groups, int *ngroups)
 {
   long int size = MAX (1, *ngroups);
 
-  gid_t *newgroups = (gid_t *) malloc ((size + 1) * sizeof (gid_t));
+  gid_t *newgroups = (gid_t *) malloc (size * sizeof (gid_t));
   if (__builtin_expect (newgroups == NULL, 0))
     /* No more memory.  */
     // XXX This is wrong.  The user provided memory, we have to use