diff options
| author | Anastasia Klimchuk <aklm@chromium.org> | 2021-10-15 16:14:11 +1100 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2021-10-19 08:10:03 +0000 |
| commit | 99330d67b009583d3638c77e348e4e098764959c (patch) | |
| tree | 79cf167603ced5f66da54cc0f995b3fbd285ecf2 | |
| parent | 92989f2496e3b138e784d10ef4b2536a757c0953 (diff) | |
| download | flashrom-99330d67b009583d3638c77e348e4e098764959c.tar.gz flashrom-99330d67b009583d3638c77e348e4e098764959c.tar.bz2 flashrom-99330d67b009583d3638c77e348e4e098764959c.zip | |
tests: Fix fread wrap to return success by default
Successful return value for fread is the number of items read, and
default behaviour for all wraps is to return success.
This worked previously because all existing tests have custom mocks
for fread, so default behaviour hasn't been used. However next patch
in this chain adds new test which needs default fread wrap.
BUG=b:181803212
TEST=ninja test
Change-Id: I17d82d281a87129843f547b87c18f52aca23314d
Signed-off-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/58356
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
| -rw-r--r-- | tests/tests.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/tests.c b/tests/tests.c index 3eb3129ac..d3df356a9 100644 --- a/tests/tests.c +++ b/tests/tests.c @@ -168,12 +168,12 @@ char *__wrap_fgets(char *buf, int len, FILE *fp) return NULL; } -size_t __wrap_fread(void *ptr, size_t size, size_t len, FILE *fp) +size_t __wrap_fread(void *ptr, size_t size, size_t nmemb, FILE *fp) { LOG_ME; if (get_io() && get_io()->fread) - return get_io()->fread(get_io()->state, ptr, size, len, fp); - return 0; + return get_io()->fread(get_io()->state, ptr, size, nmemb, fp); + return nmemb; } size_t __wrap_fwrite(const void *ptr, size_t size, size_t nmemb, FILE *fp) |