From mboxrd@z Thu Jan 1 00:00:00 1970 From: Martin Doucha Date: Fri, 13 Mar 2020 16:36:31 +0100 Subject: [LTP] [PATCH 2/2] Add new test cases to syscalls/readv01 In-Reply-To: <20200313153631.751-1-mdoucha@suse.cz> References: <20200313153631.751-1-mdoucha@suse.cz> Message-ID: <20200313153631.751-2-mdoucha@suse.cz> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ltp@lists.linux.it Split the original test scenario into two test cases and add: - read into buffers bigger than input file - read into multiple buffers - read into non-NULL buffer with size = 0 (test for kernel commit 19f18459) Also use guarded buffers in all IO vectors. Fixes #382 Signed-off-by: Martin Doucha --- testcases/kernel/syscalls/readv/readv01.c | 90 +++++++++++++++-------- 1 file changed, 60 insertions(+), 30 deletions(-) diff --git a/testcases/kernel/syscalls/readv/readv01.c b/testcases/kernel/syscalls/readv/readv01.c index ad0ab187b..fc17100eb 100644 --- a/testcases/kernel/syscalls/readv/readv01.c +++ b/testcases/kernel/syscalls/readv/readv01.c @@ -20,57 +20,78 @@ #include "tst_test.h" +/* Note: multi_iovec test assumes CHUNK is divisible by 4 */ #define CHUNK 64 static char buf[CHUNK]; +static struct iovec *rd_iovec, *big_iovec, *multi_iovec, *lockup_iovec; +static int fd; -static struct iovec rd_iovec[] = { - {buf, CHUNK}, - {NULL, 0}, - {NULL, 0}, +static struct testcase { + struct iovec **iov; + int iov_count, exp_ret; + const char *name; +} testcase_list[] = { + {&rd_iovec, 0, 0, "readv() with 0 I/O vectors"}, + {&rd_iovec, 3, CHUNK, "readv() with NULL I/O vectors"}, + {&big_iovec, 2, CHUNK, "readv() with too big I/O vectors"}, + {&multi_iovec, 2, 3*CHUNK/4, "readv() with multiple I/O vectors"}, + {&lockup_iovec, 2, CHUNK, "readv() with zero-len buffer"}, }; -static int fd; - -static void run(void) +static void test_readv(unsigned int n) { - int i, fail; - char *vec; + int i, fpos, fail = 0; + size_t j; + char *ptr; + const struct testcase *tc = testcase_list + n; + struct iovec *vec; SAFE_LSEEK(fd, 0, SEEK_SET); + vec = *tc->iov; - if (readv(fd, rd_iovec, 0) == -1) - tst_res(TFAIL | TERRNO, "readv failed unexpectedly"); - else - tst_res(TPASS, "readv read 0 io vectors"); + for (i = 0; i < tc->iov_count; i++) + if (vec[i].iov_base && vec[i].iov_len) + memset(vec[i].iov_base, 0, vec[i].iov_len); + + TEST(readv(fd, vec, tc->iov_count)); + + if (TST_RET == -1) + tst_res(TFAIL | TTERRNO, "readv() failed unexpectedly"); + else if (TST_RET < 0) + tst_res(TFAIL | TTERRNO, "readv() returned invalid value"); + else if (TST_RET != tc->exp_ret) + tst_res(TFAIL, "readv() returned unexpected value %ld", + TST_RET); - memset(rd_iovec[0].iov_base, 0x00, CHUNK); + if (TST_RET != tc->exp_ret) + return; - if (readv(fd, rd_iovec, 3) != CHUNK) { - tst_res(TFAIL, "readv failed reading %d bytes, " - "followed by two NULL vectors", CHUNK); - } else { - fail = 0; - vec = rd_iovec[0].iov_base; + tst_res(TPASS, "%s", tc->name); - for (i = 0; i < CHUNK; i++) { - if (vec[i] != 0x42) + for (i = 0, fpos = 0; i < tc->iov_count; i++) { + ptr = vec[i].iov_base; + + for (j = 0; j < vec[i].iov_len; j++, fpos++) { + if (ptr[j] != (fpos < tc->exp_ret ? 0x42 : 0)) fail++; } - - if (fail) - tst_res(TFAIL, "Wrong buffer content"); - else - tst_res(TPASS, "readv passed reading %d bytes " - "followed by two NULL vectors", CHUNK); } + + if (fail) + tst_res(TFAIL, "Wrong buffer content"); + else + tst_res(TPASS, "readv() correctly read %d bytes ", tc->exp_ret); } static void setup(void) { + /* replace the default NULL pointer with end of guarded buffer */ + lockup_iovec[0].iov_base = rd_iovec[0].iov_base + rd_iovec[0].iov_len; + memset(buf, 0x42, sizeof(buf)); - fd = SAFE_OPEN("data_file", O_WRONLY | O_CREAT, 0666); + fd = SAFE_OPEN("data_file", O_WRONLY | O_CREAT | O_TRUNC, 0666); SAFE_WRITE(1, fd, buf, sizeof(buf)); SAFE_CLOSE(fd); fd = SAFE_OPEN("data_file", O_RDONLY); @@ -85,6 +106,15 @@ static void cleanup(void) static struct tst_test test = { .setup = setup, .cleanup = cleanup, - .test_all = run, + .test = test_readv, + .tcnt = ARRAY_SIZE(testcase_list), .needs_tmpdir = 1, + .timeout = 15, + .bufs = (struct tst_buffers[]) { + {&rd_iovec, .iov_sizes = (int[]){CHUNK, 0, 0, -1}}, + {&big_iovec, .iov_sizes = (int[]){2*CHUNK, CHUNK, -1}}, + {&multi_iovec, .iov_sizes = (int[]){CHUNK/4, CHUNK/2, -1}}, + {&lockup_iovec, .iov_sizes = (int[]){0, CHUNK, -1}}, + {} + } }; -- 2.25.1