Commit 04a7d0e1 authored by Ard Biesheuvel's avatar Ard Biesheuvel

efi/libstub/x86: Avoid overflowing code32_start on PE entry

When using the native PE entry point (as opposed to the EFI handover
protocol entry point that is used more widely), we set code32_start,
which is a 32-bit wide field, to the effective symbol address of
startup_32, which could overflow given that the EFI loader may have
located the running image anywhere in memory, and we haven't reached
the point yet where we relocate ourselves.

Since we relocate ourselves if code32_start != pref_address, this
isn't likely to lead to problems in practice, given how unlikely
it is that the truncated effective address of startup_32 happens
to equal pref_address. But it is better to defer the assignment
of code32_start to after the relocation, when it is guaranteed to
fit.

While at it, move the call to efi_relocate_kernel() to an earlier
stage so it is more likely that our preferred offset in memory has
not been occupied by other memory allocations done in the mean time.
Signed-off-by: default avatarArd Biesheuvel <ardb@kernel.org>
parent e6d832ea
...@@ -439,8 +439,6 @@ efi_status_t __efiapi efi_pe_entry(efi_handle_t handle, ...@@ -439,8 +439,6 @@ efi_status_t __efiapi efi_pe_entry(efi_handle_t handle,
boot_params->ext_ramdisk_image = (u64)ramdisk_addr >> 32; boot_params->ext_ramdisk_image = (u64)ramdisk_addr >> 32;
boot_params->ext_ramdisk_size = (u64)ramdisk_size >> 32; boot_params->ext_ramdisk_size = (u64)ramdisk_size >> 32;
hdr->code32_start = (u32)(unsigned long)startup_32;
efi_stub_entry(handle, sys_table, boot_params); efi_stub_entry(handle, sys_table, boot_params);
/* not reached */ /* not reached */
...@@ -707,6 +705,7 @@ struct boot_params *efi_main(efi_handle_t handle, ...@@ -707,6 +705,7 @@ struct boot_params *efi_main(efi_handle_t handle,
efi_system_table_t *sys_table_arg, efi_system_table_t *sys_table_arg,
struct boot_params *boot_params) struct boot_params *boot_params)
{ {
unsigned long bzimage_addr = (unsigned long)startup_32;
struct setup_header *hdr = &boot_params->hdr; struct setup_header *hdr = &boot_params->hdr;
efi_status_t status; efi_status_t status;
unsigned long cmdline_paddr; unsigned long cmdline_paddr;
...@@ -717,6 +716,23 @@ struct boot_params *efi_main(efi_handle_t handle, ...@@ -717,6 +716,23 @@ struct boot_params *efi_main(efi_handle_t handle,
if (sys_table->hdr.signature != EFI_SYSTEM_TABLE_SIGNATURE) if (sys_table->hdr.signature != EFI_SYSTEM_TABLE_SIGNATURE)
goto fail; goto fail;
/*
* If the kernel isn't already loaded at the preferred load
* address, relocate it.
*/
if (bzimage_addr != hdr->pref_address) {
status = efi_relocate_kernel(&bzimage_addr,
hdr->init_size, hdr->init_size,
hdr->pref_address,
hdr->kernel_alignment,
LOAD_PHYSICAL_ADDR);
if (status != EFI_SUCCESS) {
efi_printk("efi_relocate_kernel() failed!\n");
goto fail;
}
}
hdr->code32_start = (u32)bzimage_addr;
/* /*
* make_boot_params() may have been called before efi_main(), in which * make_boot_params() may have been called before efi_main(), in which
* case this is the second time we parse the cmdline. This is ok, * case this is the second time we parse the cmdline. This is ok,
...@@ -746,26 +762,6 @@ struct boot_params *efi_main(efi_handle_t handle, ...@@ -746,26 +762,6 @@ struct boot_params *efi_main(efi_handle_t handle,
setup_quirks(boot_params); setup_quirks(boot_params);
/*
* If the kernel isn't already loaded at the preferred load
* address, relocate it.
*/
if (hdr->pref_address != hdr->code32_start) {
unsigned long bzimage_addr = hdr->code32_start;
status = efi_relocate_kernel(&bzimage_addr,
hdr->init_size, hdr->init_size,
hdr->pref_address,
hdr->kernel_alignment,
LOAD_PHYSICAL_ADDR);
if (status != EFI_SUCCESS) {
efi_printk("efi_relocate_kernel() failed!\n");
goto fail;
}
hdr->pref_address = hdr->code32_start;
hdr->code32_start = bzimage_addr;
}
status = exit_boot(boot_params, handle); status = exit_boot(boot_params, handle);
if (status != EFI_SUCCESS) { if (status != EFI_SUCCESS) {
efi_printk("exit_boot() failed!\n"); efi_printk("exit_boot() failed!\n");
......
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