diff options
| author |   Steve French <stfrench@microsoft.com> | 2020-05-31 14:36:56 -0500 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2020-06-04 13:50:55 -0500 |
| commit | edb161353680e6d488d94cbcaf967745bee98d17 (patch) | |
| tree | b416a7d4eab9daa8a8267d8a9dabe67eb8ea78b9 /fs/cifs | |
| parent | cifs: multichannel: move channel selection above transport layer (diff) | |
| download | wireguard-linux-edb161353680e6d488d94cbcaf967745bee98d17.tar.xz wireguard-linux-edb161353680e6d488d94cbcaf967745bee98d17.zip | |
smb3: remove static checker warning
Remove static checker warning pointed out by Dan Carpenter:
The patch feeaec621c09: "cifs: multichannel: move channel selection
above transport layer" from Apr 24, 2020, leads to the following
static checker warning:
fs/cifs/smb2pdu.c:149 smb2_hdr_assemble()
error: we previously assumed 'tcon->ses' could be null (see line 133)
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
CC: Aurelien Aptel <aptel@suse.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/cifs')
| -rw-r--r-- | fs/cifs/smb2pdu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifs/smb2pdu.c b/fs/cifs/smb2pdu.c index fe5a8452b213..2497e0e428ac 100644 --- a/fs/cifs/smb2pdu.c +++ b/fs/cifs/smb2pdu.c @@ -85,7 +85,7 @@ static const int smb2_req_struct_sizes[NUMBER_OF_SMB2_COMMANDS] = { int smb3_encryption_required(const struct cifs_tcon *tcon) { - if (!tcon) + if (!tcon || !tcon->ses) return 0; if ((tcon->ses->session_flags & SMB2_SESSION_FLAG_ENCRYPT_DATA) || (tcon->share_flags & SHI1005_FLAGS_ENCRYPT_DATA)) |