commit 0329f9c2ec6a1ab951ad06d12e7706d63f6f0d8f
author Shelley Chen <shchen@chromium.org> Mon Oct 09 11:04:30 2017 -0700
committer chrome-bot <chrome-bot@chromium.org> Wed Oct 18 06:15:06 2017 -0700
tree 71ef8f85b94604d57511df31c7e4adf13f21de56
parent 3b805725c15022783f0737a72b4f27962abf48cd

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 <shchen@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/707496
Reviewed-by: Randall Spangler <rspangler@chromium.org>

firmware/lib/vboot_ui_menu.c

diff --git a/firmware/lib/vboot_ui_menu.c b/firmware/lib/vboot_ui_menu.c
index 9d6b37d..e569292 100644
--- a/firmware/lib/vboot_ui_menu.c
+++ b/firmware/lib/vboot_ui_menu.c
@@ -1129,6 +1129,8 @@
 
 				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);