diff options
| author | Jan Janssen <medhefgo@web.de> | 2023-03-14 14:32:43 +0100 |
|---|---|---|
| committer | Luca Boccassi <luca.boccassi@gmail.com> | 2023-03-15 00:24:17 +0000 |
| commit | ccde62954ccf5ee73b40774b8f5d3a9dd2869201 (patch) | |
| tree | 1f837376f6c59ce265753c4010704422b9cccddb /src/boot | |
| parent | 544471bf8340f4f1de48b3b83438aea8361135fc (diff) | |
| download | systemd-ccde62954ccf5ee73b40774b8f5d3a9dd2869201.tar.gz | |
stub: Remove overlapping PE section warning
Now that we always create PE images with a non-zero image base we cannot
run into this issue anymore. Any tool that still uses the old hard-coded
section offsets will end up creating bad images with sections below the
image base. objcopy will warn about this and any PE loader will simply
refuse to load such an image. Meanwhile updated tools do not use
hard-coded offsets anymore.
Diffstat (limited to 'src/boot')
| -rw-r--r-- | src/boot/efi/pe.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/src/boot/efi/pe.c b/src/boot/efi/pe.c index 9b1b10d4a1..e516417c07 100644 --- a/src/boot/efi/pe.c +++ b/src/boot/efi/pe.c @@ -153,17 +153,9 @@ static void locate_sections( assert(offsets); assert(sizes); - size_t prev_section_addr = 0; - for (size_t i = 0; i < n_table; i++) { const PeSectionHeader *sect = section_table + i; - if (in_memory) { - if (prev_section_addr > sect->VirtualAddress) - log_error("Overlapping PE sections detected. Boot may fail due to image memory corruption!"); - prev_section_addr = sect->VirtualAddress + sect->VirtualSize; - } - for (size_t j = 0; sections[j]; j++) { if (memcmp(sect->Name, sections[j], strlen8(sections[j])) != 0) continue; |