diff --git a/ChangeLog b/ChangeLog index 5505dc53e5..d4abd10507 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +Tue Mar 5 01:03:16 2013 KOSAKI Motohiro + + * file.c (rb_group_member): get rid of NGROUPS dependency. + [Bug #7886] [ruby-core:52537] + Tue Mar 5 00:16:56 2013 Eric Hodel * ext/openssl/ossl.c (class OpenSSL): Use only inner parenthesis in diff --git a/file.c b/file.c index 08cd5f8dcf..1cc6066e99 100644 --- a/file.c +++ b/file.c @@ -1013,39 +1013,52 @@ rb_file_lstat(VALUE obj) #endif } +/* Linux allow 65536 groups and it is maximum value as far as we know. */ +#define RUBY_GROUP_MAX 65536 + static int rb_group_member(GETGROUPS_T gid) { +#ifdef _WIN32 + return FALSE; +#else int rv = FALSE; -#ifndef _WIN32 + int groups = 16; + VALUE v = 0; + GETGROUPS_T *gary; + int anum; + if (getgid() == gid || getegid() == gid) return TRUE; -# ifdef HAVE_GETGROUPS -# ifndef NGROUPS -# ifdef NGROUPS_MAX -# define NGROUPS NGROUPS_MAX -# else -# define NGROUPS 32 -# endif -# endif - { - GETGROUPS_T *gary; - int anum; - - gary = xmalloc(NGROUPS * sizeof(GETGROUPS_T)); - anum = getgroups(NGROUPS, gary); - while (--anum >= 0) { - if (gary[anum] == gid) { - rv = TRUE; - break; - } + /* + * On Mac OS X (Mountain Lion), NGROUPS is 16. But libc and kernel + * accept more larger value. + * So we don't trunk NGROUPS anymore. + */ + while (groups <= RUBY_GROUP_MAX) { + gary = ALLOCV_N(GETGROUPS_T, v, groups); + anum = getgroups(groups, gary); + if (anum != groups) + break; + groups *= 2; + if (v) { + ALLOCV_END(v); + v = 0; } - xfree(gary); } -# endif -#endif + + while (--anum >= 0) { + if (gary[anum] == gid) { + rv = TRUE; + break; + } + } + if (v) + ALLOCV_END(v); + return rv; +#endif } #ifndef S_IXUGO diff --git a/version.h b/version.h index fe49ed840a..b619e191e3 100644 --- a/version.h +++ b/version.h @@ -1,6 +1,6 @@ #define RUBY_VERSION "2.0.0" #define RUBY_RELEASE_DATE "2013-03-05" -#define RUBY_PATCHLEVEL 17 +#define RUBY_PATCHLEVEL 18 #define RUBY_RELEASE_YEAR 2013 #define RUBY_RELEASE_MONTH 3