diff options
author | 2011-08-09 14:30:39 -0500 | |
---|---|---|
committer | 2011-08-11 18:23:45 +0000 | |
commit | e22906c564c2f9c73ee4621ef3b93fe374539f00 (patch) | |
tree | 4f59f2616b6723204dccc28ec3d8d6033573e111 /lib/dma-debug.c | |
parent | [CIFS] Cleanup use of CONFIG_CIFS_STATS2 ifdef to make transport routines more readable (diff) | |
download | wireguard-linux-e22906c564c2f9c73ee4621ef3b93fe374539f00.tar.xz wireguard-linux-e22906c564c2f9c73ee4621ef3b93fe374539f00.zip |
cifs: Do not set cifs/ntfs acl using a file handle (try #4)
Set security descriptor using path name instead of a file handle.
We can't be sure that the file handle has adequate permission to
set a security descriptor (to modify DACL).
Function set_cifs_acl_by_fid() has been removed since we can't be
sure how a file was opened for writing, a valid request can fail
if the file was not opened with two above mentioned permissions.
We could have opted to add on WRITE_DAC and WRITE_OWNER permissions
to file opens and then use that file handle but adding addtional
permissions such as WRITE_DAC and WRITE_OWNER could cause an
any open to fail.
And it was incorrect to look for read file handle to set a
security descriptor anyway.
Signed-off-by: Shirish Pargaonkar <shirishpargaonkar@gmail.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'lib/dma-debug.c')
0 files changed, 0 insertions, 0 deletions