diff options
author | Thomas Weißschuh <linux@weissschuh.net> | 2023-08-03 10:28:55 +0300 |
---|---|---|
committer | Willy Tarreau <w@1wt.eu> | 2023-08-23 06:17:07 +0300 |
commit | 37266a9ec7f5a3144c2070baeea2628c30ef07e5 (patch) | |
tree | df3b056af84056c79322736c4805a6792157f322 /tools/testing/selftests/nolibc | |
parent | 711f91fdec714d6307f1fa044fb74d0a742b1494 (diff) | |
download | linux-37266a9ec7f5a3144c2070baeea2628c30ef07e5.tar.xz |
selftests/nolibc: use correct return type for read() and write()
Avoid truncating values before comparing them.
As printf in nolibc doesn't support ssize_t add casts to int for
printing.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
Signed-off-by: Willy Tarreau <w@1wt.eu>
Diffstat (limited to 'tools/testing/selftests/nolibc')
-rw-r--r-- | tools/testing/selftests/nolibc/nolibc-test.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/tools/testing/selftests/nolibc/nolibc-test.c b/tools/testing/selftests/nolibc/nolibc-test.c index c2646707ccc2..23a5e4c57083 100644 --- a/tools/testing/selftests/nolibc/nolibc-test.c +++ b/tools/testing/selftests/nolibc/nolibc-test.c @@ -1019,7 +1019,8 @@ int run_stdlib(int min, int max) static int expect_vfprintf(int llen, int c, const char *expected, const char *fmt, ...) { - int ret, fd, w, r; + int ret, fd; + ssize_t w, r; char buf[100]; FILE *memfile; va_list args; @@ -1041,7 +1042,7 @@ static int expect_vfprintf(int llen, int c, const char *expected, const char *fm va_end(args); if (w != c) { - llen += printf(" written(%d) != %d", w, c); + llen += printf(" written(%d) != %d", (int)w, c); result(llen, FAIL); return 1; } @@ -1055,7 +1056,7 @@ static int expect_vfprintf(int llen, int c, const char *expected, const char *fm fclose(memfile); if (r != w) { - llen += printf(" written(%d) != read(%d)", w, r); + llen += printf(" written(%d) != read(%d)", (int)w, (int)r); result(llen, FAIL); return 1; } |