From 2338d3176bdb6fb607a8ce81534e88522d8c516c Mon Sep 17 00:00:00 2001 From: Matt Kraai Date: Mon, 6 Aug 2001 16:09:09 +0000 Subject: [PATCH] Use ferror(3) to check for errors, rather than inspecting errno. Thanks to David Douthitt for reporting, and shame on me for writing such crappy code. --- coreutils/head.c | 4 +--- head.c | 4 +--- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/coreutils/head.c b/coreutils/head.c index 0c8ef3d59..688c250b1 100644 --- a/coreutils/head.c +++ b/coreutils/head.c @@ -22,7 +22,6 @@ * */ -#include #include #include #include @@ -82,10 +81,9 @@ int head_main(int argc, char **argv) printf("==> %s <==\n", argv[optind]); } head(len, fp); - if (errno) { + if (ferror(fp)) { perror_msg("%s", argv[optind]); status = EXIT_FAILURE; - errno = 0; } if (optind < argc - 1) putchar('\n'); diff --git a/head.c b/head.c index 0c8ef3d59..688c250b1 100644 --- a/head.c +++ b/head.c @@ -22,7 +22,6 @@ * */ -#include #include #include #include @@ -82,10 +81,9 @@ int head_main(int argc, char **argv) printf("==> %s <==\n", argv[optind]); } head(len, fp); - if (errno) { + if (ferror(fp)) { perror_msg("%s", argv[optind]); status = EXIT_FAILURE; - errno = 0; } if (optind < argc - 1) putchar('\n'); -- 2.25.1