tree 20f2d03a1cf5e5f145f7ed3c933a52b81f43a065
parent 7ec3c6f031edb998bdb1d3ec9a712d7f5793be0d
author Masahisa Kojima <masahisa.kojima@linaro.org> 1671417193 +0900
committer Heinrich Schuchardt <heinrich.schuchardt@canonical.com> 1671548808 +0100

eficonfig: avoid SetVariable between GetNextVariableName calls

The current code calls efi_set_variable_int() to delete the
invalid boot option between calls to efi_get_next_variable_name_int(),
it may produce unpredictable results.

This commit moves removal of the invalid boot option outside
of the efi_get_next_variable_name_int() calls.
EFI_NOT_FOUND returned from efi_get_next_variable_name_int()
indicates we retrieved all EFI variables, it should be treated
as EFI_SUCEESS.

To address the checkpatch warning of too many leading tabs,
combine two if statement into one.

Signed-off-by: Masahisa Kojima <masahisa.kojima@linaro.org>
Reviewed-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>