diff options
| author | Steve French <smfrench@gmail.com> | 2015-03-26 00:51:57 +0100 |
|---|---|---|
| committer | Steve French <smfrench@gmail.com> | 2015-04-01 07:01:46 +0200 |
| commit | 75fdfc849ae0636853add4b7dbdc5753160ad0cb (patch) | |
| tree | 1d8368ae3deaa9d577ce307050fd7e5f8917eefd /fs/cifs | |
| parent | cifs: potential memory leaks when parsing mnt opts (diff) | |
| download | linux-75fdfc849ae0636853add4b7dbdc5753160ad0cb.tar.xz linux-75fdfc849ae0636853add4b7dbdc5753160ad0cb.zip | |
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>
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 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, struct kvec iov[2]; int rc = 0; int len; - int resp_buftype; + int resp_buftype = CIFS_NO_BUFFER; unsigned char *bufptr; struct TCP_Server_Info *server; struct cifs_ses *ses = tcon->ses; |