diff options
| author | Pavel Shilovsky <pshilov@microsoft.com> | 2019-12-10 20:44:52 +0100 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2019-12-13 07:49:57 +0100 |
| commit | d9191319358da13ee6a332fb9bf745f2181a612a (patch) | |
| tree | 4ab99f916e843ee00f63d7b2c32b062bdef55499 /fs/cifs/smb2proto.h | |
| parent | SMB3: Fix crash in SMB2_open_init due to uninitialized field in compounding path (diff) | |
| download | linux-d9191319358da13ee6a332fb9bf745f2181a612a.tar.xz linux-d9191319358da13ee6a332fb9bf745f2181a612a.zip | |
CIFS: Close cached root handle only if it has a lease
SMB2_tdis() checks if a root handle is valid in order to decide
whether it needs to close the handle or not. However if another
thread has reference for the handle, it may end up with putting
the reference twice. The extra reference that we want to put
during the tree disconnect is the reference that has a directory
lease. So, track the fact that we have a directory lease and
close the handle only in that case.
Signed-off-by: Pavel Shilovsky <pshilov@microsoft.com>
Reviewed-by: Ronnie Sahlberg <lsahlber@redhat.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/cifs/smb2proto.h')
| -rw-r--r-- | fs/cifs/smb2proto.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/cifs/smb2proto.h b/fs/cifs/smb2proto.h index a18272c987fe..27d29f2eb6c8 100644 --- a/fs/cifs/smb2proto.h +++ b/fs/cifs/smb2proto.h @@ -70,6 +70,8 @@ extern int smb3_handle_read_data(struct TCP_Server_Info *server, extern int open_shroot(unsigned int xid, struct cifs_tcon *tcon, struct cifs_fid *pfid); extern void close_shroot(struct cached_fid *cfid); +extern void close_shroot_lease(struct cached_fid *cfid); +extern void close_shroot_lease_locked(struct cached_fid *cfid); extern void move_smb2_info_to_cifs(FILE_ALL_INFO *dst, struct smb2_file_all_info *src); extern int smb2_query_path_info(const unsigned int xid, struct cifs_tcon *tcon, |