diff options
author | Masahisa Kojima <masahisa.kojima@linaro.org> | 2022-12-19 11:33:13 +0900 |
---|---|---|
committer | Heinrich Schuchardt <heinrich.schuchardt@canonical.com> | 2022-12-20 16:06:48 +0100 |
commit | 645502743ac10ffaa5abf944bf0b751b32b47220 (patch) | |
tree | 20f2d03a1cf5e5f145f7ed3c933a52b81f43a065 /lib | |
parent | ce3270849ba2e8449ca5e70d40cd752ba7a13b63 (diff) |
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>
Diffstat (limited to 'lib')
0 files changed, 0 insertions, 0 deletions