diff options
author | Jeremy Allison <jra@samba.org> | 2012-10-02 10:15:54 -0700 |
---|---|---|
committer | Karolin Seeger <kseeger@samba.org> | 2012-10-05 09:51:00 +0200 |
commit | 7dcb017fc1d8e8af5878b2b0139686829c0c1594 (patch) | |
tree | 93f79db64fd04a8b7609eae36cbdba6644818a33 | |
parent | 580f61622c449aee8420e3519e764706d11c20fc (diff) | |
download | samba-7dcb017fc1d8e8af5878b2b0139686829c0c1594.tar.gz |
When setting a non-default ACL, don't forget to apply masks to SMB_ACL_USER and SMB_ACL_GROUP entries.
Fix bug #9236 - ACL masks incorrectly applied when setting ACLs.
-rw-r--r-- | source3/smbd/posix_acls.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/source3/smbd/posix_acls.c b/source3/smbd/posix_acls.c index f7258bd66d7..646efa4b69c 100644 --- a/source3/smbd/posix_acls.c +++ b/source3/smbd/posix_acls.c @@ -1386,6 +1386,16 @@ static bool ensure_canon_entry_valid(canon_ace **pp_ace, } got_other = True; pace_other = pace; + + } else if (pace->type == SMB_ACL_USER || pace->type == SMB_ACL_GROUP) { + + /* + * Ensure create mask/force create mode is respected on set. + */ + + if (setting_acl && !is_default_acl) { + apply_default_perms(params, is_directory, pace, S_IRGRP); + } } } |