From e730cda5505a4c097c036590cbe067f493eea82e Mon Sep 17 00:00:00 2001 From: John Safranek Date: Tue, 4 Sep 2018 13:39:26 -0700 Subject: [PATCH] Validate Date The ValidateDate() function is wrapped by a macro so it may be replaced in environments without ASN_TIME built in. In two cases, OCSP and CRL, ValidateDate() is called directly instead of by the macro. This change fixes that. --- src/crl.c | 2 +- src/ocsp.c | 6 ++++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/crl.c b/src/crl.c index 93a77eec5..7b59faacc 100644 --- a/src/crl.c +++ b/src/crl.c @@ -303,7 +303,7 @@ static int CheckCertCRLList(WOLFSSL_CRL* crl, DecodedCert* cert, int *pFoundEntr if (doNextDate) { #ifndef NO_ASN_TIME - if (!ValidateDate(crle->nextDate,crle->nextDateFormat, AFTER)) { + if (!XVALIDATE_DATE(crle->nextDate,crle->nextDateFormat, AFTER)) { WOLFSSL_MSG("CRL next date is no longer valid"); ret = ASN_AFTER_DATE_E; } diff --git a/src/ocsp.c b/src/ocsp.c index 7863f5672..a0afe50a8 100644 --- a/src/ocsp.c +++ b/src/ocsp.c @@ -224,9 +224,11 @@ static int GetOcspStatus(WOLFSSL_OCSP* ocsp, OcspRequest* request, } else if (*status) { #ifndef NO_ASN_TIME - if (ValidateDate((*status)->thisDate, (*status)->thisDateFormat, BEFORE) + if (XVALIDATE_DATE((*status)->thisDate, + (*status)->thisDateFormat, BEFORE) && ((*status)->nextDate[0] != 0) - && ValidateDate((*status)->nextDate, (*status)->nextDateFormat, AFTER)) + && XVALIDATE_DATE((*status)->nextDate, + (*status)->nextDateFormat, AFTER)) #endif { ret = xstat2err((*status)->status);