git: ee10c61d7a0a - main - stress2: Fix a off-by-one error. Added check for argument file type
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Mon, 06 Mar 2023 09:04:29 UTC
The branch main has been updated by pho:
URL: https://cgit.FreeBSD.org/src/commit/?id=ee10c61d7a0ae11a9d76c54a393587bd3933698e
commit ee10c61d7a0ae11a9d76c54a393587bd3933698e
Author: Peter Holm <pho@FreeBSD.org>
AuthorDate: 2023-03-06 09:03:43 +0000
Commit: Peter Holm <pho@FreeBSD.org>
CommitDate: 2023-03-06 09:03:43 +0000
stress2: Fix a off-by-one error. Added check for argument file type
---
tools/test/stress2/tools/flip.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/tools/test/stress2/tools/flip.c b/tools/test/stress2/tools/flip.c
index 8d01e9d78ea1..d73212c748bf 100644
--- a/tools/test/stress2/tools/flip.c
+++ b/tools/test/stress2/tools/flip.c
@@ -57,7 +57,7 @@ flip(void *ap, size_t len)
unsigned char bit, buf, mask, old __unused;
cp = (unsigned char *)ap;
- byte = random_long(0, len);
+ byte = random_long(0, len - 1);
bit = random_long(0,7);
mask = ~(1 << bit);
buf = cp[byte];
@@ -111,6 +111,8 @@ main(int argc, char *argv[])
if (size == 0) {
if (fstat(fd, &st) == -1)
err(1, "stat %s", argv[0]);
+ if ((st.st_mode & S_IFREG) == 0)
+ errx(1, "%s must be a regular file\n", argv[0]);
size = st.st_size;
}