]> git.proxmox.com Git - grub2.git/commitdiff
util/mkimage: Remove unused code to add BSS section
authorJavier Martinez Canillas <javierm@redhat.com>
Thu, 11 Feb 2021 16:06:49 +0000 (17:06 +0100)
committerColin Watson <cjwatson@debian.org>
Sun, 13 Jun 2021 23:40:46 +0000 (00:40 +0100)
The code is compiled out so there is no reason to keep it.

Additionally, don't set bss_size field since we do not add a BSS section.

Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
Patch-Name: 2021-02-security/100-util-mkimage-Remove-unused-code-to-add-BSS-section.patch

util/mkimage.c

index 37d6249f16b3df0a90ce74c40b8b3304f1465aaa..32bb8ea68b6adfe34eecda4e6ad4257911ac31a7 100644 (file)
@@ -1304,7 +1304,6 @@ grub_install_generate_image (const char *dir, const char *prefix,
            o->code_size = grub_host_to_target32 (layout.exec_size);
            o->data_size = grub_cpu_to_le32 (reloc_addr - layout.exec_size
                                             - header_size);
-           o->bss_size = grub_cpu_to_le32 (layout.bss_size);
            o->entry_addr = grub_cpu_to_le32 (layout.start_address);
            o->code_base = grub_cpu_to_le32 (header_size);
 
@@ -1342,7 +1341,6 @@ grub_install_generate_image (const char *dir, const char *prefix,
            o->code_size = grub_host_to_target32 (layout.exec_size);
            o->data_size = grub_cpu_to_le32 (reloc_addr - layout.exec_size
                                             - header_size);
-           o->bss_size = grub_cpu_to_le32 (layout.bss_size);
            o->entry_addr = grub_cpu_to_le32 (layout.start_address);
            o->code_base = grub_cpu_to_le32 (header_size);
            o->image_base = 0;
@@ -1387,21 +1385,6 @@ grub_install_generate_image (const char *dir, const char *prefix,
          = grub_cpu_to_le32_compile_time (GRUB_PE32_SCN_CNT_INITIALIZED_DATA
                              | GRUB_PE32_SCN_MEM_READ
                              | GRUB_PE32_SCN_MEM_WRITE);
-
-#if 0
-       bss_section = data_section + 1;
-       strcpy (bss_section->name, ".bss");
-       bss_section->virtual_size = grub_cpu_to_le32 (layout.bss_size);
-       bss_section->virtual_address = grub_cpu_to_le32 (header_size + layout.kernel_size);
-       bss_section->raw_data_size = 0;
-       bss_section->raw_data_offset = 0;
-       bss_section->characteristics
-         = grub_cpu_to_le32_compile_time (GRUB_PE32_SCN_MEM_READ
-                             | GRUB_PE32_SCN_MEM_WRITE
-                             | GRUB_PE32_SCN_ALIGN_64BYTES
-                             | GRUB_PE32_SCN_CNT_INITIALIZED_DATA
-                             | 0x80);
-#endif
     
        mods_section = data_section + 1;
        strcpy (mods_section->name, "mods");