git.samba.org / kai / samba.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 133fb0e) | patch
s3:modules fix Bug 8330 NFSv4 ACL merging logic is broken
authorChristian Ambach <ambi@samba.org>
Wed, 27 Jul 2011 12:46:00 +0000 (14:46 +0200)
committerChristian Ambach <ambi@samba.org>
Wed, 27 Jul 2011 14:03:51 +0000 (16:03 +0200)
commitabf3573c21510717edb0fb9ce5080c1a24c35714
tree3e8134d974408615f2b673117f859ded27b8fd07tree
parent133fb0ebcc070bd3ae4aebcc3a759eca98101495commit | diff
s3:modules fix Bug 8330 NFSv4 ACL merging logic is broken

we should not merge ACEs with different flags (e.g. CI/OI/I/)
Otherwise ACLs get wrong entries and thus wrong semantics

Example:
ACL:BUILTIN\Users:ALLOWED/0x0/FULL
ACL:BUILTIN\Users:ALLOWED/I/READ
got merged to
ACL:BUILTIN\Users:ALLOWED/I/FULL

This is not the same and also leads to wrong displays
in the Windows ACL dialog

Autobuild-User: Christian Ambach <ambi@samba.org>
Autobuild-Date: Wed Jul 27 16:03:51 CEST 2011 on sn-devel-104
source3/modules/nfs4_acls.c diff | blob | history
Kai's WIP code