diff options
| author | Stefan Metzmacher <metze@samba.org> | 2012-05-21 11:44:09 +0200 |
|---|---|---|
| committer | Stefan Metzmacher <metze@samba.org> | 2012-05-21 19:27:43 +0200 |
| commit | ce11eb5b9427e4ba5b86c6cd0378a7300ce1218f (patch) | |
| tree | 85234d77967c1014194f3b683136ada4f491aeaf | |
| parent | 1d53e57bbfc6ea7ef4f18f41a9cba61f19d66793 (diff) | |
| download | samba-ce11eb5b9427e4ba5b86c6cd0378a7300ce1218f.tar.gz samba-ce11eb5b9427e4ba5b86c6cd0378a7300ce1218f.tar.bz2 samba-ce11eb5b9427e4ba5b86c6cd0378a7300ce1218f.zip | |
s3:smb2_ioctl: Fix Coverity ID 701771 Uninitialized scalar variable
metze
Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Mon May 21 19:27:44 CEST 2012 on sn-devel-104
| -rw-r--r-- | source3/smbd/smb2_ioctl.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/source3/smbd/smb2_ioctl.c b/source3/smbd/smb2_ioctl.c index b1a9e32f36..37acf11767 100644 --- a/source3/smbd/smb2_ioctl.c +++ b/source3/smbd/smb2_ioctl.c @@ -417,6 +417,11 @@ static struct tevent_req *smbd_smb2_ioctl_send(TALLOC_CTX *mem_ctx, in_security_mode = SVAL(in_input.data, 0x14); in_max_dialect = SVAL(in_input.data, 0x16); + status = GUID_from_ndr_blob(&in_guid_blob, &in_guid); + if (tevent_req_nterror(req, status)) { + return tevent_req_post(req, ev); + } + max_dialect = conn->smb2.client.dialects[conn->smb2.client.num_dialects-1]; if (in_max_dialect != max_dialect) { state->disconnect = true; @@ -494,6 +499,11 @@ static struct tevent_req *smbd_smb2_ioctl_send(TALLOC_CTX *mem_ctx, return tevent_req_post(req, ev); } + status = GUID_from_ndr_blob(&in_guid_blob, &in_guid); + if (tevent_req_nterror(req, status)) { + return tevent_req_post(req, ev); + } + if (in_num_dialects != conn->smb2.client.num_dialects) { state->disconnect = true; tevent_req_nterror(req, NT_STATUS_ACCESS_DENIED); |