summaryrefslogtreecommitdiff
path: root/source3/smbd
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2008-05-08 18:09:07 -0700
committerJeremy Allison <jra@samba.org>2008-05-08 18:09:07 -0700
commit00b2cdf75e9bea25034440054b4acd91a179c86d (patch)
tree1e049c04c17693bd64c1d8d80a6fd3f0d85c878d /source3/smbd
parentf7d39653f5c5bea025a77191555231e8a28b25b8 (diff)
downloadsamba-00b2cdf75e9bea25034440054b4acd91a179c86d.tar.gz
samba-00b2cdf75e9bea25034440054b4acd91a179c86d.tar.bz2
samba-00b2cdf75e9bea25034440054b4acd91a179c86d.zip
Yay ! Remove a VFS entry. Removed the set_nt_acl() call,
this can only be done via fset_nt_acl() using an open file/directory handle. I'd like to do the same with get_nt_acl() but am concerned about efficiency problems with "hide unreadable/hide unwritable" when doing a directory listing (this would mean opening every file in the dir on list). Moving closer to rationalizing the ACL model and maybe moving the POSIX calls into a posix_acl VFS module rather than having them as first class citizens of the VFS. Jeremy. (This used to be commit f487f742cb903a06fbf2be006ddc9ce9063339ed)
Diffstat (limited to 'source3/smbd')
-rw-r--r--source3/smbd/nttrans.c8
1 files changed, 1 insertions, 7 deletions
diff --git a/source3/smbd/nttrans.c b/source3/smbd/nttrans.c
index ec4126c1f7..362823d78a 100644
--- a/source3/smbd/nttrans.c
+++ b/source3/smbd/nttrans.c
@@ -770,13 +770,7 @@ static NTSTATUS set_sd(files_struct *fsp, uint8 *data, uint32 sd_len,
security_info_sent &= ~DACL_SECURITY_INFORMATION;
}
- if (fsp->fh->fd != -1) {
- status = SMB_VFS_FSET_NT_ACL(fsp, security_info_sent, psd);
- }
- else {
- status = SMB_VFS_SET_NT_ACL(fsp, fsp->fsp_name,
- security_info_sent, psd);
- }
+ status = SMB_VFS_FSET_NT_ACL(fsp, security_info_sent, psd);
TALLOC_FREE(psd);