aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2003-04-23 21:27:36 +0000
committerUlrich Drepper <drepper@redhat.com>2003-04-23 21:27:36 +0000
commitb9b9a51e4911b17e75b39fd3db9f7268e233ce20 (patch)
tree95c02630a80ee0b0c094bdde7594401d350cf58e
parent370b4d44dc9f8c29f842d1799cc096825bcc1ed7 (diff)
downloadglibc-b9b9a51e4911b17e75b39fd3db9f7268e233ce20.tar
glibc-b9b9a51e4911b17e75b39fd3db9f7268e233ce20.tar.gz
glibc-b9b9a51e4911b17e75b39fd3db9f7268e233ce20.tar.bz2
glibc-b9b9a51e4911b17e75b39fd3db9f7268e233ce20.zip
Update.
2003-04-23 Ulrich Drepper <drepper@redhat.com> * grp/initgroups.c (getgrouplist): Don't copy too much into the user buffer if more groups are found than fit into it. * nis/nss_nis/nis-initgroups.c (_nss_nis_initgroups_dyn): Use extend_alloca.
-rw-r--r--ChangeLog8
-rw-r--r--grp/initgroups.c8
-rw-r--r--nis/nss_nis/nis-initgroups.c12
3 files changed, 18 insertions, 10 deletions
diff --git a/ChangeLog b/ChangeLog
index d64c21bebe..6643aa3ff1 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2003-04-23 Ulrich Drepper <drepper@redhat.com>
+
+ * grp/initgroups.c (getgrouplist): Don't copy too much into the
+ user buffer if more groups are found than fit into it.
+
+ * nis/nss_nis/nis-initgroups.c (_nss_nis_initgroups_dyn): Use
+ extend_alloca.
+
2003-04-23 Jakub Jelinek <jakub@redhat.com>
* sysdeps/posix/getaddrinfo.c (gaih_inet): Check for rc == ERANGE,
diff --git a/grp/initgroups.c b/grp/initgroups.c
index dd7a8fde1c..1e4c2b5c2b 100644
--- a/grp/initgroups.c
+++ b/grp/initgroups.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 1989,91,93,1996-2001, 2002 Free Software Foundation, Inc.
+/* Copyright (C) 1989,91,93,1996-2002, 2003 Free Software Foundation, Inc.
This file is part of the GNU C Library.
The GNU C Library is free software; you can redistribute it and/or
@@ -23,6 +23,7 @@
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
+#include <sys/param.h>
#include <sys/types.h>
#include <nsswitch.h>
@@ -207,6 +208,9 @@ getgrouplist (const char *user, gid_t group, gid_t *groups, int *ngroups)
return -1;
result = internal_getgrouplist (user, group, &size, &newgroups, -1);
+
+ memcpy (groups, newgroups, MIN (*ngroups, result) * sizeof (gid_t));
+
if (result > *ngroups)
{
*ngroups = result;
@@ -215,8 +219,6 @@ getgrouplist (const char *user, gid_t group, gid_t *groups, int *ngroups)
else
*ngroups = result;
- memcpy (groups, newgroups, *ngroups * sizeof (gid_t));
-
free (newgroups);
return result;
}
diff --git a/nis/nss_nis/nis-initgroups.c b/nis/nss_nis/nis-initgroups.c
index de7444e870..e3a8f57fcc 100644
--- a/nis/nss_nis/nis-initgroups.c
+++ b/nis/nss_nis/nis-initgroups.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 1998, 1999, 2000, 2002 Free Software Foundation, Inc.
+/* Copyright (C) 1998, 1999, 2000, 2002, 2003 Free Software Foundation, Inc.
This file is part of the GNU C Library.
Contributed by Thorsten Kukuk <kukuk@suse.de>, 1998.
@@ -17,10 +17,11 @@
Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA
02111-1307 USA. */
-#include <nss.h>
-#include <grp.h>
+#include <alloca.h>
#include <ctype.h>
#include <errno.h>
+#include <grp.h>
+#include <nss.h>
#include <string.h>
#include <unistd.h>
#include <rpcsvc/yp.h>
@@ -159,10 +160,7 @@ _nss_nis_initgroups_dyn (const char *user, gid_t group, long int *start,
internal_getgrent_r (&grpbuf, tmpbuf, buflen, errnop,
&intern)) == NSS_STATUS_TRYAGAIN
&& *errnop == ERANGE)
- {
- buflen *= 2;
- tmpbuf = __alloca (buflen);
- }
+ tmpbuf = extend_alloca (tmpbuf, buflen, 2 * buflen);
if (status != NSS_STATUS_SUCCESS)
goto done;