summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--libc/sysdeps/linux/common/getgroups.c10
-rw-r--r--libc/sysdeps/linux/common/setgroups.c16
2 files changed, 21 insertions, 5 deletions
diff --git a/libc/sysdeps/linux/common/getgroups.c b/libc/sysdeps/linux/common/getgroups.c
index c863489b9..b2918c6d9 100644
--- a/libc/sysdeps/linux/common/getgroups.c
+++ b/libc/sysdeps/linux/common/getgroups.c
@@ -10,6 +10,7 @@
#define sysconf __sysconf
#include "syscalls.h"
+#include <stdlib.h>
#include <unistd.h>
#define MIN(a,b) (((a)<(b))?(a):(b))
@@ -21,11 +22,17 @@ static inline _syscall2(int, __syscall_getgroups,
int attribute_hidden __getgroups(int n, gid_t * groups)
{
if (unlikely(n < 0)) {
+ret_error:
__set_errno(EINVAL);
return -1;
} else {
int i, ngids;
- __kernel_gid_t kernel_groups[n = MIN(n, sysconf(_SC_NGROUPS_MAX))];
+ __kernel_gid_t *kernel_groups;
+
+ n = MIN(n, sysconf(_SC_NGROUPS_MAX));
+ kernel_groups = (__kernel_gid_t *)malloc(sizeof(*kernel_groups) * n);
+ if (kernel_groups == NULL)
+ goto ret_error;
ngids = __syscall_getgroups(n, kernel_groups);
if (n != 0 && ngids > 0) {
@@ -33,6 +40,7 @@ int attribute_hidden __getgroups(int n, gid_t * groups)
groups[i] = kernel_groups[i];
}
}
+ free(kernel_groups);
return ngids;
}
}
diff --git a/libc/sysdeps/linux/common/setgroups.c b/libc/sysdeps/linux/common/setgroups.c
index 96428edb3..21823ad9b 100644
--- a/libc/sysdeps/linux/common/setgroups.c
+++ b/libc/sysdeps/linux/common/setgroups.c
@@ -10,6 +10,7 @@
#define sysconf __sysconf
#include "syscalls.h"
+#include <stdlib.h>
#include <unistd.h>
#include <grp.h>
@@ -20,20 +21,27 @@ static inline _syscall2(int, __syscall_setgroups,
int attribute_hidden __setgroups(size_t n, const gid_t * groups)
{
if (n > (size_t) sysconf(_SC_NGROUPS_MAX)) {
+ret_error:
__set_errno(EINVAL);
return -1;
} else {
size_t i;
- __kernel_gid_t kernel_groups[n];
+ __kernel_gid_t *kernel_groups;
+
+ kernel_groups = (__kernel_gid_t *)malloc(sizeof(*kernel_groups) * n);
+ if (kernel_groups == NULL)
+ goto ret_error;
for (i = 0; i < n; i++) {
kernel_groups[i] = (groups)[i];
if (groups[i] != (gid_t) ((__kernel_gid_t) groups[i])) {
- __set_errno(EINVAL);
- return -1;
+ goto ret_error;
}
}
- return (__syscall_setgroups(n, kernel_groups));
+
+ i = __syscall_setgroups(n, kernel_groups);
+ free(kernel_groups);
+ return i;
}
}
strong_alias(__setgroups,setgroups)