Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set Mask Correctly for POSIX ACLs #1253

Closed
MRayermannMSFT opened this issue Mar 26, 2019 · 0 comments
Closed

Set Mask Correctly for POSIX ACLs #1253

MRayermannMSFT opened this issue Mar 26, 2019 · 0 comments
Assignees
Labels
🪲 bug Issue is not intended behavior ✅ merged A fix for this issue has been merged
Milestone

Comments

@MRayermannMSFT
Copy link
Member

The mask is the upper bound of permissions granted to any entry in the group class. The group class consists of named users and named groups plus the owning group. When a user adds a named user or group to the ACL and grants it permissions, the mask must be updated to include those permissions. This is described in https://www.usenix.org/legacy/publications/library/proceedings/usenix03/tech/freenix03/full_papers/gruenbacher/gruenbacher_html/main.html.

So we need to update the mask every time a named user, named group, or the owning group has a change in permissions.

@MRayermannMSFT MRayermannMSFT added the 🪲 bug Issue is not intended behavior label Mar 26, 2019
@MRayermannMSFT MRayermannMSFT added this to the 1.8.0 milestone Mar 26, 2019
@MRayermannMSFT MRayermannMSFT self-assigned this Mar 26, 2019
@MRayermannMSFT MRayermannMSFT added this to Committed in Storage Explorer via automation Mar 26, 2019
@craxal craxal moved this from Committed to Under Review in Storage Explorer Apr 16, 2019
Storage Explorer automation moved this from Under Review to Done Apr 16, 2019
@MRayermannMSFT MRayermannMSFT added the ✅ merged A fix for this issue has been merged label Apr 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪲 bug Issue is not intended behavior ✅ merged A fix for this issue has been merged
Projects
Development

No branches or pull requests

1 participant