diff options
| author | Namjae Jeon <namjae.jeon@samsung.com> | 2014-08-20 12:39:41 +0200 |
|---|---|---|
| committer | Steve French <smfrench@gmail.com> | 2014-08-21 19:12:59 +0200 |
| commit | 7de975e349b295f387f34eed38f115223f17d5ee (patch) | |
| tree | b02d7afcccf57ccd3b61837e35c7e3486f285cf0 /fs | |
| parent | cifs: fix memory leak when password is supplied multiple times (diff) | |
| download | linux-7de975e349b295f387f34eed38f115223f17d5ee.tar.xz linux-7de975e349b295f387f34eed38f115223f17d5ee.zip | |
cifs: fix a possible use of uninit variable in SMB2_sess_setup
In case of error, goto ssetup_exit can be hit and we could end up using
uninitialized value of resp_buftype
Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: Ashish Sangwan <a.sangwan@samsung.com>
Signed-off-by: Steve French <smfrench@gmail.com>
Diffstat (limited to 'fs')
| -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 fa0dd044213b..9df5d8effe47 100644 --- a/fs/cifs/smb2pdu.c +++ b/fs/cifs/smb2pdu.c @@ -530,7 +530,7 @@ SMB2_sess_setup(const unsigned int xid, struct cifs_ses *ses, struct smb2_sess_setup_rsp *rsp = NULL; struct kvec iov[2]; int rc = 0; - int resp_buftype; + int resp_buftype = CIFS_NO_BUFFER; __le32 phase = NtLmNegotiate; /* NTLMSSP, if needed, is multistage */ struct TCP_Server_Info *server = ses->server; u16 blob_length = 0; |