diff options
| author | Germano Percossi <germano.percossi@citrix.com> | 2017-04-07 12:29:37 +0100 |
|---|---|---|
| committer | Steve French <smfrench@gmail.com> | 2017-04-10 23:36:39 -0500 |
| commit | a0918f1ce6a43ac980b42b300ec443c154970979 (patch) | |
| tree | 14b785e0423455cfc826ca41c894d8c0be8e3a5a /fs/cifs/smb2pdu.c | |
| parent | 18ea43113f5b74a97dd4be9bddbac10d68b1a6ce (diff) | |
| download | lwn-a0918f1ce6a43ac980b42b300ec443c154970979.tar.gz lwn-a0918f1ce6a43ac980b42b300ec443c154970979.zip | |
CIFS: remove bad_network_name flag
STATUS_BAD_NETWORK_NAME can be received during node failover,
causing the flag to be set and making the reconnect thread
always unsuccessful, thereafter.
Once the only place where it is set is removed, the remaining
bits are rendered moot.
Removing it does not prevent "mount" from failing when a non
existent share is passed.
What happens when the share really ceases to exist while the
share is mounted is undefined now as much as it was before.
Signed-off-by: Germano Percossi <germano.percossi@citrix.com>
Reviewed-by: Pavel Shilovsky <pshilov@microsoft.com>
CC: Stable <stable@vger.kernel.org>
Signed-off-by: Steve French <smfrench@gmail.com>
Diffstat (limited to 'fs/cifs/smb2pdu.c')
| -rw-r--r-- | fs/cifs/smb2pdu.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/fs/cifs/smb2pdu.c b/fs/cifs/smb2pdu.c index d09e98bb8584..1bd5d3033fc8 100644 --- a/fs/cifs/smb2pdu.c +++ b/fs/cifs/smb2pdu.c @@ -1171,9 +1171,6 @@ SMB2_tcon(const unsigned int xid, struct cifs_ses *ses, const char *tree, else return -EIO; - if (tcon && tcon->bad_network_name) - return -ENOENT; - unc_path = kmalloc(MAX_SHARENAME_LENGTH * 2, GFP_KERNEL); if (unc_path == NULL) return -ENOMEM; @@ -1277,8 +1274,6 @@ tcon_exit: tcon_error_exit: if (rsp->hdr.sync_hdr.Status == STATUS_BAD_NETWORK_NAME) { cifs_dbg(VFS, "BAD_NETWORK_NAME: %s\n", tree); - if (tcon) - tcon->bad_network_name = true; } goto tcon_exit; } |