Boot: Synchronize MCUBoot with v1.4.0
Perform a partial synchronization of the MCUBoot code base in TF-M
with mainstream MCUBoot version 1.4.0. Hash of the source commit in the
original repository: ac55554059147fff718015be9f4bd3108123f50a.
Main changes:
- Add overflow-safe 32 and 16-bit add functions
Change-Id: I36f04bcc9c7fb9c7609bb45b621ece518da91ad0
Signed-off-by: David Vincze <david.vincze@arm.com>
diff --git a/bl2/ext/mcuboot/bootutil/src/bootutil_misc.c b/bl2/ext/mcuboot/bootutil/src/bootutil_misc.c
index e295fa8..4cddec3 100644
--- a/bl2/ext/mcuboot/bootutil/src/bootutil_misc.c
+++ b/bl2/ext/mcuboot/bootutil/src/bootutil_misc.c
@@ -20,7 +20,7 @@
/*
* Original code taken from mcuboot project at:
* https://github.com/JuulLabs-OSS/mcuboot
- * Git SHA of the original version: 510fddb8e06d76e2442b2a4603d3e1cbefe28be4
+ * Git SHA of the original version: ac55554059147fff718015be9f4bd3108123f50a
* Modifications are Copyright (c) 2019 Arm Limited.
*/
@@ -49,9 +49,6 @@
#define BOOT_MAGIC_ARR_SZ \
(sizeof boot_img_magic / sizeof boot_img_magic[0])
-const uint32_t BOOT_MAGIC_SZ = sizeof(boot_img_magic);
-const uint32_t BOOT_MAX_ALIGN = MAX_FLASH_ALIGN;
-
struct boot_swap_table {
uint8_t magic_primary_slot;
uint8_t magic_secondary_slot;
diff --git a/bl2/ext/mcuboot/bootutil/src/bootutil_priv.h b/bl2/ext/mcuboot/bootutil/src/bootutil_priv.h
index 5d5e19d..f2965e5 100644
--- a/bl2/ext/mcuboot/bootutil/src/bootutil_priv.h
+++ b/bl2/ext/mcuboot/bootutil/src/bootutil_priv.h
@@ -20,7 +20,7 @@
/*
* Original code taken from mcuboot project at:
* https://github.com/JuulLabs-OSS/mcuboot
- * Git SHA of the original version: 510fddb8e06d76e2442b2a4603d3e1cbefe28be4
+ * Git SHA of the original version: ac55554059147fff718015be9f4bd3108123f50a
* Modifications are Copyright (c) 2018-2019 Arm Limited.
*/
@@ -174,7 +174,7 @@
#define BOOT_STATUS_SOURCE_SCRATCH 1
#define BOOT_STATUS_SOURCE_PRIMARY_SLOT 2
-extern const uint32_t BOOT_MAGIC_SZ;
+#define BOOT_MAGIC_SZ (sizeof boot_img_magic)
/**
* Compatibility shim for flash sector type.
@@ -236,6 +236,41 @@
bool boot_add_uint32_overflow_check(uint32_t a, uint32_t b);
bool boot_add_uint16_overflow_check(uint16_t a, uint16_t b);
+/**
+ * Safe (non-overflowing) uint32_t addition. Returns true, and stores
+ * the result in *dest if it can be done without overflow. Otherwise,
+ * returns false.
+ */
+static inline bool boot_u32_safe_add(uint32_t *dest, uint32_t a, uint32_t b)
+{
+ /*
+ * "a + b <= UINT32_MAX", subtract 'b' from both sides to avoid
+ * the overflow.
+ */
+ if (a > UINT32_MAX - b) {
+ return false;
+ } else {
+ *dest = a + b;
+ return true;
+ }
+}
+
+/**
+ * Safe (non-overflowing) uint16_t addition. Returns true, and stores
+ * the result in *dest if it can be done without overflow. Otherwise,
+ * returns false.
+ */
+static inline bool boot_u16_safe_add(uint16_t *dest, uint16_t a, uint16_t b)
+{
+ uint32_t tmp = a + b;
+ if (tmp > UINT16_MAX) {
+ return false;
+ } else {
+ *dest = tmp;
+ return true;
+ }
+}
+
/*
* Accessors for the contents of struct boot_loader_state.
*/
diff --git a/bl2/ext/mcuboot/bootutil/src/image_rsa.c b/bl2/ext/mcuboot/bootutil/src/image_rsa.c
index 9aaf403..ea7c78f 100644
--- a/bl2/ext/mcuboot/bootutil/src/image_rsa.c
+++ b/bl2/ext/mcuboot/bootutil/src/image_rsa.c
@@ -20,7 +20,7 @@
/*
* Original code taken from mcuboot project at:
* https://github.com/JuulLabs-OSS/mcuboot
- * Git SHA of the original version: 510fddb8e06d76e2442b2a4603d3e1cbefe28be4
+ * Git SHA of the original version: ac55554059147fff718015be9f4bd3108123f50a
* Modifications are Copyright (c) 2018-2019 Arm Limited.
*/
diff --git a/bl2/ext/mcuboot/bootutil/src/image_validate.c b/bl2/ext/mcuboot/bootutil/src/image_validate.c
index 3b18010..18dd3e1 100644
--- a/bl2/ext/mcuboot/bootutil/src/image_validate.c
+++ b/bl2/ext/mcuboot/bootutil/src/image_validate.c
@@ -20,7 +20,7 @@
/*
* Original code taken from mcuboot project at:
* https://github.com/JuulLabs-OSS/mcuboot
- * Git SHA of the original version: 510fddb8e06d76e2442b2a4603d3e1cbefe28be4
+ * Git SHA of the original version: ac55554059147fff718015be9f4bd3108123f50a
* Modifications are Copyright (c) 2018-2019 Arm Limited.
*/
diff --git a/bl2/ext/mcuboot/bootutil/src/loader.c b/bl2/ext/mcuboot/bootutil/src/loader.c
index ab25e43..9387174 100644
--- a/bl2/ext/mcuboot/bootutil/src/loader.c
+++ b/bl2/ext/mcuboot/bootutil/src/loader.c
@@ -20,7 +20,7 @@
/*
* Original code taken from mcuboot project at:
* https://github.com/JuulLabs-OSS/mcuboot
- * Git SHA of the original version: 510fddb8e06d76e2442b2a4603d3e1cbefe28be4
+ * Git SHA of the original version: ac55554059147fff718015be9f4bd3108123f50a
* Modifications are Copyright (c) 2018-2019 Arm Limited.
*/