diff options
| author | Shyam Prasad N <nspmangalore@gmail.com> | 2023-07-14 11:56:34 +0300 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2023-07-14 19:24:14 +0300 |
| commit | c071b34f62ddbf8435491ebb0e21eba9dc29f901 (patch) | |
| tree | 81e24b9dc33dd34779fbf94713bb75b9dfa1fe41 /fs/smb/client/connect.c | |
| parent | bf99f6be2d20146942bce6f9e90a0ceef12cbc1e (diff) | |
| download | linux-c071b34f62ddbf8435491ebb0e21eba9dc29f901.tar.xz | |
cifs: is_network_name_deleted should return a bool
Currently, is_network_name_deleted and it's implementations
do not return anything if the network name did get deleted.
So the function doesn't fully achieve what it advertizes.
Changed the function to return a bool instead. It will now
return true if the error returned is STATUS_NETWORK_NAME_DELETED
and the share (tree id) was found to be connected. It returns
false otherwise.
Signed-off-by: Shyam Prasad N <sprasad@microsoft.com>
Acked-by: Paulo Alcantara (SUSE) <pc@manguebit.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/smb/client/connect.c')
| -rw-r--r-- | fs/smb/client/connect.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/fs/smb/client/connect.c b/fs/smb/client/connect.c index 85dd1b373974..aad0edc5d5d1 100644 --- a/fs/smb/client/connect.c +++ b/fs/smb/client/connect.c @@ -1226,9 +1226,14 @@ next_pdu: if (mids[i] != NULL) { mids[i]->resp_buf_size = server->pdu_size; - if (bufs[i] && server->ops->is_network_name_deleted) - server->ops->is_network_name_deleted(bufs[i], - server); + if (bufs[i] != NULL) { + if (server->ops->is_network_name_deleted && + server->ops->is_network_name_deleted(bufs[i], + server)) { + cifs_server_dbg(FYI, + "Share deleted. Reconnect needed"); + } + } if (!mids[i]->multiRsp || mids[i]->multiEnd) mids[i]->callback(mids[i]); |