From 8e376ca602cd2377681e5c533c0a061764e184e7 Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Wed, 28 Dec 2005 21:20:42 +0000 Subject: make sure we handle the (malloc(0)==NULL) case as Aubrey points out via the e-mail list --- libc/inet/rpc/auth_unix.c | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) (limited to 'libc/inet') diff --git a/libc/inet/rpc/auth_unix.c b/libc/inet/rpc/auth_unix.c index 3cb286cc4..bb14df068 100644 --- a/libc/inet/rpc/auth_unix.c +++ b/libc/inet/rpc/auth_unix.c @@ -183,12 +183,14 @@ __authunix_create_default (void) uid_t uid; gid_t gid; int max_nr_groups = sysconf (_SC_NGROUPS_MAX); - gid_t *gids; + gid_t *gids = NULL; AUTH *ret_auth; - gids = (gid_t*)malloc(sizeof(*gids) * max_nr_groups); - if (gids == NULL) - abort (); + if (max_nr_groups) { + gids = (gid_t*)malloc(sizeof(*gids) * max_nr_groups); + if (gids == NULL) + abort (); + } if (gethostname (machname, MAX_MACHINE_NAME) == -1) abort (); @@ -202,7 +204,8 @@ __authunix_create_default (void) list of groups to NGRPS members since the code in authuxprot.c transforms a fixed array. Grrr. */ ret_auth = __authunix_create (machname, uid, gid, MIN (NGRPS, len), gids); - free (gids); + if (gids) + free (gids); return ret_auth; } strong_alias(__authunix_create_default,authunix_create_default) -- cgit v1.2.3