strtol(3) can only report ERANGE, if the base is valid (and it is). Link: <https://savannah.gnu.org/bugs/?65451> Cc: "G. Branden Robinson" <bran...@debian.org> Cc: Dave Kemper <saint.s...@gmail.com> Cc: "James K. Lowden" <jklow...@schemamania.org> Signed-off-by: Alejandro Colomar <a...@kernel.org> --- src/libs/libgroff/curtime.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/libs/libgroff/curtime.cpp b/src/libs/libgroff/curtime.cpp index 490fc3018..4623ad9a0 100644 --- a/src/libs/libgroff/curtime.cpp +++ b/src/libs/libgroff/curtime.cpp @@ -42,7 +42,7 @@ struct tm *current_time() char *endptr; long epoch = strtol(source_date_epoch, &endptr, 10); - if ((errno == ERANGE) || (errno != 0 && epoch == 0)) + if (errno == ERANGE) fatal("$SOURCE_DATE_EPOCH: strtol: %1", strerror(errno)); if (endptr == source_date_epoch) fatal("$SOURCE_DATE_EPOCH: no digits found: '%1'", endptr); -- 2.43.0
signature.asc
Description: PGP signature