diff options
author | jwalch <jeremy.walch@gmail.com> | 2021-02-19 13:02:27 -0500 |
---|---|---|
committer | Pauli <ppzgs1@gmail.com> | 2021-02-25 08:39:07 +1000 |
commit | 75de54363506e2b2480fc6baf0cd45b1f7fc8816 (patch) | |
tree | ef4cd8c7d9f2e589ea8d9c80bf72f79c70e33b62 /crypto/o_time.c | |
parent | 5eb73cfb372a3701a25f9d4f5e109ba21669af61 (diff) | |
download | openssl-new-75de54363506e2b2480fc6baf0cd45b1f7fc8816.tar.gz |
Fix an integer overflow in o_time.c
If input offset_sec is sufficiently large (> INT32_MAX * SECS_PER_DAY, which is possible for a long on 64-bit platforms), then the first assignment contains an overflow.
I think leaving offset_hms as an int is still safe.
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/14252)
Diffstat (limited to 'crypto/o_time.c')
-rw-r--r-- | crypto/o_time.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/crypto/o_time.c b/crypto/o_time.c index 632e19e367..f367945a18 100644 --- a/crypto/o_time.c +++ b/crypto/o_time.c @@ -133,8 +133,8 @@ int OPENSSL_gmtime_diff(int *pday, int *psec, static int julian_adj(const struct tm *tm, int off_day, long offset_sec, long *pday, int *psec) { - int offset_hms, offset_day; - long time_jd; + int offset_hms; + long offset_day, time_jd; int time_year, time_month, time_day; /* split offset into days and day seconds */ offset_day = offset_sec / SECS_PER_DAY; |