diff options
| author | Ed Maste <emaste@FreeBSD.org> | 2026-07-21 19:19:40 +0000 |
|---|---|---|
| committer | Ed Maste <emaste@FreeBSD.org> | 2026-07-21 21:23:11 +0000 |
| commit | 2b1df6149e8a2d50a09d13c64d1574dad91e10b1 (patch) | |
| tree | 4a9b2175e78fcd370490549fef0911ef9d6d425f | |
| parent | 1092aba1260ce5efb0d718352aa22158605c7506 (diff) | |
loader: Allocate trampoline as EfiLoaderCode, not Data
Firmware on a test machine applied NX to non-code allocations, which
resulted in a fault when jumping to the trampoline.
Reviewed by: kib
Tested by: Jim Huang Chen <jim.chen.1827@gmail.com>
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D58383
| -rw-r--r-- | stand/efi/loader/arch/amd64/elf64_freebsd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/stand/efi/loader/arch/amd64/elf64_freebsd.c b/stand/efi/loader/arch/amd64/elf64_freebsd.c index 62716d7b1369..2ba5c1419168 100644 --- a/stand/efi/loader/arch/amd64/elf64_freebsd.c +++ b/stand/efi/loader/arch/amd64/elf64_freebsd.c @@ -102,7 +102,7 @@ elf64_exec(struct preloaded_file *fp) trampcode = copy_staging == COPY_STAGING_ENABLE ? (vm_offset_t)G(1) : (vm_offset_t)G(4); - err = BS->AllocatePages(AllocateMaxAddress, EfiLoaderData, 1, + err = BS->AllocatePages(AllocateMaxAddress, EfiLoaderCode, 1, (EFI_PHYSICAL_ADDRESS *)&trampcode); if (EFI_ERROR(err)) { printf("Unable to allocate trampoline\n"); |