summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2012-01-23 11:20:52 -0800
committerKarolin Seeger <kseeger@samba.org>2012-01-23 21:44:08 +0100
commit2f90581a5e68da48d03127c453a0c3becb2170e2 (patch)
tree335f6c9ac90e096b4ae7d861497725c2b682fe6e
parentd6f0ddae114ed77e0afec4f9c72b55a41fd800eb (diff)
downloadsamba-2f90581a5e68da48d03127c453a0c3becb2170e2.tar.gz
Second part of fix for 8636 - When returning an ACL without SECINFO_DACL requested, we still set SEC_DESC_DACL_PRESENT in the type field.
Ensure we always ask for the set: OWNER_SECURITY_INFORMATION | GROUP_SECURITY_INFORMATION | DACL_SECURITY_INFORMATION | SACL_SECURITY_INFORMATION when getting an ACL inside the module. (cherry picked from commit 4eb17aff84dc0dccec23e066db7a88581cf7668c)
-rw-r--r--source3/modules/vfs_acl_common.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/source3/modules/vfs_acl_common.c b/source3/modules/vfs_acl_common.c
index 75917b7c09e..016bb9f5e2e 100644
--- a/source3/modules/vfs_acl_common.c
+++ b/source3/modules/vfs_acl_common.c
@@ -539,7 +539,8 @@ static NTSTATUS get_parent_acl_common(vfs_handle_struct *handle,
parent_name,
(SECINFO_OWNER |
SECINFO_GROUP |
- SECINFO_DACL),
+ SECINFO_DACL |
+ SECINFO_SACL),
pp_parent_desc);
if (!NT_STATUS_IS_OK(status)) {
@@ -622,7 +623,8 @@ static int open_acl_common(vfs_handle_struct *handle,
fname,
(SECINFO_OWNER |
SECINFO_GROUP |
- SECINFO_DACL),
+ SECINFO_DACL |
+ SECINFO_SACL),
&pdesc);
if (NT_STATUS_IS_OK(status)) {
/* See if we can access it. */