diff options
author | Alejandro Colomar <alx@kernel.org> | 2024-01-14 20:54:32 +0100 |
---|---|---|
committer | Alejandro Colomar <alx@kernel.org> | 2024-03-16 13:30:07 +0100 |
commit | 0aea166ce231df7cdec212270671eaa3502414bd (patch) | |
tree | 9f9fa9e878636ea33711aa5c94b1d989688934b2 | |
parent | c2e017009874355facd5bbe9ad847225bfd88a3d (diff) |
[libgroff]: Remove redundant checks.
ERANGE can only happen if strtol(3) returns either LONG_MIN or LONG_MAX.
Fixes: e4290210f2c1 ("Implement `SOURCE_DATE_EPOCH' for reproducible builds.")
Link: <https://savannah.gnu.org/bugs/?65451>
Cc: "G. Branden Robinson" <branden@debian.org>
Cc: Dave Kemper <saint.snit@gmail.com>
Cc: "James K. Lowden" <jklowden@schemamania.org>
Cc: Colin Watson <cjwatson@debian.org>
Cc: Werner LEMBERG <wl@gnu.org>
Cc: James Clark <jjc@jclark.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>
-rw-r--r-- | src/libs/libgroff/curtime.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/libs/libgroff/curtime.cpp b/src/libs/libgroff/curtime.cpp index 277755cab..490fc3018 100644 --- a/src/libs/libgroff/curtime.cpp +++ b/src/libs/libgroff/curtime.cpp @@ -42,8 +42,7 @@ struct tm *current_time() char *endptr; long epoch = strtol(source_date_epoch, &endptr, 10); - if ((errno == ERANGE && (epoch == LONG_MAX || epoch == LONG_MIN)) || - (errno != 0 && epoch == 0)) + if ((errno == ERANGE) || (errno != 0 && epoch == 0)) fatal("$SOURCE_DATE_EPOCH: strtol: %1", strerror(errno)); if (endptr == source_date_epoch) fatal("$SOURCE_DATE_EPOCH: no digits found: '%1'", endptr); |