From 2ca654f03ab3039d21e628388285e5e9670c601f Mon Sep 17 00:00:00 2001 From: Shelley Chen Date: Thu, 14 Sep 2017 14:12:21 -0700 Subject: [PATCH] detachables: fix setting selected_idx of DEV_WARNING menu Selecting cancel from DEV_MENU should not set the selected_idx to loc. It should be set to power off. BUG=b:35585623 BRANCH=None TEST=go to DEV_MENU, hit cancel and make sure that power off is selected item Change-Id: I8cae397c7174b5bd52a3a27736cd3d5a57412e63 Signed-off-by: Shelley Chen Reviewed-on: https://chromium-review.googlesource.com/667933 Reviewed-by: Julius Werner --- firmware/lib/vboot_ui_menu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/firmware/lib/vboot_ui_menu.c b/firmware/lib/vboot_ui_menu.c index 746bfe13f1..6f43f151c3 100644 --- a/firmware/lib/vboot_ui_menu.c +++ b/firmware/lib/vboot_ui_menu.c @@ -479,7 +479,7 @@ VbError_t vb2_update_menu(struct vb2_context *ctx) * 2. Default to power off option. */ vb2_set_menu_items(VB_MENU_DEV_WARNING, - loc); + VB_WARN_POWER_OFF); break; case VB_DEV_POWER_OFF: /* Power off */