diff options
| author | David Disseldorp <ddiss@suse.de> | 2011-09-21 21:50:29 +0200 |
|---|---|---|
| committer | David Disseldorp <ddiss@samba.org> | 2011-09-29 13:14:21 +0200 |
| commit | bd5e9758f8350b5137dccf0d5ebf24aefb970997 (patch) | |
| tree | 458a2c859fc211961da4ef099a2a834e94435305 /source4/torture/smb2 | |
| parent | 27195b3ac5970037445e8d05efb77b39cbfdd9ba (diff) | |
| download | samba-bd5e9758f8350b5137dccf0d5ebf24aefb970997.tar.gz samba-bd5e9758f8350b5137dccf0d5ebf24aefb970997.tar.bz2 samba-bd5e9758f8350b5137dccf0d5ebf24aefb970997.zip | |
s4-torture: remove unchecked read from smb2 create
Add a corresponding test case to smb2.read.
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Diffstat (limited to 'source4/torture/smb2')
| -rw-r--r-- | source4/torture/smb2/read.c | 9 | ||||
| -rw-r--r-- | source4/torture/smb2/util.c | 9 |
2 files changed, 9 insertions, 9 deletions
diff --git a/source4/torture/smb2/read.c b/source4/torture/smb2/read.c index f27ce24413..92b3ce0cf7 100644 --- a/source4/torture/smb2/read.c +++ b/source4/torture/smb2/read.c @@ -57,9 +57,18 @@ static bool test_read_eof(struct torture_context *torture, struct smb2_tree *tre ZERO_STRUCT(buf); + smb2_util_unlink(tree, FNAME); + status = torture_smb2_testfile(tree, FNAME, &h); CHECK_STATUS(status, NT_STATUS_OK); + ZERO_STRUCT(rd); + rd.in.file.handle = h; + rd.in.length = 5; + rd.in.offset = 0; + status = smb2_read(tree, tree, &rd); + CHECK_STATUS(status, NT_STATUS_END_OF_FILE); + status = smb2_util_write(tree, h, buf, 0, ARRAY_SIZE(buf)); CHECK_STATUS(status, NT_STATUS_OK); diff --git a/source4/torture/smb2/util.c b/source4/torture/smb2/util.c index 54aea81e0b..a94b0d76b6 100644 --- a/source4/torture/smb2/util.c +++ b/source4/torture/smb2/util.c @@ -298,7 +298,6 @@ NTSTATUS torture_smb2_testfile(struct smb2_tree *tree, const char *fname, struct smb2_handle *handle) { struct smb2_create io; - struct smb2_read r; NTSTATUS status; ZERO_STRUCT(io); @@ -318,14 +317,6 @@ NTSTATUS torture_smb2_testfile(struct smb2_tree *tree, const char *fname, *handle = io.out.file.handle; - ZERO_STRUCT(r); - r.in.file.handle = *handle; - r.in.length = 5; - r.in.offset = 0; - - // What is the purpose of this? Server returns EOF. - smb2_read(tree, tree, &r); - return NT_STATUS_OK; } |