| Nick Child | 5d881c3 | 2022-02-28 10:09:16 -0600 | [diff] [blame] | 1 | /* | 
|  | 2 | *  Copyright The Mbed TLS Contributors | 
| Dave Rodgman | 16799db | 2023-11-02 19:47:20 +0000 | [diff] [blame] | 3 | *  SPDX-License-Identifier: Apache-2.0 OR GPL-2.0-or-later | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 4 | */ | 
|  | 5 | #include "common.h" | 
|  | 6 |  | 
|  | 7 | #include "mbedtls/build_info.h" | 
|  | 8 | #if defined(MBEDTLS_PKCS7_C) | 
|  | 9 | #include "mbedtls/pkcs7.h" | 
| Valerio Setti | 25b282e | 2024-01-17 10:55:32 +0100 | [diff] [blame] | 10 | #include "x509_internal.h" | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 11 | #include "mbedtls/asn1.h" | 
|  | 12 | #include "mbedtls/x509_crt.h" | 
|  | 13 | #include "mbedtls/x509_crl.h" | 
|  | 14 | #include "mbedtls/oid.h" | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 15 | #include "mbedtls/error.h" | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 16 |  | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 17 | #if defined(MBEDTLS_FS_IO) | 
|  | 18 | #include <sys/types.h> | 
|  | 19 | #include <sys/stat.h> | 
|  | 20 | #endif | 
|  | 21 |  | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 22 | #include "mbedtls/platform.h" | 
|  | 23 | #include "mbedtls/platform_util.h" | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 24 |  | 
|  | 25 | #if defined(MBEDTLS_HAVE_TIME) | 
|  | 26 | #include "mbedtls/platform_time.h" | 
|  | 27 | #endif | 
|  | 28 | #if defined(MBEDTLS_HAVE_TIME_DATE) | 
|  | 29 | #include <time.h> | 
|  | 30 | #endif | 
|  | 31 |  | 
|  | 32 | /** | 
| Dave Rodgman | efbc5f7 | 2023-03-13 12:15:49 +0000 | [diff] [blame] | 33 | * Initializes the mbedtls_pkcs7 structure. | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 34 | */ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 35 | void mbedtls_pkcs7_init(mbedtls_pkcs7 *pkcs7) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 36 | { | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 37 | memset(pkcs7, 0, sizeof(*pkcs7)); | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 38 | } | 
|  | 39 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 40 | static int pkcs7_get_next_content_len(unsigned char **p, unsigned char *end, | 
|  | 41 | size_t *len) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 42 | { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 43 | int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 44 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 45 | ret = mbedtls_asn1_get_tag(p, end, len, MBEDTLS_ASN1_CONSTRUCTED | 
|  | 46 | | MBEDTLS_ASN1_CONTEXT_SPECIFIC); | 
|  | 47 | if (ret != 0) { | 
| Demi Marie Obenour | 6cfc469 | 2022-11-28 00:46:00 -0500 | [diff] [blame] | 48 | ret = MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_CONTENT_INFO, ret); | 
| Demi Marie Obenour | 512818b | 2022-11-27 22:48:55 -0500 | [diff] [blame] | 49 | } else if ((size_t) (end - *p) != *len) { | 
| Demi Marie Obenour | 6cfc469 | 2022-11-28 00:46:00 -0500 | [diff] [blame] | 50 | ret = MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_CONTENT_INFO, | 
| Demi Marie Obenour | 512818b | 2022-11-27 22:48:55 -0500 | [diff] [blame] | 51 | MBEDTLS_ERR_ASN1_LENGTH_MISMATCH); | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 52 | } | 
|  | 53 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 54 | return ret; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 55 | } | 
|  | 56 |  | 
|  | 57 | /** | 
|  | 58 | * version Version | 
|  | 59 | * Version ::= INTEGER | 
|  | 60 | **/ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 61 | static int pkcs7_get_version(unsigned char **p, unsigned char *end, int *ver) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 62 | { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 63 | int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 64 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 65 | ret = mbedtls_asn1_get_int(p, end, ver); | 
|  | 66 | if (ret != 0) { | 
|  | 67 | ret = MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_VERSION, ret); | 
|  | 68 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 69 |  | 
|  | 70 | /* If version != 1, return invalid version */ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 71 | if (*ver != MBEDTLS_PKCS7_SUPPORTED_VERSION) { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 72 | ret = MBEDTLS_ERR_PKCS7_INVALID_VERSION; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 73 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 74 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 75 | return ret; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 76 | } | 
|  | 77 |  | 
|  | 78 | /** | 
|  | 79 | * ContentInfo ::= SEQUENCE { | 
|  | 80 | *      contentType ContentType, | 
|  | 81 | *      content | 
|  | 82 | *              [0] EXPLICIT ANY DEFINED BY contentType OPTIONAL } | 
|  | 83 | **/ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 84 | static int pkcs7_get_content_info_type(unsigned char **p, unsigned char *end, | 
| Nick Child | ec81709 | 2022-12-15 15:54:03 -0600 | [diff] [blame] | 85 | unsigned char **seq_end, | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 86 | mbedtls_pkcs7_buf *pkcs7) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 87 | { | 
|  | 88 | size_t len = 0; | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 89 | int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED; | 
| Nayna Jain | 673a226 | 2020-12-14 22:44:49 +0000 | [diff] [blame] | 90 | unsigned char *start = *p; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 91 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 92 | ret = mbedtls_asn1_get_tag(p, end, &len, MBEDTLS_ASN1_CONSTRUCTED | 
|  | 93 | | MBEDTLS_ASN1_SEQUENCE); | 
|  | 94 | if (ret != 0) { | 
| Nick Child | c448c94 | 2021-07-01 15:29:50 -0400 | [diff] [blame] | 95 | *p = start; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 96 | return MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_CONTENT_INFO, ret); | 
| Nick Child | c448c94 | 2021-07-01 15:29:50 -0400 | [diff] [blame] | 97 | } | 
| Nick Child | ec81709 | 2022-12-15 15:54:03 -0600 | [diff] [blame] | 98 | *seq_end = *p + len; | 
|  | 99 | ret = mbedtls_asn1_get_tag(p, *seq_end, &len, MBEDTLS_ASN1_OID); | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 100 | if (ret != 0) { | 
| Nayna Jain | 673a226 | 2020-12-14 22:44:49 +0000 | [diff] [blame] | 101 | *p = start; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 102 | return MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_CONTENT_INFO, ret); | 
| Nayna Jain | 673a226 | 2020-12-14 22:44:49 +0000 | [diff] [blame] | 103 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 104 |  | 
|  | 105 | pkcs7->tag = MBEDTLS_ASN1_OID; | 
|  | 106 | pkcs7->len = len; | 
|  | 107 | pkcs7->p = *p; | 
| Nick Child | bb82ab7 | 2022-10-28 12:28:54 -0500 | [diff] [blame] | 108 | *p += len; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 109 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 110 | return ret; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 111 | } | 
|  | 112 |  | 
|  | 113 | /** | 
|  | 114 | * DigestAlgorithmIdentifier ::= AlgorithmIdentifier | 
|  | 115 | * | 
|  | 116 | * This is from x509.h | 
|  | 117 | **/ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 118 | static int pkcs7_get_digest_algorithm(unsigned char **p, unsigned char *end, | 
|  | 119 | mbedtls_x509_buf *alg) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 120 | { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 121 | int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 122 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 123 | if ((ret = mbedtls_asn1_get_alg_null(p, end, alg)) != 0) { | 
|  | 124 | ret = MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_ALG, ret); | 
|  | 125 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 126 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 127 | return ret; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 128 | } | 
|  | 129 |  | 
|  | 130 | /** | 
|  | 131 | * DigestAlgorithmIdentifiers :: SET of DigestAlgorithmIdentifier | 
|  | 132 | **/ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 133 | static int pkcs7_get_digest_algorithm_set(unsigned char **p, | 
|  | 134 | unsigned char *end, | 
|  | 135 | mbedtls_x509_buf *alg) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 136 | { | 
|  | 137 | size_t len = 0; | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 138 | int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 139 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 140 | ret = mbedtls_asn1_get_tag(p, end, &len, MBEDTLS_ASN1_CONSTRUCTED | 
|  | 141 | | MBEDTLS_ASN1_SET); | 
|  | 142 | if (ret != 0) { | 
|  | 143 | return MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_ALG, ret); | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 144 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 145 |  | 
|  | 146 | end = *p + len; | 
|  | 147 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 148 | ret = mbedtls_asn1_get_alg_null(p, end, alg); | 
|  | 149 | if (ret != 0) { | 
|  | 150 | return MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_ALG, ret); | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 151 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 152 |  | 
| Nick Child | 34d5e93 | 2022-09-14 14:44:03 -0500 | [diff] [blame] | 153 | /** For now, it assumes there is only one digest algorithm specified **/ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 154 | if (*p != end) { | 
|  | 155 | return MBEDTLS_ERR_PKCS7_FEATURE_UNAVAILABLE; | 
|  | 156 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 157 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 158 | return 0; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 159 | } | 
|  | 160 |  | 
|  | 161 | /** | 
|  | 162 | * certificates :: SET OF ExtendedCertificateOrCertificate, | 
|  | 163 | * ExtendedCertificateOrCertificate ::= CHOICE { | 
|  | 164 | *      certificate Certificate -- x509, | 
|  | 165 | *      extendedCertificate[0] IMPLICIT ExtendedCertificate } | 
|  | 166 | * Return number of certificates added to the signed data, | 
|  | 167 | * 0 or higher is valid. | 
|  | 168 | * Return negative error code for failure. | 
|  | 169 | **/ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 170 | static int pkcs7_get_certificates(unsigned char **p, unsigned char *end, | 
|  | 171 | mbedtls_x509_crt *certs) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 172 | { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 173 | int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 174 | size_t len1 = 0; | 
|  | 175 | size_t len2 = 0; | 
| Nick Child | bb82ab7 | 2022-10-28 12:28:54 -0500 | [diff] [blame] | 176 | unsigned char *end_set, *end_cert, *start; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 177 |  | 
| Demi Marie Obenour | 6cfc469 | 2022-11-28 00:46:00 -0500 | [diff] [blame] | 178 | ret = mbedtls_asn1_get_tag(p, end, &len1, MBEDTLS_ASN1_CONSTRUCTED | 
|  | 179 | | MBEDTLS_ASN1_CONTEXT_SPECIFIC); | 
|  | 180 | if (ret == MBEDTLS_ERR_ASN1_UNEXPECTED_TAG) { | 
|  | 181 | return 0; | 
|  | 182 | } | 
|  | 183 | if (ret != 0) { | 
|  | 184 | return MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_FORMAT, ret); | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 185 | } | 
|  | 186 | start = *p; | 
|  | 187 | end_set = *p + len1; | 
|  | 188 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 189 | ret = mbedtls_asn1_get_tag(p, end_set, &len2, MBEDTLS_ASN1_CONSTRUCTED | 
|  | 190 | | MBEDTLS_ASN1_SEQUENCE); | 
|  | 191 | if (ret != 0) { | 
|  | 192 | return MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_CERT, ret); | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 193 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 194 |  | 
|  | 195 | end_cert = *p + len2; | 
|  | 196 |  | 
|  | 197 | /* | 
|  | 198 | * This is to verify that there is only one signer certificate. It seems it is | 
|  | 199 | * not easy to differentiate between the chain vs different signer's certificate. | 
|  | 200 | * So, we support only the root certificate and the single signer. | 
|  | 201 | * The behaviour would be improved with addition of multiple signer support. | 
|  | 202 | */ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 203 | if (end_cert != end_set) { | 
|  | 204 | return MBEDTLS_ERR_PKCS7_FEATURE_UNAVAILABLE; | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 205 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 206 |  | 
| Demi Marie Obenour | 55d9df2 | 2022-11-28 00:29:32 -0500 | [diff] [blame] | 207 | if ((ret = mbedtls_x509_crt_parse_der(certs, start, len1)) < 0) { | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 208 | return MBEDTLS_ERR_PKCS7_INVALID_CERT; | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 209 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 210 |  | 
| Demi Marie Obenour | 55d9df2 | 2022-11-28 00:29:32 -0500 | [diff] [blame] | 211 | *p = end_cert; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 212 |  | 
| Nick Child | 5f9456f | 2022-09-19 10:01:25 -0500 | [diff] [blame] | 213 | /* | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 214 | * Since in this version we strictly support single certificate, and reaching | 
| Nick Child | 5f9456f | 2022-09-19 10:01:25 -0500 | [diff] [blame] | 215 | * here implies we have parsed successfully, we return 1. | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 216 | */ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 217 | return 1; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 218 | } | 
|  | 219 |  | 
|  | 220 | /** | 
|  | 221 | * EncryptedDigest ::= OCTET STRING | 
|  | 222 | **/ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 223 | static int pkcs7_get_signature(unsigned char **p, unsigned char *end, | 
|  | 224 | mbedtls_pkcs7_buf *signature) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 225 | { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 226 | int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 227 | size_t len = 0; | 
|  | 228 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 229 | ret = mbedtls_asn1_get_tag(p, end, &len, MBEDTLS_ASN1_OCTET_STRING); | 
|  | 230 | if (ret != 0) { | 
|  | 231 | return ret; | 
|  | 232 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 233 |  | 
|  | 234 | signature->tag = MBEDTLS_ASN1_OCTET_STRING; | 
|  | 235 | signature->len = len; | 
|  | 236 | signature->p = *p; | 
|  | 237 |  | 
|  | 238 | *p = *p + len; | 
|  | 239 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 240 | return 0; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 241 | } | 
|  | 242 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 243 | static void pkcs7_free_signer_info(mbedtls_pkcs7_signer_info *signer) | 
| Bence Szépkúti | f764154 | 2022-12-12 21:37:36 +0100 | [diff] [blame] | 244 | { | 
|  | 245 | mbedtls_x509_name *name_cur; | 
|  | 246 | mbedtls_x509_name *name_prv; | 
|  | 247 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 248 | if (signer == NULL) { | 
| Bence Szépkúti | f764154 | 2022-12-12 21:37:36 +0100 | [diff] [blame] | 249 | return; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 250 | } | 
| Bence Szépkúti | f764154 | 2022-12-12 21:37:36 +0100 | [diff] [blame] | 251 |  | 
|  | 252 | name_cur = signer->issuer.next; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 253 | while (name_cur != NULL) { | 
| Bence Szépkúti | f764154 | 2022-12-12 21:37:36 +0100 | [diff] [blame] | 254 | name_prv = name_cur; | 
|  | 255 | name_cur = name_cur->next; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 256 | mbedtls_free(name_prv); | 
| Bence Szépkúti | f764154 | 2022-12-12 21:37:36 +0100 | [diff] [blame] | 257 | } | 
|  | 258 | signer->issuer.next = NULL; | 
|  | 259 | } | 
|  | 260 |  | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 261 | /** | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 262 | * SignerInfo ::= SEQUENCE { | 
|  | 263 | *      version Version; | 
|  | 264 | *      issuerAndSerialNumber   IssuerAndSerialNumber, | 
|  | 265 | *      digestAlgorithm DigestAlgorithmIdentifier, | 
|  | 266 | *      authenticatedAttributes | 
|  | 267 | *              [0] IMPLICIT Attributes OPTIONAL, | 
|  | 268 | *      digestEncryptionAlgorithm DigestEncryptionAlgorithmIdentifier, | 
|  | 269 | *      encryptedDigest EncryptedDigest, | 
|  | 270 | *      unauthenticatedAttributes | 
|  | 271 | *              [1] IMPLICIT Attributes OPTIONAL, | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 272 | * Returns 0 if the signerInfo is valid. | 
|  | 273 | * Return negative error code for failure. | 
| Nick Child | bb82ab7 | 2022-10-28 12:28:54 -0500 | [diff] [blame] | 274 | * Structure must not contain vales for authenticatedAttributes | 
|  | 275 | * and unauthenticatedAttributes. | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 276 | **/ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 277 | static int pkcs7_get_signer_info(unsigned char **p, unsigned char *end, | 
| Demi Marie Obenour | 35598ad | 2022-11-30 02:06:07 -0500 | [diff] [blame] | 278 | mbedtls_pkcs7_signer_info *signer, | 
|  | 279 | mbedtls_x509_buf *alg) | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 280 | { | 
| Nick Child | ec81709 | 2022-12-15 15:54:03 -0600 | [diff] [blame] | 281 | unsigned char *end_signer, *end_issuer_and_sn; | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 282 | int asn1_ret = 0, ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED; | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 283 | size_t len = 0; | 
|  | 284 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 285 | asn1_ret = mbedtls_asn1_get_tag(p, end, &len, MBEDTLS_ASN1_CONSTRUCTED | 
|  | 286 | | MBEDTLS_ASN1_SEQUENCE); | 
|  | 287 | if (asn1_ret != 0) { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 288 | goto out; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 289 | } | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 290 |  | 
|  | 291 | end_signer = *p + len; | 
|  | 292 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 293 | ret = pkcs7_get_version(p, end_signer, &signer->version); | 
|  | 294 | if (ret != 0) { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 295 | goto out; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 296 | } | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 297 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 298 | asn1_ret = mbedtls_asn1_get_tag(p, end_signer, &len, | 
|  | 299 | MBEDTLS_ASN1_CONSTRUCTED | MBEDTLS_ASN1_SEQUENCE); | 
|  | 300 | if (asn1_ret != 0) { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 301 | goto out; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 302 | } | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 303 |  | 
| Nick Child | ec81709 | 2022-12-15 15:54:03 -0600 | [diff] [blame] | 304 | end_issuer_and_sn = *p + len; | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 305 | /* Parsing IssuerAndSerialNumber */ | 
|  | 306 | signer->issuer_raw.p = *p; | 
|  | 307 |  | 
| Nick Child | 3bd17f2 | 2023-01-31 20:42:26 +0000 | [diff] [blame] | 308 | asn1_ret = mbedtls_asn1_get_tag(p, end_issuer_and_sn, &len, | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 309 | MBEDTLS_ASN1_CONSTRUCTED | MBEDTLS_ASN1_SEQUENCE); | 
|  | 310 | if (asn1_ret != 0) { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 311 | goto out; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 312 | } | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 313 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 314 | ret  = mbedtls_x509_get_name(p, *p + len, &signer->issuer); | 
|  | 315 | if (ret != 0) { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 316 | goto out; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 317 | } | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 318 |  | 
| Dave Rodgman | e4a6f5a | 2023-11-04 12:20:09 +0000 | [diff] [blame] | 319 | signer->issuer_raw.len =  (size_t) (*p - signer->issuer_raw.p); | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 320 |  | 
| Nick Child | 3bd17f2 | 2023-01-31 20:42:26 +0000 | [diff] [blame] | 321 | ret = mbedtls_x509_get_serial(p, end_issuer_and_sn, &signer->serial); | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 322 | if (ret != 0) { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 323 | goto out; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 324 | } | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 325 |  | 
| Nick Child | ec81709 | 2022-12-15 15:54:03 -0600 | [diff] [blame] | 326 | /* ensure no extra or missing bytes */ | 
|  | 327 | if (*p != end_issuer_and_sn) { | 
|  | 328 | ret = MBEDTLS_ERR_PKCS7_INVALID_SIGNER_INFO; | 
|  | 329 | goto out; | 
|  | 330 | } | 
|  | 331 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 332 | ret = pkcs7_get_digest_algorithm(p, end_signer, &signer->alg_identifier); | 
|  | 333 | if (ret != 0) { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 334 | goto out; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 335 | } | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 336 |  | 
| Demi Marie Obenour | 35598ad | 2022-11-30 02:06:07 -0500 | [diff] [blame] | 337 | /* Check that the digest algorithm used matches the one provided earlier */ | 
|  | 338 | if (signer->alg_identifier.tag != alg->tag || | 
|  | 339 | signer->alg_identifier.len != alg->len || | 
|  | 340 | memcmp(signer->alg_identifier.p, alg->p, alg->len) != 0) { | 
|  | 341 | ret = MBEDTLS_ERR_PKCS7_INVALID_SIGNER_INFO; | 
|  | 342 | goto out; | 
|  | 343 | } | 
| Nick Child | bb82ab7 | 2022-10-28 12:28:54 -0500 | [diff] [blame] | 344 |  | 
| Tom Cosgrove | 5c8505f | 2023-03-07 11:39:52 +0000 | [diff] [blame] | 345 | /* Assume authenticatedAttributes is nonexistent */ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 346 | ret = pkcs7_get_digest_algorithm(p, end_signer, &signer->sig_alg_identifier); | 
|  | 347 | if (ret != 0) { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 348 | goto out; | 
| Bence Szépkúti | f764154 | 2022-12-12 21:37:36 +0100 | [diff] [blame] | 349 | } | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 350 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 351 | ret = pkcs7_get_signature(p, end_signer, &signer->sig); | 
|  | 352 | if (ret != 0) { | 
|  | 353 | goto out; | 
|  | 354 | } | 
|  | 355 |  | 
|  | 356 | /* Do not permit any unauthenticated attributes */ | 
|  | 357 | if (*p != end_signer) { | 
|  | 358 | ret = MBEDTLS_ERR_PKCS7_INVALID_SIGNER_INFO; | 
|  | 359 | } | 
|  | 360 |  | 
|  | 361 | out: | 
|  | 362 | if (asn1_ret != 0 || ret != 0) { | 
|  | 363 | pkcs7_free_signer_info(signer); | 
|  | 364 | ret = MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_SIGNER_INFO, | 
|  | 365 | asn1_ret); | 
|  | 366 | } | 
|  | 367 |  | 
|  | 368 | return ret; | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 369 | } | 
|  | 370 |  | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 371 | /** | 
|  | 372 | * SignerInfos ::= SET of SignerInfo | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 373 | * Return number of signers added to the signed data, | 
|  | 374 | * 0 or higher is valid. | 
|  | 375 | * Return negative error code for failure. | 
|  | 376 | **/ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 377 | static int pkcs7_get_signers_info_set(unsigned char **p, unsigned char *end, | 
| Demi Marie Obenour | 35598ad | 2022-11-30 02:06:07 -0500 | [diff] [blame] | 378 | mbedtls_pkcs7_signer_info *signers_set, | 
|  | 379 | mbedtls_x509_buf *digest_alg) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 380 | { | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 381 | unsigned char *end_set; | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 382 | int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED; | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 383 | int count = 0; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 384 | size_t len = 0; | 
|  | 385 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 386 | ret = mbedtls_asn1_get_tag(p, end, &len, MBEDTLS_ASN1_CONSTRUCTED | 
|  | 387 | | MBEDTLS_ASN1_SET); | 
|  | 388 | if (ret != 0) { | 
|  | 389 | return MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_SIGNER_INFO, ret); | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 390 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 391 |  | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 392 | /* Detect zero signers */ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 393 | if (len == 0) { | 
|  | 394 | return 0; | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 395 | } | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 396 |  | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 397 | end_set = *p + len; | 
|  | 398 |  | 
| Demi Marie Obenour | 35598ad | 2022-11-30 02:06:07 -0500 | [diff] [blame] | 399 | ret = pkcs7_get_signer_info(p, end_set, signers_set, digest_alg); | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 400 | if (ret != 0) { | 
|  | 401 | return ret; | 
|  | 402 | } | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 403 | count++; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 404 |  | 
| Gilles Peskine | 47a7326 | 2022-11-27 21:46:56 +0100 | [diff] [blame] | 405 | mbedtls_pkcs7_signer_info *prev = signers_set; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 406 | while (*p != end_set) { | 
| Gilles Peskine | 47a7326 | 2022-11-27 21:46:56 +0100 | [diff] [blame] | 407 | mbedtls_pkcs7_signer_info *signer = | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 408 | mbedtls_calloc(1, sizeof(mbedtls_pkcs7_signer_info)); | 
|  | 409 | if (!signer) { | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 410 | ret = MBEDTLS_ERR_PKCS7_ALLOC_FAILED; | 
|  | 411 | goto cleanup; | 
|  | 412 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 413 |  | 
| Demi Marie Obenour | 35598ad | 2022-11-30 02:06:07 -0500 | [diff] [blame] | 414 | ret = pkcs7_get_signer_info(p, end_set, signer, digest_alg); | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 415 | if (ret != 0) { | 
|  | 416 | mbedtls_free(signer); | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 417 | goto cleanup; | 
|  | 418 | } | 
|  | 419 | prev->next = signer; | 
|  | 420 | prev = signer; | 
|  | 421 | count++; | 
|  | 422 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 423 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 424 | return count; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 425 |  | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 426 | cleanup: | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 427 | pkcs7_free_signer_info(signers_set); | 
| Gilles Peskine | 47a7326 | 2022-11-27 21:46:56 +0100 | [diff] [blame] | 428 | mbedtls_pkcs7_signer_info *signer = signers_set->next; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 429 | while (signer != NULL) { | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 430 | prev = signer; | 
|  | 431 | signer = signer->next; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 432 | pkcs7_free_signer_info(prev); | 
|  | 433 | mbedtls_free(prev); | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 434 | } | 
| Gilles Peskine | 290f01b | 2022-11-27 21:28:31 +0100 | [diff] [blame] | 435 | signers_set->next = NULL; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 436 | return ret; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 437 | } | 
|  | 438 |  | 
|  | 439 | /** | 
|  | 440 | * SignedData ::= SEQUENCE { | 
|  | 441 | *      version Version, | 
|  | 442 | *      digestAlgorithms DigestAlgorithmIdentifiers, | 
|  | 443 | *      contentInfo ContentInfo, | 
|  | 444 | *      certificates | 
|  | 445 | *              [0] IMPLICIT ExtendedCertificatesAndCertificates | 
|  | 446 | *                  OPTIONAL, | 
|  | 447 | *      crls | 
|  | 448 | *              [0] IMPLICIT CertificateRevocationLists OPTIONAL, | 
|  | 449 | *      signerInfos SignerInfos } | 
|  | 450 | */ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 451 | static int pkcs7_get_signed_data(unsigned char *buf, size_t buflen, | 
|  | 452 | mbedtls_pkcs7_signed_data *signed_data) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 453 | { | 
|  | 454 | unsigned char *p = buf; | 
|  | 455 | unsigned char *end = buf + buflen; | 
| Dave Rodgman | f691268 | 2023-02-09 17:55:41 +0000 | [diff] [blame] | 456 | unsigned char *end_content_info = NULL; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 457 | size_t len = 0; | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 458 | int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 459 | mbedtls_md_type_t md_alg; | 
|  | 460 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 461 | ret = mbedtls_asn1_get_tag(&p, end, &len, MBEDTLS_ASN1_CONSTRUCTED | 
|  | 462 | | MBEDTLS_ASN1_SEQUENCE); | 
|  | 463 | if (ret != 0) { | 
|  | 464 | return MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_FORMAT, ret); | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 465 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 466 |  | 
| Demi Marie Obenour | 4ec8355 | 2022-11-28 00:23:00 -0500 | [diff] [blame] | 467 | if (p + len != end) { | 
|  | 468 | return MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_FORMAT, | 
|  | 469 | MBEDTLS_ERR_ASN1_LENGTH_MISMATCH); | 
|  | 470 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 471 |  | 
|  | 472 | /* Get version of signed data */ | 
| Demi Marie Obenour | 4ec8355 | 2022-11-28 00:23:00 -0500 | [diff] [blame] | 473 | ret = pkcs7_get_version(&p, end, &signed_data->version); | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 474 | if (ret != 0) { | 
|  | 475 | return ret; | 
|  | 476 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 477 |  | 
|  | 478 | /* Get digest algorithm */ | 
| Demi Marie Obenour | 4ec8355 | 2022-11-28 00:23:00 -0500 | [diff] [blame] | 479 | ret = pkcs7_get_digest_algorithm_set(&p, end, | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 480 | &signed_data->digest_alg_identifiers); | 
|  | 481 | if (ret != 0) { | 
|  | 482 | return ret; | 
|  | 483 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 484 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 485 | ret = mbedtls_oid_get_md_alg(&signed_data->digest_alg_identifiers, &md_alg); | 
|  | 486 | if (ret != 0) { | 
|  | 487 | return MBEDTLS_ERR_PKCS7_INVALID_ALG; | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 488 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 489 |  | 
| Demi Marie Obenour | e373a25 | 2022-12-13 23:50:03 -0500 | [diff] [blame] | 490 | mbedtls_pkcs7_buf content_type; | 
| Dave Rodgman | f691268 | 2023-02-09 17:55:41 +0000 | [diff] [blame] | 491 | memset(&content_type, 0, sizeof(content_type)); | 
| Demi Marie Obenour | e373a25 | 2022-12-13 23:50:03 -0500 | [diff] [blame] | 492 | ret = pkcs7_get_content_info_type(&p, end, &end_content_info, &content_type); | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 493 | if (ret != 0) { | 
|  | 494 | return ret; | 
|  | 495 | } | 
| Demi Marie Obenour | e373a25 | 2022-12-13 23:50:03 -0500 | [diff] [blame] | 496 | if (MBEDTLS_OID_CMP(MBEDTLS_OID_PKCS7_DATA, &content_type)) { | 
|  | 497 | return MBEDTLS_ERR_PKCS7_INVALID_CONTENT_INFO; | 
|  | 498 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 499 |  | 
| Nick Child | 3dafc6c | 2023-02-07 19:59:58 +0000 | [diff] [blame] | 500 | if (p != end_content_info) { | 
|  | 501 | /* Determine if valid content is present */ | 
| Dave Rodgman | 78c6f40 | 2023-02-09 09:21:14 +0000 | [diff] [blame] | 502 | ret = mbedtls_asn1_get_tag(&p, | 
|  | 503 | end_content_info, | 
|  | 504 | &len, | 
|  | 505 | MBEDTLS_ASN1_CONSTRUCTED | MBEDTLS_ASN1_CONTEXT_SPECIFIC); | 
| Nick Child | 3dafc6c | 2023-02-07 19:59:58 +0000 | [diff] [blame] | 506 | if (ret != 0) { | 
|  | 507 | return MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_CONTENT_INFO, ret); | 
|  | 508 | } | 
|  | 509 | p += len; | 
|  | 510 | if (p != end_content_info) { | 
|  | 511 | return MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_CONTENT_INFO, ret); | 
|  | 512 | } | 
|  | 513 | /* Valid content is present - this is not supported */ | 
|  | 514 | return MBEDTLS_ERR_PKCS7_FEATURE_UNAVAILABLE; | 
| Nick Child | ec81709 | 2022-12-15 15:54:03 -0600 | [diff] [blame] | 515 | } | 
|  | 516 |  | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 517 | /* Look for certificates, there may or may not be any */ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 518 | mbedtls_x509_crt_init(&signed_data->certs); | 
| Demi Marie Obenour | 4ec8355 | 2022-11-28 00:23:00 -0500 | [diff] [blame] | 519 | ret = pkcs7_get_certificates(&p, end, &signed_data->certs); | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 520 | if (ret < 0) { | 
|  | 521 | return ret; | 
|  | 522 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 523 |  | 
|  | 524 | signed_data->no_of_certs = ret; | 
|  | 525 |  | 
|  | 526 | /* | 
|  | 527 | * Currently CRLs are not supported. If CRL exist, the parsing will fail | 
|  | 528 | * at next step of getting signers info and return error as invalid | 
|  | 529 | * signer info. | 
|  | 530 | */ | 
|  | 531 |  | 
|  | 532 | signed_data->no_of_crls = 0; | 
|  | 533 |  | 
|  | 534 | /* Get signers info */ | 
| Demi Marie Obenour | 35598ad | 2022-11-30 02:06:07 -0500 | [diff] [blame] | 535 | ret = pkcs7_get_signers_info_set(&p, | 
|  | 536 | end, | 
|  | 537 | &signed_data->signers, | 
|  | 538 | &signed_data->digest_alg_identifiers); | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 539 | if (ret < 0) { | 
|  | 540 | return ret; | 
|  | 541 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 542 |  | 
|  | 543 | signed_data->no_of_signers = ret; | 
|  | 544 |  | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 545 | /* Don't permit trailing data */ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 546 | if (p != end) { | 
|  | 547 | return MBEDTLS_ERR_PKCS7_INVALID_FORMAT; | 
|  | 548 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 549 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 550 | return 0; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 551 | } | 
|  | 552 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 553 | int mbedtls_pkcs7_parse_der(mbedtls_pkcs7 *pkcs7, const unsigned char *buf, | 
|  | 554 | const size_t buflen) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 555 | { | 
| Nick Child | bb82ab7 | 2022-10-28 12:28:54 -0500 | [diff] [blame] | 556 | unsigned char *p; | 
| Demi Marie Obenour | 4ec8355 | 2022-11-28 00:23:00 -0500 | [diff] [blame] | 557 | unsigned char *end; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 558 | size_t len = 0; | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 559 | int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 560 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 561 | if (pkcs7 == NULL) { | 
|  | 562 | return MBEDTLS_ERR_PKCS7_BAD_INPUT_DATA; | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 563 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 564 |  | 
|  | 565 | /* make an internal copy of the buffer for parsing */ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 566 | pkcs7->raw.p = p = mbedtls_calloc(1, buflen); | 
|  | 567 | if (pkcs7->raw.p == NULL) { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 568 | ret = MBEDTLS_ERR_PKCS7_ALLOC_FAILED; | 
|  | 569 | goto out; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 570 | } | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 571 | memcpy(p, buf, buflen); | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 572 | pkcs7->raw.len = buflen; | 
| Nick Child | bb82ab7 | 2022-10-28 12:28:54 -0500 | [diff] [blame] | 573 | end = p + buflen; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 574 |  | 
| Demi Marie Obenour | aaf3c00 | 2022-11-28 00:20:42 -0500 | [diff] [blame] | 575 | ret = mbedtls_asn1_get_tag(&p, end, &len, MBEDTLS_ASN1_CONSTRUCTED | 
|  | 576 | | MBEDTLS_ASN1_SEQUENCE); | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 577 | if (ret != 0) { | 
| Demi Marie Obenour | aaf3c00 | 2022-11-28 00:20:42 -0500 | [diff] [blame] | 578 | ret = MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_FORMAT, ret); | 
|  | 579 | goto out; | 
|  | 580 | } | 
|  | 581 |  | 
|  | 582 | if ((size_t) (end - p) != len) { | 
|  | 583 | ret = MBEDTLS_ERROR_ADD(MBEDTLS_ERR_PKCS7_INVALID_FORMAT, | 
|  | 584 | MBEDTLS_ERR_ASN1_LENGTH_MISMATCH); | 
|  | 585 | goto out; | 
|  | 586 | } | 
|  | 587 |  | 
|  | 588 | if ((ret = mbedtls_asn1_get_tag(&p, end, &len, MBEDTLS_ASN1_OID)) != 0) { | 
|  | 589 | if (ret != MBEDTLS_ERR_ASN1_UNEXPECTED_TAG) { | 
|  | 590 | goto out; | 
|  | 591 | } | 
|  | 592 | p = pkcs7->raw.p; | 
| Nayna Jain | 673a226 | 2020-12-14 22:44:49 +0000 | [diff] [blame] | 593 | len = buflen; | 
|  | 594 | goto try_data; | 
|  | 595 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 596 |  | 
| Demi Marie Obenour | aaf3c00 | 2022-11-28 00:20:42 -0500 | [diff] [blame] | 597 | if (MBEDTLS_OID_CMP_RAW(MBEDTLS_OID_PKCS7_SIGNED_DATA, p, len)) { | 
| Dave Rodgman | a1b2bff | 2023-02-20 14:45:09 +0000 | [diff] [blame] | 598 | /* OID is not MBEDTLS_OID_PKCS7_SIGNED_DATA, which is the only supported feature */ | 
| Demi Marie Obenour | aaf3c00 | 2022-11-28 00:20:42 -0500 | [diff] [blame] | 599 | if (!MBEDTLS_OID_CMP_RAW(MBEDTLS_OID_PKCS7_DATA, p, len) | 
|  | 600 | || !MBEDTLS_OID_CMP_RAW(MBEDTLS_OID_PKCS7_ENCRYPTED_DATA, p, len) | 
|  | 601 | || !MBEDTLS_OID_CMP_RAW(MBEDTLS_OID_PKCS7_ENVELOPED_DATA, p, len) | 
|  | 602 | || !MBEDTLS_OID_CMP_RAW(MBEDTLS_OID_PKCS7_SIGNED_AND_ENVELOPED_DATA, p, len) | 
|  | 603 | || !MBEDTLS_OID_CMP_RAW(MBEDTLS_OID_PKCS7_DIGESTED_DATA, p, len)) { | 
| Dave Rodgman | a1b2bff | 2023-02-20 14:45:09 +0000 | [diff] [blame] | 604 | /* OID is valid according to the spec, but unsupported */ | 
| Demi Marie Obenour | aaf3c00 | 2022-11-28 00:20:42 -0500 | [diff] [blame] | 605 | ret =  MBEDTLS_ERR_PKCS7_FEATURE_UNAVAILABLE; | 
|  | 606 | } else { | 
| Dave Rodgman | a1b2bff | 2023-02-20 14:45:09 +0000 | [diff] [blame] | 607 | /* OID is invalid according to the spec */ | 
| Demi Marie Obenour | aaf3c00 | 2022-11-28 00:20:42 -0500 | [diff] [blame] | 608 | ret = MBEDTLS_ERR_PKCS7_BAD_INPUT_DATA; | 
|  | 609 | } | 
| Nick Child | ec81709 | 2022-12-15 15:54:03 -0600 | [diff] [blame] | 610 | goto out; | 
|  | 611 | } | 
|  | 612 |  | 
| Demi Marie Obenour | aaf3c00 | 2022-11-28 00:20:42 -0500 | [diff] [blame] | 613 | p += len; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 614 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 615 | ret = pkcs7_get_next_content_len(&p, end, &len); | 
|  | 616 | if (ret != 0) { | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 617 | goto out; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 618 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 619 |  | 
| Nick Child | ec81709 | 2022-12-15 15:54:03 -0600 | [diff] [blame] | 620 | /* ensure no extra/missing data */ | 
|  | 621 | if (p + len != end) { | 
|  | 622 | ret = MBEDTLS_ERR_PKCS7_BAD_INPUT_DATA; | 
|  | 623 | goto out; | 
|  | 624 | } | 
|  | 625 |  | 
| Nayna Jain | 673a226 | 2020-12-14 22:44:49 +0000 | [diff] [blame] | 626 | try_data: | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 627 | ret = pkcs7_get_signed_data(p, len, &pkcs7->signed_data); | 
|  | 628 | if (ret != 0) { | 
| Nayna Jain | 673a226 | 2020-12-14 22:44:49 +0000 | [diff] [blame] | 629 | goto out; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 630 | } | 
| Nayna Jain | 673a226 | 2020-12-14 22:44:49 +0000 | [diff] [blame] | 631 |  | 
| Nayna Jain | 673a226 | 2020-12-14 22:44:49 +0000 | [diff] [blame] | 632 | ret = MBEDTLS_PKCS7_SIGNED_DATA; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 633 |  | 
|  | 634 | out: | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 635 | if (ret < 0) { | 
|  | 636 | mbedtls_pkcs7_free(pkcs7); | 
|  | 637 | } | 
| Nayna Jain | 673a226 | 2020-12-14 22:44:49 +0000 | [diff] [blame] | 638 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 639 | return ret; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 640 | } | 
|  | 641 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 642 | static int mbedtls_pkcs7_data_or_hash_verify(mbedtls_pkcs7 *pkcs7, | 
| Nick Child | 73621ef | 2022-10-28 11:23:15 -0500 | [diff] [blame] | 643 | const mbedtls_x509_crt *cert, | 
|  | 644 | const unsigned char *data, | 
|  | 645 | size_t datalen, | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 646 | const int is_data_hash) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 647 | { | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 648 | int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 649 | unsigned char *hash; | 
|  | 650 | mbedtls_pk_context pk_cxt = cert->pk; | 
|  | 651 | const mbedtls_md_info_t *md_info; | 
|  | 652 | mbedtls_md_type_t md_alg; | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 653 | mbedtls_pkcs7_signer_info *signer; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 654 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 655 | if (pkcs7->signed_data.no_of_signers == 0) { | 
|  | 656 | return MBEDTLS_ERR_PKCS7_INVALID_CERT; | 
| Nick Child | 9f4fb3e | 2022-09-12 16:21:02 -0500 | [diff] [blame] | 657 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 658 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 659 | if (mbedtls_x509_time_is_past(&cert->valid_to) || | 
|  | 660 | mbedtls_x509_time_is_future(&cert->valid_from)) { | 
|  | 661 | return MBEDTLS_ERR_PKCS7_CERT_DATE_INVALID; | 
| Nick Child | 73621ef | 2022-10-28 11:23:15 -0500 | [diff] [blame] | 662 | } | 
|  | 663 |  | 
| Demi Marie Obenour | 35598ad | 2022-11-30 02:06:07 -0500 | [diff] [blame] | 664 | ret = mbedtls_oid_get_md_alg(&pkcs7->signed_data.digest_alg_identifiers, &md_alg); | 
|  | 665 | if (ret != 0) { | 
|  | 666 | return ret; | 
|  | 667 | } | 
|  | 668 |  | 
|  | 669 | md_info = mbedtls_md_info_from_type(md_alg); | 
|  | 670 | if (md_info == NULL) { | 
|  | 671 | return MBEDTLS_ERR_PKCS7_VERIFY_FAIL; | 
|  | 672 | } | 
|  | 673 |  | 
|  | 674 | hash = mbedtls_calloc(mbedtls_md_get_size(md_info), 1); | 
|  | 675 | if (hash == NULL) { | 
|  | 676 | return MBEDTLS_ERR_PKCS7_ALLOC_FAILED; | 
|  | 677 | } | 
| Demi Marie Obenour | 35598ad | 2022-11-30 02:06:07 -0500 | [diff] [blame] | 678 |  | 
| Dave Rodgman | fc64352 | 2023-02-16 16:23:09 +0000 | [diff] [blame] | 679 | /* BEGIN must free hash before jumping out */ | 
| Demi Marie Obenour | 35598ad | 2022-11-30 02:06:07 -0500 | [diff] [blame] | 680 | if (is_data_hash) { | 
|  | 681 | if (datalen != mbedtls_md_get_size(md_info)) { | 
|  | 682 | ret = MBEDTLS_ERR_PKCS7_VERIFY_FAIL; | 
|  | 683 | } else { | 
|  | 684 | memcpy(hash, data, datalen); | 
|  | 685 | } | 
|  | 686 | } else { | 
|  | 687 | ret = mbedtls_md(md_info, data, datalen, hash); | 
|  | 688 | } | 
|  | 689 | if (ret != 0) { | 
|  | 690 | mbedtls_free(hash); | 
|  | 691 | return MBEDTLS_ERR_PKCS7_VERIFY_FAIL; | 
|  | 692 | } | 
|  | 693 |  | 
|  | 694 | /* assume failure */ | 
|  | 695 | ret = MBEDTLS_ERR_PKCS7_VERIFY_FAIL; | 
|  | 696 |  | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 697 | /* | 
|  | 698 | * Potential TODOs | 
|  | 699 | * Currently we iterate over all signers and return success if any of them | 
|  | 700 | * verify. | 
|  | 701 | * | 
|  | 702 | * However, we could make this better by checking against the certificate's | 
|  | 703 | * identification and SignerIdentifier fields first. That would also allow | 
|  | 704 | * us to distinguish between 'no signature for key' and 'signature for key | 
|  | 705 | * failed to validate'. | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 706 | */ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 707 | for (signer = &pkcs7->signed_data.signers; signer; signer = signer->next) { | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 708 | ret = mbedtls_pk_verify(&pk_cxt, md_alg, hash, | 
|  | 709 | mbedtls_md_get_size(md_info), | 
|  | 710 | signer->sig.p, signer->sig.len); | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 711 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 712 | if (ret == 0) { | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 713 | break; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 714 | } | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 715 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 716 |  | 
| Demi Marie Obenour | 35598ad | 2022-11-30 02:06:07 -0500 | [diff] [blame] | 717 | mbedtls_free(hash); | 
|  | 718 | /* END must free hash before jumping out */ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 719 | return ret; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 720 | } | 
| Demi Marie Obenour | 6cfc469 | 2022-11-28 00:46:00 -0500 | [diff] [blame] | 721 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 722 | int mbedtls_pkcs7_signed_data_verify(mbedtls_pkcs7 *pkcs7, | 
|  | 723 | const mbedtls_x509_crt *cert, | 
|  | 724 | const unsigned char *data, | 
|  | 725 | size_t datalen) | 
| Nick Child | 73621ef | 2022-10-28 11:23:15 -0500 | [diff] [blame] | 726 | { | 
| Demi Marie Obenour | 6cfc469 | 2022-11-28 00:46:00 -0500 | [diff] [blame] | 727 | if (data == NULL) { | 
|  | 728 | return MBEDTLS_ERR_PKCS7_BAD_INPUT_DATA; | 
|  | 729 | } | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 730 | return mbedtls_pkcs7_data_or_hash_verify(pkcs7, cert, data, datalen, 0); | 
| Nick Child | 73621ef | 2022-10-28 11:23:15 -0500 | [diff] [blame] | 731 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 732 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 733 | int mbedtls_pkcs7_signed_hash_verify(mbedtls_pkcs7 *pkcs7, | 
|  | 734 | const mbedtls_x509_crt *cert, | 
|  | 735 | const unsigned char *hash, | 
|  | 736 | size_t hashlen) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 737 | { | 
| Demi Marie Obenour | 6cfc469 | 2022-11-28 00:46:00 -0500 | [diff] [blame] | 738 | if (hash == NULL) { | 
|  | 739 | return MBEDTLS_ERR_PKCS7_BAD_INPUT_DATA; | 
|  | 740 | } | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 741 | return mbedtls_pkcs7_data_or_hash_verify(pkcs7, cert, hash, hashlen, 1); | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 742 | } | 
|  | 743 |  | 
|  | 744 | /* | 
|  | 745 | * Unallocate all pkcs7 data | 
|  | 746 | */ | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 747 | void mbedtls_pkcs7_free(mbedtls_pkcs7 *pkcs7) | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 748 | { | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 749 | mbedtls_pkcs7_signer_info *signer_cur; | 
|  | 750 | mbedtls_pkcs7_signer_info *signer_prev; | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 751 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 752 | if (pkcs7 == NULL || pkcs7->raw.p == NULL) { | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 753 | return; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 754 | } | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 755 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 756 | mbedtls_free(pkcs7->raw.p); | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 757 |  | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 758 | mbedtls_x509_crt_free(&pkcs7->signed_data.certs); | 
|  | 759 | mbedtls_x509_crl_free(&pkcs7->signed_data.crl); | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 760 |  | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 761 | signer_cur = pkcs7->signed_data.signers.next; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 762 | pkcs7_free_signer_info(&pkcs7->signed_data.signers); | 
|  | 763 | while (signer_cur != NULL) { | 
| Daniel Axtens | 3538479 | 2020-09-02 14:48:45 +1000 | [diff] [blame] | 764 | signer_prev = signer_cur; | 
|  | 765 | signer_cur = signer_prev->next; | 
| Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 766 | pkcs7_free_signer_info(signer_prev); | 
|  | 767 | mbedtls_free(signer_prev); | 
| Nayna Jain | c9deb18 | 2020-11-16 19:03:12 +0000 | [diff] [blame] | 768 | } | 
|  | 769 |  | 
|  | 770 | pkcs7->raw.p = NULL; | 
|  | 771 | } | 
|  | 772 |  | 
|  | 773 | #endif |