|
 |
9723a8 |
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
|
9723a8 |
From: Peter Jones <pjones@redhat.com>
|
|
|
9723a8 |
Date: Mon, 15 Feb 2021 14:21:48 +0100
|
|
|
9723a8 |
Subject: [PATCH] util/mkimage: Reorder PE optional header fields set-up
|
|
|
9723a8 |
|
|
|
9723a8 |
This makes the PE32 and PE32+ header fields set-up easier to follow by
|
|
|
9723a8 |
setting them closer to the initialization of their related sections.
|
|
|
9723a8 |
|
|
|
9723a8 |
Signed-off-by: Peter Jones <pjones@redhat.com>
|
|
|
9723a8 |
Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
|
|
|
9723a8 |
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
|
|
|
9723a8 |
---
|
|
|
9723a8 |
util/mkimage.c | 16 ++++++++--------
|
|
|
9723a8 |
1 file changed, 8 insertions(+), 8 deletions(-)
|
|
|
9723a8 |
|
|
|
9723a8 |
diff --git a/util/mkimage.c b/util/mkimage.c
|
|
|
3efed6 |
index ff5462c98c1..e73a5864b22 100644
|
|
|
9723a8 |
--- a/util/mkimage.c
|
|
|
9723a8 |
+++ b/util/mkimage.c
|
|
|
3efed6 |
@@ -1320,16 +1320,12 @@ grub_install_generate_image (const char *dir, const char *prefix,
|
|
|
9723a8 |
sections = o64 + 1;
|
|
|
9723a8 |
}
|
|
|
9723a8 |
|
|
|
9723a8 |
- PE_OHDR (o32, o64, code_size) = grub_host_to_target32 (layout.exec_size);
|
|
|
9723a8 |
- PE_OHDR (o32, o64, data_size) = grub_host_to_target32 (reloc_addr - layout.exec_size - header_size);
|
|
|
9723a8 |
+ PE_OHDR (o32, o64, header_size) = grub_host_to_target32 (header_size);
|
|
|
9723a8 |
PE_OHDR (o32, o64, entry_addr) = grub_host_to_target32 (layout.start_address);
|
|
|
9723a8 |
- PE_OHDR (o32, o64, code_base) = grub_host_to_target32 (header_size);
|
|
|
9723a8 |
-
|
|
|
9723a8 |
PE_OHDR (o32, o64, image_base) = 0;
|
|
|
9723a8 |
+ PE_OHDR (o32, o64, image_size) = grub_host_to_target32 (pe_size);
|
|
|
9723a8 |
PE_OHDR (o32, o64, section_alignment) = grub_host_to_target32 (image_target->section_align);
|
|
|
9723a8 |
PE_OHDR (o32, o64, file_alignment) = grub_host_to_target32 (GRUB_PE32_FILE_ALIGNMENT);
|
|
|
9723a8 |
- PE_OHDR (o32, o64, image_size) = grub_host_to_target32 (pe_size);
|
|
|
9723a8 |
- PE_OHDR (o32, o64, header_size) = grub_host_to_target32 (header_size);
|
|
|
9723a8 |
PE_OHDR (o32, o64, subsystem) = grub_host_to_target16 (GRUB_PE32_SUBSYSTEM_EFI_APPLICATION);
|
|
|
9723a8 |
|
|
|
9723a8 |
/* Do these really matter? */
|
|
|
3efed6 |
@@ -1339,10 +1335,10 @@ grub_install_generate_image (const char *dir, const char *prefix,
|
|
|
9723a8 |
PE_OHDR (o32, o64, heap_commit_size) = grub_host_to_target32 (0x10000);
|
|
|
9723a8 |
|
|
|
9723a8 |
PE_OHDR (o32, o64, num_data_directories) = grub_host_to_target32 (GRUB_PE32_NUM_DATA_DIRECTORIES);
|
|
|
9723a8 |
- PE_OHDR (o32, o64, base_relocation_table.rva) = grub_host_to_target32 (reloc_addr);
|
|
|
9723a8 |
- PE_OHDR (o32, o64, base_relocation_table.size) = grub_host_to_target32 (layout.reloc_size);
|
|
|
9723a8 |
|
|
|
9723a8 |
/* The sections. */
|
|
|
9723a8 |
+ PE_OHDR (o32, o64, code_base) = grub_host_to_target32 (header_size);
|
|
|
9723a8 |
+ PE_OHDR (o32, o64, code_size) = grub_host_to_target32 (layout.exec_size);
|
|
|
9723a8 |
text_section = sections;
|
|
|
9723a8 |
strcpy (text_section->name, ".text");
|
|
|
9723a8 |
text_section->virtual_size = grub_host_to_target32 (layout.exec_size);
|
|
|
3efed6 |
@@ -1354,6 +1350,8 @@ grub_install_generate_image (const char *dir, const char *prefix,
|
|
|
9723a8 |
| GRUB_PE32_SCN_MEM_EXECUTE
|
|
|
9723a8 |
| GRUB_PE32_SCN_MEM_READ);
|
|
|
9723a8 |
|
|
|
9723a8 |
+ PE_OHDR (o32, o64, data_size) = grub_host_to_target32 (reloc_addr - layout.exec_size - header_size);
|
|
|
9723a8 |
+
|
|
|
9723a8 |
data_section = text_section + 1;
|
|
|
9723a8 |
strcpy (data_section->name, ".data");
|
|
|
9723a8 |
data_section->virtual_size = grub_host_to_target32 (layout.kernel_size - layout.exec_size);
|
|
|
3efed6 |
@@ -1376,6 +1374,8 @@ grub_install_generate_image (const char *dir, const char *prefix,
|
|
|
9723a8 |
| GRUB_PE32_SCN_MEM_READ
|
|
|
9723a8 |
| GRUB_PE32_SCN_MEM_WRITE);
|
|
|
9723a8 |
|
|
|
9723a8 |
+ PE_OHDR (o32, o64, base_relocation_table.rva) = grub_host_to_target32 (reloc_addr);
|
|
|
9723a8 |
+ PE_OHDR (o32, o64, base_relocation_table.size) = grub_host_to_target32 (layout.reloc_size);
|
|
|
9723a8 |
reloc_section = mods_section + 1;
|
|
|
9723a8 |
strcpy (reloc_section->name, ".reloc");
|
|
|
9723a8 |
reloc_section->virtual_size = grub_host_to_target32 (layout.reloc_size);
|