From 40be7b467f0ccbc52a816434863592ab20a82680 Mon Sep 17 00:00:00 2001 From: Rob Bradford Date: Wed, 5 Dec 2012 18:47:11 +0000 Subject: [PATCH] weston-launch: Add some error reporting for weston-launch This also avoids us passing (size_t)(-1 * sizeof(gid_t)) into malloc --- src/weston-launch.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/weston-launch.c b/src/weston-launch.c index cb69968a..6d1f7c3b 100644 --- a/src/weston-launch.c +++ b/src/weston-launch.c @@ -84,11 +84,18 @@ read_groups(void) gid_t *groups; n = getgroups(0, NULL); + + if (n < 0) { + fprintf(stderr, "Unable to retrieve groups: %m\n"); + return NULL; + } + groups = malloc(n * sizeof(gid_t)); if (!groups) return NULL; if (getgroups(n, groups) < 0) { + fprintf(stderr, "Unable to retrieve groups: %m\n"); free(groups); return NULL; }