From 76ef73f50a23ed943f9372f6dcb194eb9c855fed Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Thu, 6 May 2004 14:43:31 +0000 Subject: (xgetgroups): Use xnmalloc, rather than xmalloc. Don't add `1' to the buffer size (it was to protect against malloc implementations that fail to allocate a buffer of size zero). That is no longer necessary, since we use a malloc wrapper on such systems. --- src/id.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'src/id.c') diff --git a/src/id.c b/src/id.c index db4883c83..0fec74fe2 100644 --- a/src/id.c +++ b/src/id.c @@ -261,8 +261,7 @@ xgetgroups (const char *username, gid_t gid, int *n_groups, else max_n_groups = getugroups (0, NULL, username, gid); - /* Add 1 just in case max_n_groups is zero. */ - g = xmalloc (max_n_groups * sizeof (GETGROUPS_T) + 1); + g = xnmalloc (max_n_groups, sizeof (GETGROUPS_T)); if (username == 0) ng = getgroups (max_n_groups, g); else -- cgit v1.2.3-54-g00ecf