diff options
| author | Shyam Prasad N <sprasad@microsoft.com> | 2021-02-03 22:49:52 -0800 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2021-02-16 15:35:57 -0600 |
| commit | 0f56db831456cb4bf85a15c7a900b7138d89b6eb (patch) | |
| tree | 5f4068bd598501855a5c21295de9da82ff1fc33d /fs/cifs/smb2ops.c | |
| parent | 731ddc09c25b0c5037fe51b943cb9b2ff9f18191 (diff) | |
| download | linux-0f56db831456cb4bf85a15c7a900b7138d89b6eb.tar.bz2 | |
cifs: New optype for session operations.
We used to share the CIFS_NEG_OP flag between negotiate and
session authentication. There was an assumption in the code that
CIFS_NEG_OP is used by negotiate only. So introcuded CIFS_SESS_OP
and used it for session setup optypes.
Signed-off-by: Shyam Prasad N <sprasad@microsoft.com>
Reviewed-by: Pavel Shilovsky <pshilov@microsoft.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/cifs/smb2ops.c')
| -rw-r--r-- | fs/cifs/smb2ops.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/cifs/smb2ops.c b/fs/cifs/smb2ops.c index f19274857292..84d1f265aa1d 100644 --- a/fs/cifs/smb2ops.c +++ b/fs/cifs/smb2ops.c @@ -84,7 +84,9 @@ smb2_add_credits(struct TCP_Server_Info *server, pr_warn_once("server overflowed SMB3 credits\n"); } server->in_flight--; - if (server->in_flight == 0 && (optype & CIFS_OP_MASK) != CIFS_NEG_OP) + if (server->in_flight == 0 && + ((optype & CIFS_OP_MASK) != CIFS_NEG_OP) && + ((optype & CIFS_OP_MASK) != CIFS_SESS_OP)) rc = change_conf(server); /* * Sometimes server returns 0 credits on oplock break ack - we need to |