[PATCH] arm64/efi: map the entire UEFI vendor string before reading it

Ard Biesheuvel ard.biesheuvel at linaro.org
Mon Jul 27 05:16:20 PDT 2015


On 27 July 2015 at 14:14, Catalin Marinas <catalin.marinas at arm.com> wrote:
> On Sun, Jul 26, 2015 at 02:59:00PM +0200, Ard Biesheuvel wrote:
>> At boot, the UTF-16 UEFI vendor string is copied from the system
>> table into a char array with a size of 100 bytes. However, this
>> size of 100 bytes is also used for memremapping() the source,
>> which may not be sufficient if the vendor string exceeds 50
>> UTF-16 characters, and the placement of the vendor string inside
>> a 4 KB page happens to leave the end unmapped.
>>
>> So use the correct '100 * sizeof(efi_char16_t)' for the size of
>> the mapping.
>>
>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel at linaro.org>
>> ---
>> Hello Catalin,
>>
>> I think this should go into v4.2 with a cc: stable.
>
> If it's cc stable, do you have a Fixes: tag? (it saves me some
> searching).
>

Ah sure, I forgot.

Fixes: f84d02755f5a ("arm64: add EFI runtime services")



More information about the linux-arm-kernel mailing list