Commit 0b1d9deb authored by Ard Biesheuvel's avatar Ard Biesheuvel

efi/libstub: randomalloc: Return EFI_OUT_OF_RESOURCES on failure

The logic in efi_random_alloc() will iterate over the memory map twice,
once to count the number of candidate slots, and another time to locate
the chosen slot after randomization.

If there is insufficient memory to do the allocation, the second loop
will run to completion without actually having located a slot, but we
currently return EFI_SUCCESS in this case, as we fail to initialize
status to the appropriate error value of EFI_OUT_OF_RESOURCES.
Signed-off-by: default avatarArd Biesheuvel <ardb@kernel.org>
parent fc3608aa
...@@ -101,6 +101,7 @@ efi_status_t efi_random_alloc(unsigned long size, ...@@ -101,6 +101,7 @@ efi_status_t efi_random_alloc(unsigned long size,
* to calculate the randomly chosen address, and allocate it directly * to calculate the randomly chosen address, and allocate it directly
* using EFI_ALLOCATE_ADDRESS. * using EFI_ALLOCATE_ADDRESS.
*/ */
status = EFI_OUT_OF_RESOURCES;
for (map_offset = 0; map_offset < map->map_size; map_offset += map->desc_size) { for (map_offset = 0; map_offset < map->map_size; map_offset += map->desc_size) {
efi_memory_desc_t *md = (void *)map->map + map_offset; efi_memory_desc_t *md = (void *)map->map + map_offset;
efi_physical_addr_t target; efi_physical_addr_t target;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment