diff options
| author | Alex Gaynor <alex.gaynor@gmail.com> | 2017-04-07 20:49:27 -0400 |
|---|---|---|
| committer | Rich Salz <rsalz@openssl.org> | 2017-05-22 08:03:22 -0400 |
| commit | 3a490bbb1c9355bc0fefc50742ecc056d84332c1 (patch) | |
| tree | e88b730d28ccd085d11b149d41b9f233851eeac9 /crypto/ct | |
| parent | e2580e70d5c644c5dadd33ffa8bb23223959b87a (diff) | |
Fixed #3020 -- set entry type on SCTs from X.509 and OCSP extensions
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3149)
Diffstat (limited to 'crypto/ct')
| -rw-r--r-- | crypto/ct/ct_x509v3.c | 41 |
1 files changed, 39 insertions, 2 deletions
diff --git a/crypto/ct/ct_x509v3.c b/crypto/ct/ct_x509v3.c index 805ada088a..c16005dce1 100644 --- a/crypto/ct/ct_x509v3.c +++ b/crypto/ct/ct_x509v3.c @@ -30,12 +30,49 @@ static int i2r_SCT_LIST(X509V3_EXT_METHOD *method, STACK_OF(SCT) *sct_list, return 1; } +static int set_sct_list_source(STACK_OF(SCT) *s, sct_source_t source) { + if (s != NULL) { + for (size_t i = 0; i < sk_SCT_num(s); i++) { + int res = SCT_set_source( + sk_SCT_value(s, i), SCT_SOURCE_X509V3_EXTENSION); + if (res != 1) { + return 0; + } + } + } + return 1; +} + +static STACK_OF(SCT) *x509_ext_d2i_SCT_LIST(STACK_OF(SCT) **a, + const unsigned char **pp, + long len) { + STACK_OF(SCT) *s = d2i_SCT_LIST(a, pp, len); + if (set_sct_list_source(s, SCT_SOURCE_X509V3_EXTENSION) != 1) { + SCT_LIST_free(s); + *a = NULL; + return NULL; + } + return s; +} + +static STACK_OF(SCT) *ocsp_ext_d2i_SCT_LIST(STACK_OF(SCT) **a, + const unsigned char **pp, + long len) { + STACK_OF(SCT) *s = d2i_SCT_LIST(a, pp, len); + if (set_sct_list_source(s, SCT_SOURCE_OCSP_STAPLED_RESPONSE) != 1) { + SCT_LIST_free(s); + *a = NULL; + return NULL; + } + return s; +} + /* Handlers for X509v3/OCSP Certificate Transparency extensions */ const X509V3_EXT_METHOD v3_ct_scts[3] = { /* X509v3 extension in certificates that contains SCTs */ { NID_ct_precert_scts, 0, NULL, NULL, (X509V3_EXT_FREE)SCT_LIST_free, - (X509V3_EXT_D2I)d2i_SCT_LIST, (X509V3_EXT_I2D)i2d_SCT_LIST, + (X509V3_EXT_D2I)x509_ext_d2i_SCT_LIST, (X509V3_EXT_I2D)i2d_SCT_LIST, NULL, NULL, NULL, NULL, (X509V3_EXT_I2R)i2r_SCT_LIST, NULL, @@ -52,7 +89,7 @@ const X509V3_EXT_METHOD v3_ct_scts[3] = { /* OCSP extension that contains SCTs */ { NID_ct_cert_scts, 0, NULL, 0, (X509V3_EXT_FREE)SCT_LIST_free, - (X509V3_EXT_D2I)d2i_SCT_LIST, (X509V3_EXT_I2D)i2d_SCT_LIST, + (X509V3_EXT_D2I)ocsp_ext_d2i_SCT_LIST, (X509V3_EXT_I2D)i2d_SCT_LIST, NULL, NULL, NULL, NULL, (X509V3_EXT_I2R)i2r_SCT_LIST, NULL, |