commit 2adbed8d6d19b9f00a3e4ca38bae53e0c5e7b1e3
author Andrzej Puzdrowski <andrzej.puzdrowski@nordicsemi.no> Tue Nov 14 12:34:11 2017 +0100
committer Anas Nashif <nashif@linux.intel.com> Tue Nov 14 06:58:54 2017 -0500
tree abec9c3f7b691e17029c9a8b7aff9fe9a11ab6b8
parent 1d70a39d1b14fabc933685e08f9c0746221905d5

tests: subsys: dfu: mcuboot: fix return value not handling issue

Function flash_read was calling without checking return value.
(Coverity CID: 178794)
This patch cover this issue.

Signed-off-by: Andrzej Puzdrowski <andrzej.puzdrowski@nordicsemi.no>

tests/subsys/dfu/mcuboot/src/main.c

diff --git a/tests/subsys/dfu/mcuboot/src/main.c b/tests/subsys/dfu/mcuboot/src/main.c
index 576d2d7..befebfe 100644
--- a/tests/subsys/dfu/mcuboot/src/main.c
+++ b/tests/subsys/dfu/mcuboot/src/main.c
@@ -28,7 +28,8 @@
 	for (offs = FLASH_AREA_IMAGE_1_OFFSET;
 	     offs <= FLASH_AREA_IMAGE_1_OFFSET + FLASH_AREA_IMAGE_1_SIZE;
 	     offs += sizeof(temp)) {
-		flash_read(flash_dev, offs, &temp, sizeof(temp));
+		ret = flash_read(flash_dev, offs, &temp, sizeof(temp));
+		zassert_true(ret == 0, "Reading from flash");
 		if (temp != 0xFFFFFFFF) {
 			ret = flash_write_protection_set(flash_dev, false);
 			zassert_true(ret == 0, "Disabling flash protection");