From 0329f9c2ec6a1ab951ad06d12e7706d63f6f0d8f Mon Sep 17 00:00:00 2001 From: Shelley Chen Date: Mon, 9 Oct 2017 11:04:30 -0700 Subject: [PATCH] vboot_ui_menu: Checking vb2_update_menu return value If return value is not VBERROR_SUCCESS, return with an error. BUG=None BRANCH=None TEST=None Change-Id: I1402277567b901701cdd68d74968e43e838c89da Signed-off-by: Shelley Chen Reviewed-on: https://chromium-review.googlesource.com/707496 Reviewed-by: Randall Spangler --- firmware/lib/vboot_ui_menu.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/firmware/lib/vboot_ui_menu.c b/firmware/lib/vboot_ui_menu.c index 9d6b37d854..e5692926fd 100644 --- a/firmware/lib/vboot_ui_menu.c +++ b/firmware/lib/vboot_ui_menu.c @@ -1129,6 +1129,8 @@ static VbError_t recovery_ui(struct vb2_context *ctx, VbCommonParams *cparams) if (current_menu == VB_MENU_RECOVERY_INSERT) { ret = vb2_update_menu(ctx); + if (ret != VBERROR_SUCCESS) + return ret; vb2_set_disabled_idx_mask(shared->flags); } else { vb2_update_selection(cparams, key);