diff options
author | Anastasia Klimchuk <aklm@chromium.org> | 2022-09-02 17:12:06 +1000 |
---|---|---|
committer | Thomas Heijligen <src@posteo.de> | 2022-09-12 12:43:59 +0000 |
commit | 73c0a1d8785f846bb2586a96a3f0a4808456791d (patch) | |
tree | 104c63cbfff1e54e37ace4c1b05f24c8247a20d3 | |
parent | 75d327c5509883624e4ef1366c8d1d430666035e (diff) | |
download | flashrom-73c0a1d8785f846bb2586a96a3f0a4808456791d.tar.gz flashrom-73c0a1d8785f846bb2586a96a3f0a4808456791d.tar.bz2 flashrom-73c0a1d8785f846bb2586a96a3f0a4808456791d.zip |
tests: Use MOCK_FD instead of NON_ZERO for file operations
NON_ZERO can be a negative number, so MOCK_FD is safer option to
use as a mock file descriptor. Also it is more readable.
BUG=b:237606255
TEST=ninja test (on linux)
Change-Id: I097dd59f69c3fb532ac136796fcf5cae8839af7b
TICKET: https://ticket.coreboot.org/issues/411
Signed-off-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/67310
Reviewed-by: Felix Singer <felixsinger@posteo.net>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Alexander Goncharov <chat@joursoir.net>
Reviewed-by: Thomas Heijligen <src@posteo.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r-- | tests/tests.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/tests.c b/tests/tests.c index 7a448d4f..8e3fdde0 100644 --- a/tests/tests.c +++ b/tests/tests.c @@ -24,7 +24,7 @@ void *not_null(void) { - return (void *)NON_ZERO; + return (void *)MOCK_FD; } /* Workaround for https://github.com/clibs/cmocka/issues/17 */ @@ -253,7 +253,7 @@ int __wrap_fflush(FILE *fp) int __wrap_fileno(FILE *fp) { LOG_ME; - return NON_ZERO; + return MOCK_FD; } int __wrap_fsync(int fd) |