[openssl/openssl] d8dbad: ocsp: Reject update times X509_cmp_time() cannot c...
"'Mounir IDRASSI' via openssl-commits" <[email protected]>
| Newsgroups | gmane.comp.encryption.openssl.cvs |
|---|---|
| Message-ID | <openssl/openssl/push/refs/heads/openssl-3.5/[email protected]> |
Branch: refs/heads/openssl-3.5
Home: https://github.com/openssl/openssl
Commit: d8dbadb7bd70709e8845fc2187d790f21d93f222
https://github.com/openssl/openssl/commit/d8dbadb7bd70709e8845fc2187d790f21d93f222
Author: Mounir IDRASSI <[email protected]>
Date: 2026-08-27 (Thu, 27 Aug 2026)
Changed paths:
M crypto/ocsp/ocsp_cl.c
M test/ocspapitest.c
Log Message:
-----------
ocsp: Reject update times X509_cmp_time() cannot compare
OCSP_check_validity() accepts GeneralizedTime forms that
X509_cmp_time() cannot compare. The latter requires the canonical RFC
5280 form and returns 0 for numeric UTC offsets and fractional seconds.
That error was treated as an in-range result, allowing OCSP freshness
checks to be bypassed.
Treat errors from the initial thisUpdate and nextUpdate comparisons as
field errors. Run the optional maximum-age check only after the initial
thisUpdate comparison succeeds, avoiding a duplicate field error.
Add regression coverage for offset and fractional forms, including the
maximum-age path.
Fixes #32377
Assisted-by: Codex:gpt-5.6-sol
Signed-off-by: Mounir IDRASSI <[email protected]>
Reviewed-by: Bob Beck <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
Merge-date: Thu Aug 27 15:15:28 2026
Merged-from: https://github.com/openssl/openssl/pull/32396
(cherry picked from commit 6ceaf1dfe6729853961c6f53e97e635e82da9168)
To unsubscribe from these emails, change your notification settings at https://github.com/openssl/openssl/settings/notifications
--
You received this message because you are subscribed to the Google Groups "openssl-commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [email protected].
To view this discussion visit https://groups.google.com/a/openssl.org/d/msgid/openssl-commits/openssl/openssl/push/refs/heads/openssl-3.5/7a84de-d8dbad%40github.com.