summaryrefslogtreecommitdiff
path: root/sfeed_gopher.c
diff options
context:
space:
mode:
authorHiltjo Posthuma <hiltjo@codemadness.org>2022-03-14 13:25:52 +0100
committerHiltjo Posthuma <hiltjo@codemadness.org>2022-03-14 13:25:52 +0100
commitb6f7a3fe15f2f253a1653454f514b467aa20f821 (patch)
treec0fdddaa251dbb0a4da66a86b547032bb84d3bc8 /sfeed_gopher.c
parent3dbdf1ccfa5f109f8d7489f913ed57aa8e9b5f11 (diff)
improve time(NULL) error checking
Use errx, time(NULL) does not set errno. For sfeed_curses reset errno so it doesn't print a random error if it failed. POSIX recommends checking against (time_t)-1 on failure. Note that some implementation, like the OpenBSD man page says time() cannot fail.
Diffstat (limited to 'sfeed_gopher.c')
-rw-r--r--sfeed_gopher.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/sfeed_gopher.c b/sfeed_gopher.c
index 9e8e10a..0b95b3d 100644
--- a/sfeed_gopher.c
+++ b/sfeed_gopher.c
@@ -137,8 +137,8 @@ main(int argc, char *argv[])
err(1, "pledge");
}
- if ((comparetime = time(NULL)) == -1)
- err(1, "time");
+ if ((comparetime = time(NULL)) == (time_t)-1)
+ errx(1, "time");
/* 1 day is old news */
comparetime -= 86400;