diff options
author | Steve French <smfrench@gmail.com> | 2015-03-25 19:51:57 -0400 |
---|---|---|
committer | Steve French <smfrench@gmail.com> | 2015-04-01 01:01:46 -0400 |
commit | 75fdfc849ae0636853add4b7dbdc5753160ad0cb (patch) | |
tree | 1d8368ae3deaa9d577ce307050fd7e5f8917eefd | |
parent | 2bd50fb3d4d31f5168ecea221f291534cd0a96e9 (diff) |
Fix warning on uninitialized buftype
Pointed out by coverity analyzer. resp_buftype is
not initialized in one path which can rarely log
a spurious warning (buf is null so there will
not be a problem with freeing data, but if buf_type
were randomly set to wrong value could log a warning)
Reported by Coverity (CID 1269144)
Signed-off-by: Steve French <smfrench@gmail.com>
Acked-by: Shirish Pargaonkar <shirishpargaonkar@gmail.com>
Acked-by: Sachin Prabhu <sprabhu@redhat.com>
Reviewed-by: Jeff Layton <jlayton@poochiereds.net>
-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 3417340bf89e..1b906debe11d 100644 --- a/fs/cifs/smb2pdu.c +++ b/fs/cifs/smb2pdu.c | |||
@@ -2114,7 +2114,7 @@ SMB2_query_directory(const unsigned int xid, struct cifs_tcon *tcon, | |||
2114 | struct kvec iov[2]; | 2114 | struct kvec iov[2]; |
2115 | int rc = 0; | 2115 | int rc = 0; |
2116 | int len; | 2116 | int len; |
2117 | int resp_buftype; | 2117 | int resp_buftype = CIFS_NO_BUFFER; |
2118 | unsigned char *bufptr; | 2118 | unsigned char *bufptr; |
2119 | struct TCP_Server_Info *server; | 2119 | struct TCP_Server_Info *server; |
2120 | struct cifs_ses *ses = tcon->ses; | 2120 | struct cifs_ses *ses = tcon->ses; |