diff options
author | Christian Brauner <[email protected]> | 2021-08-26 10:07:05 +0900 |
---|---|---|
committer | Steve French <[email protected]> | 2021-09-03 23:29:45 -0500 |
commit | db7fb6fe3d7a8eb05f2b74c6252771c9362f3b74 (patch) | |
tree | 1fd4ac4ed92908c911947c4937ef96876da31ff3 /lib | |
parent | eb5784f0c6efbe0db720ad7e34e097cea51c1afc (diff) |
ksmbd: remove setattr preparations in set_file_basic_info()
Permission checking and copying over ownership information is the task
of the underlying filesystem not ksmbd. The order is also wrong here.
This modifies the inode before notify_change(). If notify_change() fails
this will have changed ownership nonetheless. All of this is unnecessary
though since the underlying filesystem's ->setattr handler will do all
this (if required) by itself.
Cc: Steve French <[email protected]>
Cc: Christoph Hellwig <[email protected]>
Cc: Namjae Jeon <[email protected]>
Cc: Hyunchul Lee <[email protected]>
Cc: Sergey Senozhatsky <[email protected]>
Cc: [email protected]
Signed-off-by: Christian Brauner <[email protected]>
Signed-off-by: Namjae Jeon <[email protected]>
Signed-off-by: Steve French <[email protected]>
Diffstat (limited to 'lib')
0 files changed, 0 insertions, 0 deletions