Assaf Yaari has uploaded this change for review.

View Change

commonlib: In fsal_mode_to_acl provide a flag whether to dup inherit
aces

Currently the fsal_mode_to_acl dups inherit aces which don't have the
inherit only flag.
According to RFC 8881 6.4.3.1 this is an optional behavior by the
server:
"When a new directory is created, the server MAY split any inherited ACE
that is both inheritable and effective (in other words, that has neither
ACE4_INHERIT_ONLY_ACE nor ACE4_NO_PROPAGATE_INHERIT_ACE set), into two
ACEs, one with no inheritance flags and one with ACE4_INHERIT_ONLY_ACE
set."

Adding a flag so the caller can decide whether to dup or not, as
back-ends might have limited number of slots for storing aces.

Signed-off-by: Assaf Yaari <assafy@google.com>
Change-Id: I0835ca744485e380b0b72e8308bf11f5582c4159
---
M src/FSAL/FSAL_VFS/file.c
M src/FSAL/commonlib.c
M src/Protocols/NFS/nfs_proto_tools.c
M src/include/FSAL/fsal_commonlib.h
4 files changed, 8 insertions(+), 6 deletions(-)

git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/11/1190111/1

To view, visit change 1190111. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newchange
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I0835ca744485e380b0b72e8308bf11f5582c4159
Gerrit-Change-Number: 1190111
Gerrit-PatchSet: 1
Gerrit-Owner: Assaf Yaari <assafy@google.com>