[U-Boot] [PATCH] envcrc: check return value of fwrite()

Newer toolchains will often complain about unchecked fwrite(): envcrc.c:117: warning: ignoring return value of ‘fwrite’, declared with attribute warn_unused_result
So check the return value to silence the warnings.
Signed-off-by: Mike Frysinger vapier@gentoo.org --- tools/envcrc.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/tools/envcrc.c b/tools/envcrc.c index d02a7db..d1e84b3 100644 --- a/tools/envcrc.c +++ b/tools/envcrc.c @@ -21,6 +21,7 @@ * MA 02111-1307 USA */
+#include <errno.h> #include <stdio.h> #include <stdint.h> #include <stdlib.h> @@ -114,7 +115,8 @@ int main (int argc, char **argv) } for (i = start; i != end; i += step) printf("%c", (crc & (0xFF << (i * 8))) >> (i * 8)); - fwrite(dataptr, 1, datasize, stdout); + if (fwrite(dataptr, 1, datasize, stdout) != datasize) + fprintf(stderr, "fwrite() failed: %s\n", strerror(errno)); } else { printf("CRC32 from offset %08X to %08X of environment = %08X\n", (unsigned int) (dataptr - envptr),

Dear Mike Frysinger,
In message 1255912994-4500-1-git-send-email-vapier@gentoo.org you wrote:
Newer toolchains will often complain about unchecked fwrite(): envcrc.c:117: warning: ignoring return value of `fwrite´, declared with attribute warn_unused_result
So check the return value to silence the warnings.
Signed-off-by: Mike Frysinger vapier@gentoo.org
tools/envcrc.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-)
Applied, thanks.
Best regards,
Wolfgang Denk
participants (2)
-
Mike Frysinger
-
Wolfgang Denk