From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:49273) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UDXxW-0002KT-V3 for qemu-devel@nongnu.org; Thu, 07 Mar 2013 05:22:01 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UDXxT-0003SV-WC for qemu-devel@nongnu.org; Thu, 07 Mar 2013 05:21:58 -0500 Received: from afflict.kos.to ([92.243.29.197]:33626) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UDXxT-0003SG-Ox for qemu-devel@nongnu.org; Thu, 07 Mar 2013 05:21:55 -0500 From: riku.voipio@linaro.org Date: Thu, 7 Mar 2013 10:21:31 +0000 Message-Id: <35c4acdb963b9a8188f101d404dc50b47babd7ea.1362651161.git.riku.voipio@linaro.org> In-Reply-To: References: Subject: [Qemu-devel] [PATCH 6/7] linux-user: Support setgroups syscall with no groups List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Dillon Amburgey From: Dillon Amburgey Signed-off-by: Dillon Amburgey Reviewed-by: Peter Maydell Signed-off-by: Riku Voipio --- linux-user/syscall.c | 22 ++++++++++++---------- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/linux-user/syscall.c b/linux-user/syscall.c index 7f12563..e0c71fb 100644 --- a/linux-user/syscall.c +++ b/linux-user/syscall.c @@ -7694,18 +7694,20 @@ abi_long do_syscall(void *cpu_env, int num, abi_long arg1, { int gidsetsize = arg1; target_id *target_grouplist; - gid_t *grouplist; + gid_t *grouplist = NULL; int i; - - grouplist = alloca(gidsetsize * sizeof(gid_t)); - target_grouplist = lock_user(VERIFY_READ, arg2, gidsetsize * 2, 1); - if (!target_grouplist) { - ret = -TARGET_EFAULT; - goto fail; + if (gidsetsize) { + grouplist = alloca(gidsetsize * sizeof(gid_t)); + target_grouplist = lock_user(VERIFY_READ, arg2, gidsetsize * 2, 1); + if (!target_grouplist) { + ret = -TARGET_EFAULT; + goto fail; + } + for (i = 0; i < gidsetsize; i++) { + grouplist[i] = low2highgid(tswapid(target_grouplist[i])); + } + unlock_user(target_grouplist, arg2, 0); } - for(i = 0;i < gidsetsize; i++) - grouplist[i] = low2highgid(tswapid(target_grouplist[i])); - unlock_user(target_grouplist, arg2, 0); ret = get_errno(setgroups(gidsetsize, grouplist)); } break; -- 1.8.1.2