mirror of
https://github.com/s3fs-fuse/s3fs-fuse.git
synced 2026-04-25 05:16:00 +03:00
[PR #605] [CLOSED] Group permission checks sometimes fail with large number of groups #1568
Labels
No labels
bug
bug
dataloss
duplicate
enhancement
feature request
help wanted
invalid
need info
performance
pull-request
question
question
testing
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/s3fs-fuse#1568
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/605
Author: @tlevi
Created: 5/22/2017
Status: ❌ Closed
Base:
master← Head:master📝 Commits (2)
2b7ea58Expand buffer for group information if too small and retry70bb649Fix chown_nocopy losing existing uid/gid if unspecified📊 Changes
2 files changed (+18 additions, -20 deletions)
View changed files
📝
src/s3fs.cpp(+7 -18)📝
src/s3fs_util.cpp(+11 -2)📄 Description
On some of my systems users cannot access directories, which they have group membership to access.
One if the groups in particular is quite large, with about 50 members, but I've also got ~150 groups and users in total.
I've traced the issue back to the getgrgid_r call within is_uid_include_group failing with ERANGE, indicating the buffer is too small.
The patch in this pull request solves the problem by expanding the buffer and trying again until it works, but there might be a better way.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.