[ELF] --check-sections: allow address 0xffffffff for ELFCLASS32
authorFangrui Song <i@maskray.me>
Sat, 1 Oct 2022 22:37:07 +0000 (15:37 -0700)
committerFangrui Song <i@maskray.me>
Sat, 1 Oct 2022 22:37:07 +0000 (15:37 -0700)
Fix https://github.com/llvm/llvm-project/issues/58101

lld/ELF/Writer.cpp
lld/test/ELF/linkerscript/i386-sections-max-va-overflow.s

index 26c6239..ef80082 100644 (file)
@@ -2686,7 +2686,7 @@ template <class ELFT> void Writer<ELFT>::checkSections() {
   // First, check that section's VAs fit in available address space for target.
   for (OutputSection *os : outputSections)
     if ((os->addr + os->size < os->addr) ||
-        (!ELFT::Is64Bits && os->addr + os->size > UINT32_MAX))
+        (!ELFT::Is64Bits && os->addr + os->size > uint64_t(UINT32_MAX) + 1))
       errorOrWarn("section " + os->name + " at 0x" + utohexstr(os->addr) +
                   " of size 0x" + utohexstr(os->size) +
                   " exceeds available address space");
index d424112..25bef15 100644 (file)
@@ -5,9 +5,14 @@
 # RUN: echo "           .bar : { *(.bar*) } }" >> %t.script
 # RUN: not ld.lld -o /dev/null --script %t.script %t.o 2>&1 | FileCheck %s -check-prefix=ERR
 
-## .bar section has data in [0xfffffff1, 0xfffffff1 + 0x10] == [0xffffff1, 0x1]. 
+## .bar section has data in [0xfffffff1, 0xfffffff1 + 0x10] == [0xfffffff1, 0x1].
 ## Check we can catch this overflow.
 # ERR: error: section .bar at 0xFFFFFFF1 of size 0x10 exceeds available address space
 
+## [0xfffffff1, 0x100000000) is allowed.
+# RUN: echo "SECTIONS { . = 0xfffffff0;" > %t.script
+# RUN: echo "           .bar : { *(.bar*) } }" >> %t.script
+# RUN: ld.lld -o /dev/null -T %t.script %t.o
+
 .section .bar,"ax",@progbits
 .zero 0x10