[openssl] master update
Dr. Paul Dale
pauli at openssl.org
Wed Feb 24 22:39:46 UTC 2021
The branch master has been updated
via 75de54363506e2b2480fc6baf0cd45b1f7fc8816 (commit)
from 5eb73cfb372a3701a25f9d4f5e109ba21669af61 (commit)
- Log -----------------------------------------------------------------
commit 75de54363506e2b2480fc6baf0cd45b1f7fc8816
Author: jwalch <jeremy.walch at gmail.com>
Date: Fri Feb 19 13:02:27 2021 -0500
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 at openssl.org>
Reviewed-by: Paul Dale <pauli at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/14252)
-----------------------------------------------------------------------
Summary of changes:
crypto/o_time.c | 4 ++--
1 file 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;
More information about the openssl-commits
mailing list