From afdf14348070a7df5f53a2103b56282903116de4 Mon Sep 17 00:00:00 2001 From: Tom Rini Date: Sun, 1 Jul 2018 22:10:33 -0400 Subject: [PATCH] Revert "fw_printenv: Don't bail out directly after one env read error" As pointed out by Wolfgang Denk, the problem with this fix is that while interactive users will see that we have found one part of the environment failed and are using the other, progmatic use will not see this and can lead to problems. Signed-off-by: Tom Rini --- tools/env/fw_env.c | 24 +++++++++--------------- 1 file changed, 9 insertions(+), 15 deletions(-) diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c index 3a5ad026f0..a5d75958e1 100644 --- a/tools/env/fw_env.c +++ b/tools/env/fw_env.c @@ -1427,21 +1427,14 @@ int fw_env_open(struct env_opts *opts) } dev_current = 0; - - if (!flash_io(O_RDONLY)) { - crc0 = crc32(0, (uint8_t *)environment.data, ENV_SIZE); - crc0_ok = (crc0 == *environment.crc); - } else if (have_redund_env) { - /* - * to give the redundant env a chance, maybe it's good: - * mark env crc0 invalid then test below if crc1 is ok - */ - crc0_ok = 0; - } else { + if (flash_io(O_RDONLY)) { ret = -EIO; goto open_cleanup; } + crc0 = crc32(0, (uint8_t *)environment.data, ENV_SIZE); + + crc0_ok = (crc0 == *environment.crc); if (!have_redund_env) { if (!crc0_ok) { fprintf(stderr, @@ -1469,10 +1462,8 @@ int fw_env_open(struct env_opts *opts) */ environment.image = addr1; if (flash_io(O_RDONLY)) { - crc1_ok = 0; - } else { - crc1 = crc32(0, (uint8_t *)redundant->data, ENV_SIZE); - crc1_ok = (crc1 == redundant->crc); + ret = -EIO; + goto open_cleanup; } /* Check flag scheme compatibility */ @@ -1498,6 +1489,9 @@ int fw_env_open(struct env_opts *opts) goto open_cleanup; } + crc1 = crc32(0, (uint8_t *)redundant->data, ENV_SIZE); + + crc1_ok = (crc1 == redundant->crc); flag1 = redundant->flags; if (crc0_ok && !crc1_ok) { -- 2.39.5