Bug 12241 - vfs_acl_common adds empty ACEs
vfs_acl_common adds empty ACEs
Status: ASSIGNED
Product: Samba 4.1 and newer
Classification: Unclassified
Component: VFS Modules
unspecified
All All
: P5 normal
: ---
Assigned To: Ralph Böhme
Samba QA Contact
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2016-09-06 16:34 UTC by Ralph Böhme
Modified: 2016-09-06 16:34 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ralph Böhme 2016-09-06 16:34:34 UTC
I think we should not add ACEs with zero rights to an ACL:

$ ls -ld /shares/test
drwx------. 2 slow users 4096 Sep  5 15:45 /shares/test

$ ./bin/smbcacls -U slow%x //shares/test ""
REVISION:1
CONTROL:SR|DP
OWNER:SLOWSERVER\slow
GROUP:SLOWSERVER\users
ACL:SLOWSERVER\slow:ALLOWED/0x0/FULL
ACL:SLOWSERVER\users:ALLOWED/0x0/
ACL:Everyone:ALLOWED/0x0/

I think we should just skip the "SLOWSERVER\users" and "Everyone" in this case.

Have patch, need bug number.