diff options
| author | Mary Ruthven <mruthven@chromium.org> | 2021-06-10 14:36:15 -0500 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-07-08 19:29:36 +0000 |
| commit | 33fd5ea1b399196160c03b255d38852b6199eb6e (patch) | |
| tree | 0a12c8be0f3133a9fcb51799706d257ae296a03b | |
| parent | 76c8d5d0f355998c278c520990b096d238639f2c (diff) | |
| download | chrome-ec-33fd5ea1b399196160c03b255d38852b6199eb6e.tar.gz | |
ap_ro_integrity_check: handle invalid hash responses the same way
Process FLASH_READ_FAILED the same as the rest of the errors. Put the
actual code in response and return INTERNAL_ERROR.
BUG=none
TEST=none
Change-Id: Iacbdc2db01c54da57cac28ccbdfd3210be95dc62
Signed-off-by: Mary Ruthven <mruthven@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2954320
Reviewed-by: Vadim Bendebury <vbendeb@chromium.org>
(cherry picked from commit 4547342e44b9e3e61dd9f52e532d831599e61ba9)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3004082
Reviewed-by: Andrey Pronin <apronin@chromium.org>
| -rw-r--r-- | common/ap_ro_integrity_check.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/common/ap_ro_integrity_check.c b/common/ap_ro_integrity_check.c index 584c9ca36d..f6592d1ca9 100644 --- a/common/ap_ro_integrity_check.c +++ b/common/ap_ro_integrity_check.c @@ -330,8 +330,6 @@ static enum vendor_cmd_rc vc_get_ap_ro_hash(enum vendor_cmd_cc code, return VENDOR_RC_BOGUS_ARGS; rv = ap_ro_check_unsupported(false); - if (rv == ARCVE_FLASH_READ_FAILED) - return VENDOR_RC_READ_FLASH_FAIL; if (rv) { *response_size = 1; *response = rv; |