vboot: always return after calling vb2_ui_screen_change

Without returning, subsequent code may operate under the
assumption that the screen has *not* changed, leading to
unexpected behaviour.  The user may also be able to select
otherwise disallowed menu items.

BUG=b:181087237, chromium:1181484
TEST=make clean && make runtests

Signed-off-by: Joel Kitching <kitching@google.com>
Change-Id: I820e387417ad39e2f7bd47f65d08c387cf66d6e5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2717449
Tested-by: Joel Kitching <kitching@chromium.org>
Reviewed-by: Yu-Ping Wu <yupingso@chromium.org>
Commit-Queue: Joel Kitching <kitching@chromium.org>
2 files changed