From 9271a050b5af6594ab112a9c116854953b041d8f Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Thu, 22 Oct 1998 11:49:29 +0000 Subject: Update. 1998-10-22 H.J. Lu * sysdeps/unix/sysv/linux/i386/getgroups.c (__getgroups): Add sanity check for n. * sysdeps/unix/sysv/linux/i386/setgroups.c (setgroups): Likewise. * sysdeps/posix/fpathconf.c (__fpathconf): Set errno to EINVAL if errno == ENODEV. Tested by VSX-PCT. * sysdeps/posix/isatty.c (__isatty): Don't reset errno. Tested by VSX-PCT. * posix/execvp.c (execvp): Check "". Tested by VSX-PCT. --- sysdeps/unix/sysv/linux/i386/getgroups.c | 30 +++++++++++++++++++++--------- sysdeps/unix/sysv/linux/i386/setgroups.c | 28 ++++++++++++++++++---------- 2 files changed, 39 insertions(+), 19 deletions(-) (limited to 'sysdeps/unix') diff --git a/sysdeps/unix/sysv/linux/i386/getgroups.c b/sysdeps/unix/sysv/linux/i386/getgroups.c index c306dd1..5484f72 100644 --- a/sysdeps/unix/sysv/linux/i386/getgroups.c +++ b/sysdeps/unix/sysv/linux/i386/getgroups.c @@ -17,10 +17,13 @@ Boston, MA 02111-1307, USA. */ #include -#include #include +#include +#include #include +#include + #include extern int __syscall_getgroups __P ((int, __kernel_gid_t *)); @@ -32,14 +35,23 @@ __getgroups (n, groups) int n; gid_t *groups; { - int i, ngids; - __kernel_gid_t kernel_groups[n]; - - ngids = INLINE_SYSCALL (getgroups, 2, n, kernel_groups); - if (n != 0 && ngids > 0) - for (i = 0; i < ngids; i++) - groups[i] = kernel_groups[i]; - return ngids; + if (n < 0) + { + __set_errno (EINVAL); + return -1; + } + else + { + int i, ngids; + __kernel_gid_t kernel_groups[n = MIN (n, __sysconf (_SC_NGROUPS_MAX))]; + + ngids = INLINE_SYSCALL (getgroups, 2, n, kernel_groups); + if (n != 0 && ngids > 0) + for (i = 0; i < ngids; i++) + groups[i] = kernel_groups[i]; + + return ngids; + } } weak_alias (__getgroups, getgroups) diff --git a/sysdeps/unix/sysv/linux/i386/setgroups.c b/sysdeps/unix/sysv/linux/i386/setgroups.c index ad2a6b5..67197e6 100644 --- a/sysdeps/unix/sysv/linux/i386/setgroups.c +++ b/sysdeps/unix/sysv/linux/i386/setgroups.c @@ -36,18 +36,26 @@ setgroups (n, groups) size_t n; const gid_t *groups; { - size_t i; - __kernel_gid_t kernel_groups[n]; - - for (i = 0; i < n; i++) + if (n < 0 || n > __sysconf (_SC_NGROUPS_MAX)) + { + __set_errno (EINVAL); + return -1; + } + else if { - kernel_groups[i] = groups[i]; - if (groups[i] != (gid_t) ((__kernel_gid_t) groups[i])) + size_t i; + __kernel_gid_t kernel_groups[n]; + + for (i = 0; i < n; i++) { - __set_errno (EINVAL); - return -1; + kernel_groups[i] = groups[i]; + if (groups[i] != (gid_t) ((__kernel_gid_t) groups[i])) + { + __set_errno (EINVAL); + return -1; + } } - } - return INLINE_SYSCALL (setgroups, 2, n, kernel_groups); + return INLINE_SYSCALL (setgroups, 2, n, kernel_groups); + } } -- cgit v1.1