coreutils/gl/lib/mgetgroups.c, branch v7.6Mirror of https://https.git.savannah.gnu.org/git/coreutils.git/
https://git.shady.money/coreutils/atom?h=v7.62009-08-25T07:21:00Zglobal: convert indentation-TABs to spaces2009-08-25T07:21:00ZJim Meyeringmeyering@redhat.com2009-08-22T16:56:06Zurn:sha1:5e778f7c8d1ecf3d8f11385db013af2ba026e2a5
Transformed via this shell code:
t=$'\t'
git ls-files \
| grep -vE '(^|/)((GNU)?[Mm]akefile|ChangeLog)|\.(am|mk)$' \
| grep -vE 'tests/pr/|help2man' \
| xargs grep -lE "^ *$t" \
| xargs perl -MText::Tabs -ni -le \
'$m=/^( *\t[ \t]*)(.*)/; print $m ? expand($1) . $2 : $_'
id: fix infinite loop on some systems2009-04-09T13:07:29ZPádraig BradyP@draigBrady.com2009-04-08T09:43:15Zurn:sha1:bf87a2c8ea4487ca4448c9fe42a9c9858400acbd
Steven Parkes reported that `id -G $USER` went into an infinite loop
on Darwin systems for users in more than 10 groups:
http://bugs.gentoo.org/show_bug.cgi?id=264007
* gl/lib/mgetgroups.c (mgetgroups): Work around buggy getgrouplist
implementations that don't update the required size correctly,
by doubling the result buffer and retrying. Also return the
parameter updated by getgrouplist rather than its return value,
as the documentation doesn't actually state the number of groups
stored is returned by getgrouplist.
* tests/misc/id-groups: Add test to exercise this logic
* tests/Makefile.am: Reference new test
* NEWS: Mention the fix
* THANKS: Update
simplify mgetgroups() and avoid -Wsign-compare warnings2009-01-16T11:06:09ZPádraig BradyP@draigBrady.com2009-01-14T19:17:39Zurn:sha1:46d6df933ec6171fcfa0220dfb8f18f68293797b
* gl/lib/mgetgroups.c: Avoid -Wsign-compare warning by using unsigned
types for the parameters of the new function realloc_groupbuf().
mgetgroups() was refactored to use this function rather than
explicitly allocating and copying from automatic storage itself.
* src/group-list.c: Use int rather than size_t as variable is
used in signed comparisons.
* src/id.c: ditto.
* gl/lib/mgetgroups.c (mgetgroups): Parenthesize correctly.2008-02-24T17:12:15ZJim Meyeringmeyering@redhat.com2008-02-24T17:12:15Zurn:sha1:9396e9104e9965b9608a3652c18d7c2d94ea3351* gl/lib/mgetgroups.c: Include <stdlib.h>.2008-02-24T13:56:19ZJim Meyeringmeyering@redhat.com2008-02-24T13:56:19Zurn:sha1:dacff7cb2b2d8765828b10674dc515292584a432id: avoid race when a group is added between getgrouplist calls2008-02-22T19:47:54ZJim Meyeringmeyering@redhat.com2008-02-22T09:01:36Zurn:sha1:a15329798c52c57cc16fc24265327d8b1c73ab41
* gl/lib/mgetgroups.c (mgetgroups) [N_GROUPS_INIT]: Rename enum.
Use a larger value.
Update *groups only upon success.
Iterate upon failed getgrouplist.
id: use getgrouplist when possible2008-02-21T14:02:07ZJames Youngmanjay@gnu.org2008-02-21T14:01:15Zurn:sha1:49f7ebaac45f4d20a70c83c8302444b64259c6d3
* gl/m4/mgetgroups.m4: Check for getgrouplist.
* gl/lib/mgetgroups.c (mgetgroups): Use getgrouplist, if available.
* TODO: Remove the item about switching to getgrouplist.
* NEWS: mention this
Update all copyright notices to use the newer form.2007-07-23T12:35:58ZJim Meyeringjim@meyering.net2007-07-23T12:35:58Zurn:sha1:71aa3ea88084d17bcb4fc1031ad7b66f8647115eChange "version 2" to "version 3" in all copyright notices.2007-07-10T11:35:56ZJim Meyeringjim@meyering.net2007-07-10T10:21:15Zurn:sha1:33342c1a0a464f198df3f177432a0e98e482e7a4Fix typo in comment.2007-07-06T23:49:12ZJim Meyeringjim@meyering.net2007-07-06T23:49:12Zurn:sha1:685d3c3f2adfabbf79b626f84be17170b59d449d