Aren
/
NetBSD
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

games/caesar: fix error handling for I/O 

The '!= 0' is needed for an upcoming fix to lint's strict bool mode.

The additional text in the error messages is necessary because errx only
outputs the given string, with no details from errno.  Since ferror does
not set errno, no such details are available, so just output a generic
message, which is still better than just "caesar: <stdout>".
This commit is contained in:
rillig 2021-11-16 20:42:47 +00:00
parent 12726c0065
commit dfa14757fe
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
games/caesar/caesar.c
View File

@ -1,4 +1,4 @@
/* $NetBSD: caesar.c,v 1.23 2021/05/01 20:21:25 rillig Exp $ */
/* $NetBSD: caesar.c,v 1.24 2021/11/16 20:42:47 rillig Exp $ */
/*
* Copyright (c) 1989, 1993
@ -48,7 +48,7 @@ __COPYRIGHT("@(#) Copyright (c) 1989, 1993\
#if 0
static char sccsid[] = "@(#)caesar.c 8.1 (Berkeley) 5/31/93";
#else
__RCSID("$NetBSD: caesar.c,v 1.23 2021/05/01 20:21:25 rillig Exp $");
__RCSID("$NetBSD: caesar.c,v 1.24 2021/11/16 20:42:47 rillig Exp $");
#endif
#endif /* not lint */
@ -203,14 +203,14 @@ main(int argc, char **argv)
/* NOTREACHED */
}
if (ferror(stdin)) {
errx(EXIT_FAILURE, "<stdin>");
if (ferror(stdin) != 0) {
errx(EXIT_FAILURE, "<stdin>: read error");
/* NOTREACHED */
}
(void)fflush(stdout);
if (ferror(stdout)) {
errx(EXIT_FAILURE, "<stdout>");
if (ferror(stdout) != 0) {
errx(EXIT_FAILURE, "<stdout>: write error");
/* NOTREACHED */
}