summaryrefslogtreecommitdiff
path: root/tests/fs-tests/stress/atoms
diff options
context:
space:
mode:
authorDavid Oberhollenzer <david.oberhollenzer@sigma-star.at>2020-01-24 23:01:57 +0100
committerDavid Oberhollenzer <david.oberhollenzer@sigma-star.at>2020-02-09 22:13:18 +0100
commitb636250e211198210ab996671bccc2983300c6f5 (patch)
treeda1793fcd90dfd2b411afc1bc7cc2249b47c8e4f /tests/fs-tests/stress/atoms
parent11bc41c32cc933a141545be6a9729122ac436cfe (diff)
mtd-utils: Fix potential negative arguments passed to close(2)
Many tools open a file descriptor, close it a the end and have some form of error path in between that jumps to the end. In some cases, if opening the file fails the error path is taken and the utility ends up closing one or more invalid file descriptors. It's technically not a real issue but something that pretty much any static analysis tool barks at. Signed-off-by: David Oberhollenzer <david.oberhollenzer@sigma-star.at>
Diffstat (limited to 'tests/fs-tests/stress/atoms')
-rw-r--r--tests/fs-tests/stress/atoms/fwrite00.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/tests/fs-tests/stress/atoms/fwrite00.c b/tests/fs-tests/stress/atoms/fwrite00.c
index 3406bba..877c63c 100644
--- a/tests/fs-tests/stress/atoms/fwrite00.c
+++ b/tests/fs-tests/stress/atoms/fwrite00.c
@@ -138,7 +138,9 @@ static void filestress00(void)
deleted = 1;
}
}
- CHECK(close(fd) != -1);
+ if (fd > 0) {
+ CHECK(close(fd) != -1);
+ }
/* Sleep */
if (tests_sleep_parameter > 0) {
unsigned us = tests_sleep_parameter * 1000;