cifs: Do not set cifs/ntfs acl using a file handle (try #4)
authorShirish Pargaonkar <shirishpargaonkar@gmail.com>
Tue, 9 Aug 2011 19:30:39 +0000 (14:30 -0500)
committerSteve French <sfrench@us.ibm.com>
Thu, 11 Aug 2011 18:23:45 +0000 (18:23 +0000)
commite22906c564c2f9c73ee4621ef3b93fe374539f00
tree4f59f2616b6723204dccc28ec3d8d6033573e111
parent789e66612367f9975d704c9e4990025cbbbb45ec
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>
fs/cifs/cifsacl.c