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