u-boot.git/lib, branch v2021.01

efi_loader: use after free in efi_exit()

2020-12-29T01:09:04+00:00

Do not use data from the loaded image object after deleting it.

Fixes: 126a43f15b36 ("efi_loader: unload applications upon Exit()")
Signed-off-by: Heinrich Schuchardt

efi_loader: efi_signal_event() fix comment typos

2020-12-29T01:09:04+00:00

Add missing commas.

Signed-off-by: Heinrich Schuchardt

efi_loader: avoid invalid free

2020-12-29T01:09:04+00:00

load_options passed from do_efibootmgr() to do_bootefi_exec() may contain
invalid data from the stack which will lead to an invalid free().

Fixes: 0ad64007feb9 ("efi_loader: set load options in boot manager")
Signed-off-by: Heinrich Schuchardt

efi_loader: escape key handling

2020-12-29T01:09:04+00:00

Up to now the escape key was not correctly detected in UEFI applications.
We had to hit it twice for a single escape to be recognized.

Use a 10 ms delay to detect if we are dealing with the escape key or an
escape sequence.

Signed-off-by: Heinrich Schuchardt

efi_loader: missing parentheses after if

2020-12-29T01:09:04+00:00

IS_ENABLED() contains parentheses. But we should still put extra
parentheses around it in an if statement for readability.

Signed-off-by: Heinrich Schuchardt

efi_loader: Extra checks while opening an OPTEE session

2020-12-26T11:47:46+00:00

When opening an OP-TEE session we need to check the internal return
value of OP-TEE call arguments as well the return code of the
function itself.
The code was also ignoring to close the OP-TEE session in case the
shared memory registration failed.

Fixes: f042e47e8fb43 ("efi_loader: Implement EFI variable handling via OP-TEE")
Signed-off-by: Ilias Apalodimas 
Signed-off-by: Heinrich Schuchardt

efi_loader: make variable store size customizable

2020-12-20T16:40:06+00:00

Currently the size of the buffer to keep UEFI variables in memory is fixed
at 16384 bytes. This size has proven to be too small for some use cases.

Make the size of the memory buffer for UEFI variables customizable.

Reported-by: Paulo Alcantara (SUSE) 
Signed-off-by: Heinrich Schuchardt 
Acked-by: Ilias Apalodimas

binman: Remove additional backslash

2020-11-30T20:14:43+00:00

The origin patch didn't have this change and it was caused by manual
resolution where additional backslash was added.

Fixes: 6723b4c6ca7b ("binman: Call helper function binman_set_rom_offset() to fill offset")
Signed-off-by: Michal Simek 
Reviewed-by: Simon Glass

charset: make u16_strnlen accessible at runtime

2020-11-29T04:18:37+00:00

commit 1fabfeef506c ("efi_loader: parameter check in GetNextVariableName()")
introduces a check using u16_strnlen(). This code is used on EFI
runtime variables as well, so unless we mark it as runtime, the kernel
will crash trying to access it.

Fixes: 1fabfeef506c ("efi_loader: parameter check in GetNextVariableName()")
Signed-off-by: Ilias Apalodimas 
Reviewed-by: Heinrich Schuchardt

efi_loader: enable EFI_SET_TIME on sandbox and QEMU ARM

2020-11-29T04:18:37+00:00

Enable EFI_SET_TIME on the sandbox and QEMU ARM to ensure that we compile
and test the relevant code.

Signed-off-by: Heinrich Schuchardt