blob: fd4f4ce262715db369257c83dce86631e1cf42b0 [file] [log] [blame]
Christopher Collins92ea77f2016-12-12 15:59:26 -08001/*
David Brownaac71112020-02-03 16:13:42 -07002 * SPDX-License-Identifier: Apache-2.0
3 *
4 * Copyright (c) 2017-2019 Linaro LTD
5 * Copyright (c) 2016-2019 JUUL Labs
6 * Copyright (c) 2019-2020 Arm Limited
7 *
8 * Original license:
9 *
Christopher Collins92ea77f2016-12-12 15:59:26 -080010 * Licensed to the Apache Software Foundation (ASF) under one
11 * or more contributor license agreements. See the NOTICE file
12 * distributed with this work for additional information
13 * regarding copyright ownership. The ASF licenses this file
14 * to you under the Apache License, Version 2.0 (the
15 * "License"); you may not use this file except in compliance
16 * with the License. You may obtain a copy of the License at
17 *
18 * http://www.apache.org/licenses/LICENSE-2.0
19 *
20 * Unless required by applicable law or agreed to in writing,
21 * software distributed under the License is distributed on an
22 * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
23 * KIND, either express or implied. See the License for the
24 * specific language governing permissions and limitations
25 * under the License.
26 */
27
Christopher Collins92ea77f2016-12-12 15:59:26 -080028#include <stddef.h>
David Vinczec3084132020-02-18 14:50:47 +010029#include <stdint.h>
Christopher Collins92ea77f2016-12-12 15:59:26 -080030#include <inttypes.h>
31#include <string.h>
32
Andrzej Puzdrowskib788c712018-04-12 12:42:49 +020033#include <flash_map_backend/flash_map_backend.h>
34
Christopher Collins92ea77f2016-12-12 15:59:26 -080035#include "bootutil/image.h"
Blaž Hrastnik4f4833d2020-09-14 13:53:31 +090036#include "bootutil/crypto/sha256.h"
Christopher Collins92ea77f2016-12-12 15:59:26 -080037#include "bootutil/sign_key.h"
David Vinczec3084132020-02-18 14:50:47 +010038#include "bootutil/security_cnt.h"
Raef Colese8fe6cf2020-05-26 13:07:40 +010039#include "bootutil/fault_injection_hardening.h"
Christopher Collins92ea77f2016-12-12 15:59:26 -080040
Fabio Utzigba1fbe62017-07-21 14:01:20 -030041#include "mcuboot_config/mcuboot_config.h"
Fabio Utzigeed80b62017-06-10 08:03:05 -030042
Fabio Utzigba829042018-09-18 08:29:34 -030043#ifdef MCUBOOT_ENC_IMAGES
44#include "bootutil/enc_key.h"
45#endif
46#if defined(MCUBOOT_SIGN_RSA)
Christopher Collins92ea77f2016-12-12 15:59:26 -080047#include "mbedtls/rsa.h"
David Brownd7e350d2017-04-24 13:29:28 -060048#endif
Fabio Utzig19356bf2017-05-11 16:19:36 -030049#if defined(MCUBOOT_SIGN_EC) || defined(MCUBOOT_SIGN_EC256)
Christopher Collins92ea77f2016-12-12 15:59:26 -080050#include "mbedtls/ecdsa.h"
David Brownd7e350d2017-04-24 13:29:28 -060051#endif
Arvin Farahmandf8240192020-05-05 11:43:52 -040052#if defined(MCUBOOT_ENC_IMAGES) || defined(MCUBOOT_SIGN_RSA) || \
53 defined(MCUBOOT_SIGN_EC) || defined(MCUBOOT_SIGN_EC256)
Christopher Collins92ea77f2016-12-12 15:59:26 -080054#include "mbedtls/asn1.h"
Arvin Farahmandf8240192020-05-05 11:43:52 -040055#endif
Christopher Collins92ea77f2016-12-12 15:59:26 -080056
57#include "bootutil_priv.h"
58
59/*
60 * Compute SHA256 over the image.
61 */
62static int
Fabio Utzig10ee6482019-08-01 12:04:52 -030063bootutil_img_hash(struct enc_key_data *enc_state, int image_index,
64 struct image_header *hdr, const struct flash_area *fap,
65 uint8_t *tmp_buf, uint32_t tmp_buf_sz, uint8_t *hash_result,
66 uint8_t *seed, int seed_len)
Christopher Collins92ea77f2016-12-12 15:59:26 -080067{
David Browne629bf32017-04-24 13:37:33 -060068 bootutil_sha256_context sha256_ctx;
Christopher Collins92ea77f2016-12-12 15:59:26 -080069 uint32_t blk_sz;
70 uint32_t size;
Fabio Utzig2fc80df2018-12-14 06:47:38 -020071 uint16_t hdr_size;
Christopher Collins92ea77f2016-12-12 15:59:26 -080072 uint32_t off;
73 int rc;
Fabio Utzigba829042018-09-18 08:29:34 -030074 uint32_t blk_off;
Fabio Utzige52c08e2019-09-11 19:32:00 -030075 uint32_t tlv_off;
Christopher Collins92ea77f2016-12-12 15:59:26 -080076
Tamas Banfe031092020-09-10 17:32:39 +020077#if (BOOT_IMAGE_NUMBER == 1) || !defined(MCUBOOT_ENC_IMAGES) || \
78 defined(MCUBOOT_RAM_LOAD)
Fabio Utzig10ee6482019-08-01 12:04:52 -030079 (void)enc_state;
Fabio Utzigb0f04732019-07-31 09:49:19 -030080 (void)image_index;
Fabio Utzigc9621352019-08-08 12:15:51 -030081 (void)hdr_size;
Fabio Utzige52c08e2019-09-11 19:32:00 -030082 (void)blk_off;
83 (void)tlv_off;
Tamas Banfe031092020-09-10 17:32:39 +020084#ifdef MCUBOOT_RAM_LOAD
85 (void)blk_sz;
86 (void)off;
87 (void)rc;
David Brown122f9e72021-05-21 15:58:56 -060088 (void)fap;
89 (void)tmp_buf;
90 (void)tmp_buf_sz;
Tamas Banfe031092020-09-10 17:32:39 +020091#endif
Fabio Utzigb0f04732019-07-31 09:49:19 -030092#endif
93
Fabio Utzigbc077932019-08-26 11:16:34 -030094#ifdef MCUBOOT_ENC_IMAGES
95 /* Encrypted images only exist in the secondary slot */
96 if (MUST_DECRYPT(fap, image_index, hdr) &&
97 !boot_enc_valid(enc_state, image_index, fap)) {
98 return -1;
99 }
100#endif
101
David Browne629bf32017-04-24 13:37:33 -0600102 bootutil_sha256_init(&sha256_ctx);
Christopher Collins92ea77f2016-12-12 15:59:26 -0800103
104 /* in some cases (split image) the hash is seeded with data from
105 * the loader image */
Fabio Utzig53986042017-12-12 14:57:07 -0200106 if (seed && (seed_len > 0)) {
David Browne629bf32017-04-24 13:37:33 -0600107 bootutil_sha256_update(&sha256_ctx, seed, seed_len);
Christopher Collins92ea77f2016-12-12 15:59:26 -0800108 }
109
David Vinczee32483f2019-06-13 10:46:24 +0200110 /* Hash is computed over image header and image itself. */
Fabio Utzige52c08e2019-09-11 19:32:00 -0300111 size = hdr_size = hdr->ih_hdr_size;
112 size += hdr->ih_img_size;
113 tlv_off = size;
David Vinczee32483f2019-06-13 10:46:24 +0200114
Fabio Utzige52c08e2019-09-11 19:32:00 -0300115 /* If protected TLVs are present they are also hashed. */
116 size += hdr->ih_protect_tlv_size;
David Vinczee32483f2019-06-13 10:46:24 +0200117
Tamas Banfe031092020-09-10 17:32:39 +0200118#ifdef MCUBOOT_RAM_LOAD
David Brown9bd7f902021-05-26 16:31:14 -0600119 bootutil_sha256_update(&sha256_ctx,
120 (void*)(IMAGE_RAM_BASE + hdr->ih_load_addr),
121 size);
Tamas Banfe031092020-09-10 17:32:39 +0200122#else
Christopher Collins92ea77f2016-12-12 15:59:26 -0800123 for (off = 0; off < size; off += blk_sz) {
124 blk_sz = size - off;
Fabio Utzige52c08e2019-09-11 19:32:00 -0300125 if (blk_sz > tmp_buf_sz) {
126 blk_sz = tmp_buf_sz;
127 }
Fabio Utzigc21c2102019-09-10 10:10:42 -0300128#ifdef MCUBOOT_ENC_IMAGES
129 /* The only data that is encrypted in an image is the payload;
130 * both header and TLVs (when protected) are not.
131 */
132 if ((off < hdr_size) && ((off + blk_sz) > hdr_size)) {
133 /* read only the header */
134 blk_sz = hdr_size - off;
Fabio Utzige52c08e2019-09-11 19:32:00 -0300135 }
136 if ((off < tlv_off) && ((off + blk_sz) > tlv_off)) {
137 /* read only up to the end of the image payload */
138 blk_sz = tlv_off - off;
Fabio Utzigc21c2102019-09-10 10:10:42 -0300139 }
140#endif
Christopher Collins92ea77f2016-12-12 15:59:26 -0800141 rc = flash_area_read(fap, off, tmp_buf, blk_sz);
142 if (rc) {
Blaž Hrastnik4f4833d2020-09-14 13:53:31 +0900143 bootutil_sha256_drop(&sha256_ctx);
Christopher Collins92ea77f2016-12-12 15:59:26 -0800144 return rc;
145 }
Fabio Utzigba829042018-09-18 08:29:34 -0300146#ifdef MCUBOOT_ENC_IMAGES
Fabio Utzigc21c2102019-09-10 10:10:42 -0300147 if (MUST_DECRYPT(fap, image_index, hdr)) {
Fabio Utzige52c08e2019-09-11 19:32:00 -0300148 /* Only payload is encrypted (area between header and TLVs) */
149 if (off >= hdr_size && off < tlv_off) {
Fabio Utzigc21c2102019-09-10 10:10:42 -0300150 blk_off = (off - hdr_size) & 0xf;
151 boot_encrypt(enc_state, image_index, fap, off - hdr_size,
152 blk_sz, blk_off, tmp_buf);
153 }
Fabio Utzigba829042018-09-18 08:29:34 -0300154 }
155#endif
David Browne629bf32017-04-24 13:37:33 -0600156 bootutil_sha256_update(&sha256_ctx, tmp_buf, blk_sz);
Christopher Collins92ea77f2016-12-12 15:59:26 -0800157 }
Tamas Banfe031092020-09-10 17:32:39 +0200158#endif /* MCUBOOT_RAM_LOAD */
David Browne629bf32017-04-24 13:37:33 -0600159 bootutil_sha256_finish(&sha256_ctx, hash_result);
Blaž Hrastnik4f4833d2020-09-14 13:53:31 +0900160 bootutil_sha256_drop(&sha256_ctx);
Christopher Collins92ea77f2016-12-12 15:59:26 -0800161
162 return 0;
163}
164
165/*
David Brown43cda332017-09-01 09:53:23 -0600166 * Currently, we only support being able to verify one type of
167 * signature, because there is a single verification function that we
168 * call. List the type of TLV we are expecting. If we aren't
169 * configured for any signature, don't define this macro.
170 */
Fabio Utzig48764842019-05-10 19:28:24 -0300171#if (defined(MCUBOOT_SIGN_RSA) + \
172 defined(MCUBOOT_SIGN_EC) + \
173 defined(MCUBOOT_SIGN_EC256) + \
174 defined(MCUBOOT_SIGN_ED25519)) > 1
Fabio Utzig3501c012019-05-13 15:07:25 -0700175#error "Only a single signature type is supported!"
176#endif
177
David Brown43cda332017-09-01 09:53:23 -0600178#if defined(MCUBOOT_SIGN_RSA)
Fabio Utzig3501c012019-05-13 15:07:25 -0700179# if MCUBOOT_SIGN_RSA_LEN == 2048
180# define EXPECTED_SIG_TLV IMAGE_TLV_RSA2048_PSS
181# elif MCUBOOT_SIGN_RSA_LEN == 3072
182# define EXPECTED_SIG_TLV IMAGE_TLV_RSA3072_PSS
183# else
184# error "Unsupported RSA signature length"
David Brown43cda332017-09-01 09:53:23 -0600185# endif
Fabio Utzig3501c012019-05-13 15:07:25 -0700186# define SIG_BUF_SIZE (MCUBOOT_SIGN_RSA_LEN / 8)
187# define EXPECTED_SIG_LEN(x) ((x) == SIG_BUF_SIZE) /* 2048 bits */
David Brown43cda332017-09-01 09:53:23 -0600188#elif defined(MCUBOOT_SIGN_EC)
189# define EXPECTED_SIG_TLV IMAGE_TLV_ECDSA224
Fabio Utzig3501c012019-05-13 15:07:25 -0700190# define SIG_BUF_SIZE 128
David Browna3608262019-12-12 15:35:31 -0700191# define EXPECTED_SIG_LEN(x) (1) /* always true, ASN.1 will validate */
David Brown43cda332017-09-01 09:53:23 -0600192#elif defined(MCUBOOT_SIGN_EC256)
193# define EXPECTED_SIG_TLV IMAGE_TLV_ECDSA256
Fabio Utzig3501c012019-05-13 15:07:25 -0700194# define SIG_BUF_SIZE 128
David Browna3608262019-12-12 15:35:31 -0700195# define EXPECTED_SIG_LEN(x) (1) /* always true, ASN.1 will validate */
Fabio Utzig48764842019-05-10 19:28:24 -0300196#elif defined(MCUBOOT_SIGN_ED25519)
197# define EXPECTED_SIG_TLV IMAGE_TLV_ED25519
198# define SIG_BUF_SIZE 64
199# define EXPECTED_SIG_LEN(x) ((x) == SIG_BUF_SIZE)
Fabio Utzig3501c012019-05-13 15:07:25 -0700200#else
201# define SIG_BUF_SIZE 32 /* no signing, sha256 digest only */
David Brown43cda332017-09-01 09:53:23 -0600202#endif
203
204#ifdef EXPECTED_SIG_TLV
David Vincze03368b82020-04-01 12:53:53 +0200205#if !defined(MCUBOOT_HW_KEY)
David Brown43cda332017-09-01 09:53:23 -0600206static int
207bootutil_find_key(uint8_t *keyhash, uint8_t keyhash_len)
208{
209 bootutil_sha256_context sha256_ctx;
210 int i;
211 const struct bootutil_key *key;
212 uint8_t hash[32];
213
Fabio Utzig15c14672019-08-09 07:45:20 -0300214 if (keyhash_len > 32) {
215 return -1;
216 }
David Brown43cda332017-09-01 09:53:23 -0600217
218 for (i = 0; i < bootutil_key_cnt; i++) {
219 key = &bootutil_keys[i];
220 bootutil_sha256_init(&sha256_ctx);
221 bootutil_sha256_update(&sha256_ctx, key->key, *key->len);
222 bootutil_sha256_finish(&sha256_ctx, hash);
223 if (!memcmp(hash, keyhash, keyhash_len)) {
Blaž Hrastnik4f4833d2020-09-14 13:53:31 +0900224 bootutil_sha256_drop(&sha256_ctx);
David Brown43cda332017-09-01 09:53:23 -0600225 return i;
226 }
227 }
Blaž Hrastnik4f4833d2020-09-14 13:53:31 +0900228 bootutil_sha256_drop(&sha256_ctx);
David Brown43cda332017-09-01 09:53:23 -0600229 return -1;
230}
David Vincze03368b82020-04-01 12:53:53 +0200231#else
232extern unsigned int pub_key_len;
233static int
234bootutil_find_key(uint8_t image_index, uint8_t *key, uint16_t key_len)
235{
236 bootutil_sha256_context sha256_ctx;
237 uint8_t hash[32];
238 uint8_t key_hash[32];
239 size_t key_hash_size = sizeof(key_hash);
240 int rc;
Raef Colese8fe6cf2020-05-26 13:07:40 +0100241 fih_int fih_rc;
David Vincze03368b82020-04-01 12:53:53 +0200242
243 bootutil_sha256_init(&sha256_ctx);
244 bootutil_sha256_update(&sha256_ctx, key, key_len);
245 bootutil_sha256_finish(&sha256_ctx, hash);
Blaž Hrastnik4f4833d2020-09-14 13:53:31 +0900246 bootutil_sha256_drop(&sha256_ctx);
David Vincze03368b82020-04-01 12:53:53 +0200247
248 rc = boot_retrieve_public_key_hash(image_index, key_hash, &key_hash_size);
249 if (rc) {
Mark Horvath7ebf0e82021-02-19 09:55:35 +0100250 return -1;
David Vincze03368b82020-04-01 12:53:53 +0200251 }
252
Raef Colese8fe6cf2020-05-26 13:07:40 +0100253 /* Adding hardening to avoid this potential attack:
254 * - Image is signed with an arbitrary key and the corresponding public
255 * key is added as a TLV field.
256 * - During public key validation (comparing against key-hash read from
257 * HW) a fault is injected to accept the public key as valid one.
258 */
259 FIH_CALL(boot_fih_memequal, fih_rc, hash, key_hash, key_hash_size);
260 if (fih_eq(fih_rc, FIH_SUCCESS)) {
David Vincze03368b82020-04-01 12:53:53 +0200261 bootutil_keys[0].key = key;
262 pub_key_len = key_len;
263 return 0;
264 }
Raef Colese8fe6cf2020-05-26 13:07:40 +0100265
David Vincze03368b82020-04-01 12:53:53 +0200266 return -1;
267}
268#endif /* !MCUBOOT_HW_KEY */
David Brown43cda332017-09-01 09:53:23 -0600269#endif
270
David Vinczec3084132020-02-18 14:50:47 +0100271/**
272 * Reads the value of an image's security counter.
273 *
274 * @param hdr Pointer to the image header structure.
275 * @param fap Pointer to a description structure of the image's
276 * flash area.
277 * @param security_cnt Pointer to store the security counter value.
278 *
279 * @return 0 on success; nonzero on failure.
280 */
281int32_t
282bootutil_get_img_security_cnt(struct image_header *hdr,
283 const struct flash_area *fap,
284 uint32_t *img_security_cnt)
285{
286 struct image_tlv_iter it;
287 uint32_t off;
288 uint16_t len;
289 int32_t rc;
290
291 if ((hdr == NULL) ||
292 (fap == NULL) ||
293 (img_security_cnt == NULL)) {
294 /* Invalid parameter. */
295 return BOOT_EBADARGS;
296 }
297
298 /* The security counter TLV is in the protected part of the TLV area. */
299 if (hdr->ih_protect_tlv_size == 0) {
300 return BOOT_EBADIMAGE;
301 }
302
303 rc = bootutil_tlv_iter_begin(&it, hdr, fap, IMAGE_TLV_SEC_CNT, true);
304 if (rc) {
305 return rc;
306 }
307
308 /* Traverse through the protected TLV area to find
309 * the security counter TLV.
310 */
311
312 rc = bootutil_tlv_iter_next(&it, &off, &len, NULL);
313 if (rc != 0) {
314 /* Security counter TLV has not been found. */
315 return -1;
316 }
317
318 if (len != sizeof(*img_security_cnt)) {
319 /* Security counter is not valid. */
320 return BOOT_EBADIMAGE;
321 }
322
Tamas Banfe031092020-09-10 17:32:39 +0200323 rc = LOAD_IMAGE_DATA(hdr, fap, off, img_security_cnt, len);
David Vinczec3084132020-02-18 14:50:47 +0100324 if (rc != 0) {
325 return BOOT_EFLASH;
326 }
327
328 return 0;
329}
David Vinczec3084132020-02-18 14:50:47 +0100330
David Brown43cda332017-09-01 09:53:23 -0600331/*
Christopher Collins92ea77f2016-12-12 15:59:26 -0800332 * Verify the integrity of the image.
333 * Return non-zero if image could not be validated/does not validate.
334 */
Raef Colese8fe6cf2020-05-26 13:07:40 +0100335fih_int
Fabio Utzig10ee6482019-08-01 12:04:52 -0300336bootutil_img_validate(struct enc_key_data *enc_state, int image_index,
337 struct image_header *hdr, const struct flash_area *fap,
338 uint8_t *tmp_buf, uint32_t tmp_buf_sz, uint8_t *seed,
339 int seed_len, uint8_t *out_hash)
Christopher Collins92ea77f2016-12-12 15:59:26 -0800340{
341 uint32_t off;
Fabio Utzig61fd8882019-09-14 20:00:20 -0300342 uint16_t len;
David Brownd13318a2019-12-04 17:28:40 -0700343 uint16_t type;
David Brown43cda332017-09-01 09:53:23 -0600344 int sha256_valid = 0;
345#ifdef EXPECTED_SIG_TLV
Raef Colese8fe6cf2020-05-26 13:07:40 +0100346 fih_int valid_signature = FIH_FAILURE;
David Brown43cda332017-09-01 09:53:23 -0600347 int key_id = -1;
David Vincze03368b82020-04-01 12:53:53 +0200348#ifdef MCUBOOT_HW_KEY
349 /* Few extra bytes for encoding and for public exponent. */
350 uint8_t key_buf[SIG_BUF_SIZE + 24];
Christopher Collins92ea77f2016-12-12 15:59:26 -0800351#endif
David Vincze03368b82020-04-01 12:53:53 +0200352#endif /* EXPECTED_SIG_TLV */
Fabio Utzig61fd8882019-09-14 20:00:20 -0300353 struct image_tlv_iter it;
Fabio Utzig3501c012019-05-13 15:07:25 -0700354 uint8_t buf[SIG_BUF_SIZE];
Christopher Collins92ea77f2016-12-12 15:59:26 -0800355 uint8_t hash[32];
Raef Colese8fe6cf2020-05-26 13:07:40 +0100356 int rc = 0;
357 fih_int fih_rc = FIH_FAILURE;
David Vinczec3084132020-02-18 14:50:47 +0100358#ifdef MCUBOOT_HW_ROLLBACK_PROT
Raef Colese8fe6cf2020-05-26 13:07:40 +0100359 fih_int security_cnt = fih_int_encode(INT_MAX);
David Vinczec3084132020-02-18 14:50:47 +0100360 uint32_t img_security_cnt = 0;
Raef Colese8fe6cf2020-05-26 13:07:40 +0100361 fih_int security_counter_valid = FIH_FAILURE;
David Vinczec3084132020-02-18 14:50:47 +0100362#endif
Christopher Collins92ea77f2016-12-12 15:59:26 -0800363
Fabio Utzig10ee6482019-08-01 12:04:52 -0300364 rc = bootutil_img_hash(enc_state, image_index, hdr, fap, tmp_buf,
365 tmp_buf_sz, hash, seed, seed_len);
Christopher Collins92ea77f2016-12-12 15:59:26 -0800366 if (rc) {
Raef Colese8fe6cf2020-05-26 13:07:40 +0100367 goto out;
Christopher Collins92ea77f2016-12-12 15:59:26 -0800368 }
369
370 if (out_hash) {
371 memcpy(out_hash, hash, 32);
372 }
373
Fabio Utzig61fd8882019-09-14 20:00:20 -0300374 rc = bootutil_tlv_iter_begin(&it, hdr, fap, IMAGE_TLV_ANY, false);
David Brownf5b33d82017-09-01 10:58:27 -0600375 if (rc) {
Raef Colese8fe6cf2020-05-26 13:07:40 +0100376 goto out;
David Brownf5b33d82017-09-01 10:58:27 -0600377 }
Christopher Collins92ea77f2016-12-12 15:59:26 -0800378
Andrzej Puzdrowski334b6a62022-01-21 16:59:18 +0100379 if (it.tlv_end > bootutil_max_image_size(fap)) {
380 rc = -1;
381 goto out;
382 }
383
David Brown43cda332017-09-01 09:53:23 -0600384 /*
385 * Traverse through all of the TLVs, performing any checks we know
386 * and are able to do.
387 */
Fabio Utzig61fd8882019-09-14 20:00:20 -0300388 while (true) {
389 rc = bootutil_tlv_iter_next(&it, &off, &len, &type);
390 if (rc < 0) {
Raef Colese8fe6cf2020-05-26 13:07:40 +0100391 goto out;
Fabio Utzig61fd8882019-09-14 20:00:20 -0300392 } else if (rc > 0) {
393 break;
Christopher Collins92ea77f2016-12-12 15:59:26 -0800394 }
David Brown43cda332017-09-01 09:53:23 -0600395
Fabio Utzig61fd8882019-09-14 20:00:20 -0300396 if (type == IMAGE_TLV_SHA256) {
David Brown43cda332017-09-01 09:53:23 -0600397 /*
398 * Verify the SHA256 image hash. This must always be
399 * present.
400 */
Fabio Utzig61fd8882019-09-14 20:00:20 -0300401 if (len != sizeof(hash)) {
Raef Colese8fe6cf2020-05-26 13:07:40 +0100402 rc = -1;
403 goto out;
Christopher Collins92ea77f2016-12-12 15:59:26 -0800404 }
Tamas Banfe031092020-09-10 17:32:39 +0200405 rc = LOAD_IMAGE_DATA(hdr, fap, off, buf, sizeof(hash));
David Brown43cda332017-09-01 09:53:23 -0600406 if (rc) {
Raef Colese8fe6cf2020-05-26 13:07:40 +0100407 goto out;
Christopher Collins92ea77f2016-12-12 15:59:26 -0800408 }
Raef Colese8fe6cf2020-05-26 13:07:40 +0100409
410 FIH_CALL(boot_fih_memequal, fih_rc, hash, buf, sizeof(hash));
411 if (fih_not_eq(fih_rc, FIH_SUCCESS)) {
412 goto out;
Christopher Collins92ea77f2016-12-12 15:59:26 -0800413 }
David Brown43cda332017-09-01 09:53:23 -0600414
415 sha256_valid = 1;
416#ifdef EXPECTED_SIG_TLV
David Vincze03368b82020-04-01 12:53:53 +0200417#ifndef MCUBOOT_HW_KEY
Fabio Utzig61fd8882019-09-14 20:00:20 -0300418 } else if (type == IMAGE_TLV_KEYHASH) {
Christopher Collins92ea77f2016-12-12 15:59:26 -0800419 /*
David Brown43cda332017-09-01 09:53:23 -0600420 * Determine which key we should be checking.
Christopher Collins92ea77f2016-12-12 15:59:26 -0800421 */
Fabio Utzig61fd8882019-09-14 20:00:20 -0300422 if (len > 32) {
Raef Colese8fe6cf2020-05-26 13:07:40 +0100423 rc = -1;
424 goto out;
David Brown43cda332017-09-01 09:53:23 -0600425 }
Tamas Banfe031092020-09-10 17:32:39 +0200426 rc = LOAD_IMAGE_DATA(hdr, fap, off, buf, len);
David Brown43cda332017-09-01 09:53:23 -0600427 if (rc) {
Raef Colese8fe6cf2020-05-26 13:07:40 +0100428 goto out;
David Brown43cda332017-09-01 09:53:23 -0600429 }
Fabio Utzig61fd8882019-09-14 20:00:20 -0300430 key_id = bootutil_find_key(buf, len);
David Brown43cda332017-09-01 09:53:23 -0600431 /*
432 * The key may not be found, which is acceptable. There
433 * can be multiple signatures, each preceded by a key.
434 */
David Vincze03368b82020-04-01 12:53:53 +0200435#else
436 } else if (type == IMAGE_TLV_PUBKEY) {
437 /*
438 * Determine which key we should be checking.
439 */
440 if (len > sizeof(key_buf)) {
Raef Colese8fe6cf2020-05-26 13:07:40 +0100441 rc = -1;
442 goto out;
David Vincze03368b82020-04-01 12:53:53 +0200443 }
Tamas Banfe031092020-09-10 17:32:39 +0200444 rc = LOAD_IMAGE_DATA(hdr, fap, off, key_buf, len);
David Vincze03368b82020-04-01 12:53:53 +0200445 if (rc) {
Raef Colese8fe6cf2020-05-26 13:07:40 +0100446 goto out;
David Vincze03368b82020-04-01 12:53:53 +0200447 }
448 key_id = bootutil_find_key(image_index, key_buf, len);
449 /*
450 * The key may not be found, which is acceptable. There
451 * can be multiple signatures, each preceded by a key.
452 */
453#endif /* !MCUBOOT_HW_KEY */
Fabio Utzig61fd8882019-09-14 20:00:20 -0300454 } else if (type == EXPECTED_SIG_TLV) {
David Brown43cda332017-09-01 09:53:23 -0600455 /* Ignore this signature if it is out of bounds. */
456 if (key_id < 0 || key_id >= bootutil_key_cnt) {
457 key_id = -1;
458 continue;
459 }
Fabio Utzig61fd8882019-09-14 20:00:20 -0300460 if (!EXPECTED_SIG_LEN(len) || len > sizeof(buf)) {
Raef Colese8fe6cf2020-05-26 13:07:40 +0100461 rc = -1;
462 goto out;
David Brown43cda332017-09-01 09:53:23 -0600463 }
Tamas Banfe031092020-09-10 17:32:39 +0200464 rc = LOAD_IMAGE_DATA(hdr, fap, off, buf, len);
David Brown43cda332017-09-01 09:53:23 -0600465 if (rc) {
Raef Colese8fe6cf2020-05-26 13:07:40 +0100466 goto out;
David Brown43cda332017-09-01 09:53:23 -0600467 }
Raef Colese8fe6cf2020-05-26 13:07:40 +0100468 FIH_CALL(bootutil_verify_sig, valid_signature, hash, sizeof(hash),
469 buf, len, key_id);
David Brown43cda332017-09-01 09:53:23 -0600470 key_id = -1;
David Vinczec3084132020-02-18 14:50:47 +0100471#endif /* EXPECTED_SIG_TLV */
472#ifdef MCUBOOT_HW_ROLLBACK_PROT
473 } else if (type == IMAGE_TLV_SEC_CNT) {
474 /*
475 * Verify the image's security counter.
476 * This must always be present.
477 */
478 if (len != sizeof(img_security_cnt)) {
479 /* Security counter is not valid. */
Raef Colese8fe6cf2020-05-26 13:07:40 +0100480 rc = -1;
481 goto out;
David Vinczec3084132020-02-18 14:50:47 +0100482 }
483
Tamas Banfe031092020-09-10 17:32:39 +0200484 rc = LOAD_IMAGE_DATA(hdr, fap, off, &img_security_cnt, len);
David Vinczec3084132020-02-18 14:50:47 +0100485 if (rc) {
Raef Colese8fe6cf2020-05-26 13:07:40 +0100486 goto out;
David Vinczec3084132020-02-18 14:50:47 +0100487 }
488
Raef Colese8fe6cf2020-05-26 13:07:40 +0100489 FIH_CALL(boot_nv_security_counter_get, fih_rc, image_index,
490 &security_cnt);
491 if (fih_not_eq(fih_rc, FIH_SUCCESS)) {
492 goto out;
David Vinczec3084132020-02-18 14:50:47 +0100493 }
494
495 /* Compare the new image's security counter value against the
496 * stored security counter value.
497 */
Raef Colese8fe6cf2020-05-26 13:07:40 +0100498 fih_rc = fih_int_encode_zero_equality(img_security_cnt <
499 fih_int_decode(security_cnt));
500 if (fih_not_eq(fih_rc, FIH_SUCCESS)) {
501 goto out;
David Vinczec3084132020-02-18 14:50:47 +0100502 }
503
504 /* The image's security counter has been successfully verified. */
Raef Colese8fe6cf2020-05-26 13:07:40 +0100505 security_counter_valid = fih_rc;
David Vinczec3084132020-02-18 14:50:47 +0100506#endif /* MCUBOOT_HW_ROLLBACK_PROT */
Christopher Collins92ea77f2016-12-12 15:59:26 -0800507 }
508 }
David Brown43cda332017-09-01 09:53:23 -0600509
Raef Colese8fe6cf2020-05-26 13:07:40 +0100510 rc = !sha256_valid;
511 if (rc) {
512 goto out;
513 }
David Brown43cda332017-09-01 09:53:23 -0600514#ifdef EXPECTED_SIG_TLV
Raef Colese8fe6cf2020-05-26 13:07:40 +0100515 fih_rc = fih_int_encode_zero_equality(fih_not_eq(valid_signature,
516 FIH_SUCCESS));
Christopher Collins92ea77f2016-12-12 15:59:26 -0800517#endif
David Vinczec3084132020-02-18 14:50:47 +0100518#ifdef MCUBOOT_HW_ROLLBACK_PROT
Raef Colese8fe6cf2020-05-26 13:07:40 +0100519 if (fih_not_eq(security_counter_valid, FIH_SUCCESS)) {
520 rc = -1;
521 goto out;
522 }
David Vinczec3084132020-02-18 14:50:47 +0100523#endif
Raef Colese8fe6cf2020-05-26 13:07:40 +0100524
525out:
526 if (rc) {
527 fih_rc = fih_int_encode(rc);
David Vinczec3084132020-02-18 14:50:47 +0100528 }
David Brown43cda332017-09-01 09:53:23 -0600529
Raef Colese8fe6cf2020-05-26 13:07:40 +0100530 FIH_RET(fih_rc);
Christopher Collins92ea77f2016-12-12 15:59:26 -0800531}