commit 08fa26726004e5cb43516c1e2150d7429ee498bc
author Fabio Utzig <utzig@apache.org> Wed Sep 26 12:16:18 2018 -0300
committer Fabio Utzig <utzig@utzig.org> Thu Sep 27 10:52:33 2018 -0300
tree 8ff90f4f19f6f872e6843f3b9d6e22811ef83440
parent a1927f4d6bfe4c4d55717477543913ceb3d87412

Fix copy_done setting

This compliments the previous change on `image_ok` and also removes
`flash_area_erased_val` call that is not needed anymore.

Signed-off-by: Fabio Utzig <utzig@apache.org>

boot/bootutil/src/bootutil_misc.c

diff --git a/boot/bootutil/src/bootutil_misc.c b/boot/bootutil/src/bootutil_misc.c
index 9fcea4e..f4c5d2f 100644
--- a/boot/bootutil/src/bootutil_misc.c
+++ b/boot/bootutil/src/bootutil_misc.c
@@ -539,7 +539,6 @@
 {
     const struct flash_area *fap;
     struct boot_swap_state state_slot0;
-    uint8_t erased_val;
     int rc;
 
     rc = boot_read_swap_state_by_id(FLASH_AREA_IMAGE_0, &state_slot0);
@@ -567,8 +566,7 @@
         goto done;
     }
 
-    erased_val = flash_area_erased_val(fap);
-    if (state_slot0.copy_done == erased_val) {
+    if (state_slot0.copy_done == BOOT_FLAG_UNSET) {
         /* Swap never completed.  This is unexpected. */
         rc = BOOT_EBADVECT;
         goto done;