summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2012-05-21 11:44:09 +0200
committerStefan Metzmacher <metze@samba.org>2012-05-21 19:27:43 +0200
commitce11eb5b9427e4ba5b86c6cd0378a7300ce1218f (patch)
tree85234d77967c1014194f3b683136ada4f491aeaf
parent1d53e57bbfc6ea7ef4f18f41a9cba61f19d66793 (diff)
downloadsamba-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.c10
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);