diff options
| author | Jeremy Allison <jra@samba.org> | 2008-05-08 18:09:07 -0700 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2008-05-08 18:09:07 -0700 |
| commit | 00b2cdf75e9bea25034440054b4acd91a179c86d (patch) | |
| tree | 1e049c04c17693bd64c1d8d80a6fd3f0d85c878d /source3/smbd | |
| parent | f7d39653f5c5bea025a77191555231e8a28b25b8 (diff) | |
| download | samba-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.c | 8 |
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); |