u-boot.git/lib/efi_selftest, branch v2019.01

efi_selftest: block device: avoid read after free

2018-12-23T01:04:25Z

Reading the position in a file after closing the same results in a read after free. Correct the sequence in the test. Reported-by: Marek Vasut Signed-off-by: Heinrich Schuchardt Signed-off-by: Alexander Graf

efi_selftest: rename setup_ok

2018-12-02T20:59:37Z

The variable name setup_ok might suggest a boolean with true indicating OK. Let's avoid the misleading name. %s/setup_ok/setup_status/g Suggested-by: Simon Glass Signed-off-by: Heinrich Schuchardt Reviewed-by: Simon Glass Signed-off-by: Alexander Graf

efi_selftest: fix simple network protocol test

2018-12-02T20:59:37Z

To use the simple network protocol we have to call the start service first and the initialize service second. Signed-off-by: Heinrich Schuchardt Signed-off-by: Alexander Graf

efi_selftest: check fdt is marked as runtime data

2018-12-02T20:59:37Z

Check that the memory area containing the device tree is marked as runtime data. Update the Python test to pass ${fdtcontroladdr} to bootefi. Update the description of the Python test. Signed-off-by: Heinrich Schuchardt Signed-off-by: Alexander Graf

efi_selftest: building sandbox with EFI_SELFTEST

2018-12-02T20:59:37Z

Enable building the sandbox with CONFIG_EFI_SELFTEST. Signed-off-by: Heinrich Schuchardt Signed-off-by: Alexander Graf

efi_selftest: add test for memory allocation

2018-12-02T20:59:37Z

This unit test checks the following runtime services: AllocatePages, FreePages, GetMemoryMap Signed-off-by: Heinrich Schuchardt Signed-off-by: Alexander Graf

efi_selftest: incorrect use of bitwise or

2018-12-02T20:59:37Z

We should use a logical or when combining logical values. Signed-off-by: Heinrich Schuchardt Signed-off-by: Alexander Graf

efi_loader: do not use unsupported printf code

2018-12-02T20:59:37Z

Using %zu for efi_intn_t (ssize_t) creates a build warning. Anyway %zu is not supported by efi_st_error(). So let's convert to int. Our implementation of StriColl() only returns -1, 0, or 1. Signed-off-by: Heinrich Schuchardt Signed-off-by: Alexander Graf

efi_selftest: do not write to linker generated array

2018-12-02T20:59:37Z

Linker generated arrays may be stored in code sections of memory that are not writable. So let's allocate setup_ok as an array at runtime. This avoids an illegal memory access observed in the sandbox. Reported-by: Simon Glass Signed-off-by: Heinrich Schuchardt Signed-off-by: Alexander Graf

efi_loader: UninstallMultipleProtocolInterfaces error code

2018-12-02T20:59:36Z

If UninstallMultipleProtocolInterfaces fails, we sometimes return the wrong status code. The UEFI spec mandates to always return EFI_INVALID_PARAMETER. Update unit test. Signed-off-by: Heinrich Schuchardt Signed-off-by: Alexander Graf