From 99330d67b009583d3638c77e348e4e098764959c Mon Sep 17 00:00:00 2001 From: Anastasia Klimchuk Date: Fri, 15 Oct 2021 16:14:11 +1100 Subject: 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 Reviewed-on: https://review.coreboot.org/c/flashrom/+/58356 Tested-by: build bot (Jenkins) Reviewed-by: Nico Huber --- tests/tests.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/tests.c b/tests/tests.c index 3eb3129a..d3df356a 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) -- cgit v1.2.3