From d5de9d1dc5a7d20e2bbd2578b8be2ce0a9369619 Mon Sep 17 00:00:00 2001 From: Alessandro Bono Date: Wed, 20 Jan 2021 16:36:32 +0100 Subject: [PATCH] fread.3: Swap size and nmemb params It works both way, but this one feels more right. We are reading four elements sizeof(*buffer) bytes each. Signed-off-by: Michael Kerrisk --- man3/fread.3 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/man3/fread.3 b/man3/fread.3 index e1284e05e..e3f34b7d1 100644 --- a/man3/fread.3 +++ b/man3/fread.3 @@ -153,8 +153,8 @@ main(void) unsigned char buffer[4]; - size_t ret = fread(buffer, ARRAY_SIZE(buffer), sizeof(*buffer), fp); - if (ret != sizeof(*buffer)) { + size_t ret = fread(buffer, sizeof(*buffer), ARRAY_SIZE(buffer), fp); + if (ret != ARRAY_SIZE(buffer)) { fprintf(stderr, "fread() failed: %zu\en", ret); exit(EXIT_FAILURE); }