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

Add configuration to enable POSIX ACL for client #906

Merged
merged 1 commit into from
Sep 18, 2020

Conversation

yinlei-github
Copy link
Contributor

@yinlei-github yinlei-github commented Sep 10, 2020

What this PR does / why we need it:

  • Fix CFS-01-004 WP1: Linux file permissions ineffective for ACL (High)

Which issue this PR fixes:
None.

Special notes for your reviewer:
None.

Release note:

Security
- Fix CFS-01-004 WP1: Linux file permissions ineffective for ACL (High)

@mervinkid mervinkid added the security Topic about security and authorization label Sep 10, 2020
@mervinkid mervinkid changed the title add posix acl enable config for client Add configuration to enable POSIX ACL for client Sep 10, 2020
Copy link
Member

@mervinkid mervinkid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Supplement and modificatIon copyright information to following modified third-part source code files:

  • vendor/bazil.org/fuse/fuse_kernel.go
  • vendor/bazil.org/fuse/options.go

Add modification work of bazil.org/fuse to NOTICE file.

mervinkid
mervinkid previously approved these changes Sep 16, 2020
@mervinkid mervinkid merged commit 91ae394 into cubefs:master Sep 18, 2020
leonrayang pushed a commit to leonrayang/cubefs that referenced this pull request Feb 11, 2025
Co-authored-by: HackMD <37423+hackmd-hub[bot]@users.noreply.github.com>
Signed-off-by: Nate W <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority/high security Topic about security and authorization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants