diff options
author | Shirish Pargaonkar <shirishpargaonkar@gmail.com> | 2011-08-09 14:30:39 -0500 |
---|---|---|
committer | Steve French <sfrench@us.ibm.com> | 2011-08-11 18:23:45 +0000 |
commit | e22906c564c2f9c73ee4621ef3b93fe374539f00 (patch) | |
tree | 4f59f2616b6723204dccc28ec3d8d6033573e111 /fs/notify | |
parent | 789e66612367f9975d704c9e4990025cbbbb45ec (diff) | |
download | op-kernel-dev-e22906c564c2f9c73ee4621ef3b93fe374539f00.zip op-kernel-dev-e22906c564c2f9c73ee4621ef3b93fe374539f00.tar.gz |
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 'fs/notify')
0 files changed, 0 insertions, 0 deletions