[prev in list] [next in list] [prev in thread] [next in thread]
List: linux-cifs
Subject: Re: [CIFS] Charge at least one credit, if server says that it supports multicredit
From: Steve French <smfrench () gmail ! com>
Date: 2013-06-26 22:55:48
Message-ID: CAH2r5muFAgQA5YNZ351V1Kkmar6bkGOH6_W+6bFapSO2Zxm2FA () mail ! gmail ! com
[Download RAW message or body]
Fixed. Thanks for pointing this out
Author: Steve French <smfrench@gmail.com>
Date: Wed Jun 26 17:52:17 2013 -0500
[CIFS] fix static checker warning
Dan Carpenter wrote:
The patch 7f420cee8bd6: "[CIFS] Charge at least one credit, if server
says that it supports multicredit" from Jun 23, 2013, leads to the
following Smatch complaint:
fs/cifs/smb2pdu.c:120 smb2_hdr_assemble()
warn: variable dereferenced before check 'tcon->ses' (see line 115)
CC: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Steve French <smfrench@gmail.com>
diff --git a/fs/cifs/smb2pdu.c b/fs/cifs/smb2pdu.c
index f9b74da..53275bf 100644
--- a/fs/cifs/smb2pdu.c
+++ b/fs/cifs/smb2pdu.c
@@ -111,7 +111,8 @@ smb2_hdr_assemble(struct smb2_hdr *hdr, __le16 smb2_cmd /* c
/* BB FIXME when we do write > 64K add +1 for every 64K in req or rsp */
/* GLOBAL_CAP_LARGE_MTU will only be set if dialect > SMB2.02 */
/* See sections 2.2.4 and 3.2.4.1.5 of MS-SMB2 */
- if (tcon->ses->server->capabilities & SMB2_GLOBAL_CAP_LARGE_MTU)
+ if ((tcon->ses) &&
+ (tcon->ses->server->capabilities & SMB2_GLOBAL_CAP_LARGE_MTU))
hdr->CreditCharge = cpu_to_le16(1);
/* else CreditCharge MBZ */
On Mon, Jun 24, 2013 at 7:24 AM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> Hello Steve French,
>
> This is a semi-automatic email about new static checker warnings.
>
> The patch 7f420cee8bd6: "[CIFS] Charge at least one credit, if server
> says that it supports multicredit" from Jun 23, 2013, leads to the
> following Smatch complaint:
>
> fs/cifs/smb2pdu.c:120 smb2_hdr_assemble()
> warn: variable dereferenced before check 'tcon->ses' (see line 115)
>
> fs/cifs/smb2pdu.c
> 114 if (tcon->ses->server->capabilities & SMB2_GLOBAL_CAP_LARGE_MTU)
> ^^^^^^^^^^^
> New dereference.
>
> 115 hdr->CreditCharge = cpu_to_le16(1);
> 116 /* else CreditCharge MBZ */
> 117
> 118 hdr->TreeId = tcon->tid;
> 119 /* Uid is not converted */
> 120 if (tcon->ses)
> ^^^^^^^^^
> Old check.
>
> 121 hdr->SessionId = tcon->ses->Suid;
> 122 /* BB check following DFS flags BB */
>
> regards,
> dan carpenter
--
Thanks,
Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic