diff options
| author | kosaki <kosaki@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-02-21 04:52:38 +0000 |
|---|---|---|
| committer | kosaki <kosaki@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-02-21 04:52:38 +0000 |
| commit | 8a9dc061f5737543eaca17ceafa6ebe4886cb26e (patch) | |
| tree | 8477bfac7895f5af6593e334a993544f43124b15 /file.c | |
| parent | e93a68abfa10a7eba1fd769b6cbc55da7796f8e8 (diff) | |
* file.c (rb_group_member): added an error check. SUS says,
getgroups(small_value) may return EINVAL.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@39353 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'file.c')
| -rw-r--r-- | file.c | 4 |
1 files changed, 3 insertions, 1 deletions
@@ -1036,7 +1036,7 @@ rb_group_member(GETGROUPS_T gid) while (groups <= RB_MAX_GROUPS) { gary = ALLOCV_N(GETGROUPS_T, v, groups); anum = getgroups(groups, gary); - if (anum != groups) + if (anum != -1 && anum != groups) break; groups *= 2; if (v) { @@ -1044,6 +1044,8 @@ rb_group_member(GETGROUPS_T gid) v = 0; } } + if (anum == -1) + return FALSE; while (--anum >= 0) { if (gary[anum] == gid) { |