From 8592be8c7d300c37dc44b0f5a1a381ede99a6048 Mon Sep 17 00:00:00 2001 From: Nick Alcock Date: Thu, 25 Mar 2021 16:32:46 +0000 Subject: [PATCH] ld: do not rely on the exact size of the CTF symtypetabs in test results The data object and function info sections (collectively "symtypetabs") usually (i.e. if non-indexed) have sizes defined by the size of the ELF dynamic symbol table in the object they are linked to. This means test results should not depend on the exact sizes of these sections, because adding entirely irrelevant symbols to the dynsym can cause spurious test failures. (This also means we should not match the offset of sections that follow them, since those too depend on the exact size of the symtypetab sections.) Spotted by turning the sanitizer on, which introduced new dynsym entries and expanded the symtypetab sizes to match. ld/ChangeLog 2021-03-25 Nick Alcock * testsuite/ld-ctf/array.d: Only check that the data object section is nonempty: do not check its exact size. * testsuite/ld-ctf/diag-parlabel.d: Likewise. * testsuite/ld-ctf/slice.d: Likewise. * testsuite/ld-ctf/data-func-conflicted.d: Likewise, and for the func info section too. * testsuite/ld-ctf/function.d: Likewise, for the func info section. --- ld/ChangeLog | 10 ++++++++++ ld/testsuite/ld-ctf/array.d | 2 +- ld/testsuite/ld-ctf/data-func-conflicted.d | 6 +++--- ld/testsuite/ld-ctf/diag-parlabel.d | 4 ++-- ld/testsuite/ld-ctf/function.d | 2 +- ld/testsuite/ld-ctf/slice.d | 2 +- 6 files changed, 18 insertions(+), 8 deletions(-) diff --git a/ld/ChangeLog b/ld/ChangeLog index 19b56f02b54..c10bc392fd1 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,3 +1,13 @@ +2021-03-25 Nick Alcock + + * testsuite/ld-ctf/array.d: Only check that the data object + section is nonempty: do not check its exact size. + * testsuite/ld-ctf/diag-parlabel.d: Likewise. + * testsuite/ld-ctf/slice.d: Likewise. + * testsuite/ld-ctf/data-func-conflicted.d: Likewise, and for the + func info section too. + * testsuite/ld-ctf/function.d: Likewise, for the func info section. + 2021-03-22 Martin Liska * emultempl/aix.em: Replace usage of CONST_STRNEQ with startswith. diff --git a/ld/testsuite/ld-ctf/array.d b/ld/testsuite/ld-ctf/array.d index 16375620c03..ab262d915b5 100644 --- a/ld/testsuite/ld-ctf/array.d +++ b/ld/testsuite/ld-ctf/array.d @@ -13,7 +13,7 @@ Contents of CTF section .ctf: Magic number: 0xdff2 Version: 4 \(CTF_VERSION_3\) #... - Data object section: .* \(0x8 bytes\) + Data object section: .* \(0x[1-9a-f][0-9a-f]* bytes\) Type section: .* \(0x5c bytes\) String section: .* diff --git a/ld/testsuite/ld-ctf/data-func-conflicted.d b/ld/testsuite/ld-ctf/data-func-conflicted.d index 1e7c19a5b36..87dd4b61e3b 100644 --- a/ld/testsuite/ld-ctf/data-func-conflicted.d +++ b/ld/testsuite/ld-ctf/data-func-conflicted.d @@ -13,8 +13,8 @@ Contents of CTF section \.ctf: Magic number: 0xdff2 Version: 4 \(CTF_VERSION_3\) #... - Data object section: .* \(0xc bytes\) - Function info section: .* \(0x44 bytes\) + Data object section: .* \(0x[1-9a-f][0-9a-f]* bytes\) + Function info section: .* \(0x[1-9a-f][0-9a-f]* bytes\) Object index section: .* \(0xc bytes\) Type section: .* \(0xf4 bytes\) String section: .* @@ -39,7 +39,7 @@ CTF archive member: .*/data-func-1\.c: #... Parent name: \.ctf Compilation unit name: .*/data-func-1\.c - Data object section: .* \(0xfa4 bytes\) + Data object section: .* \(0x[1-9a-f][0-9a-f]* bytes\) Type section: .* \(0xc bytes\) String section: .* diff --git a/ld/testsuite/ld-ctf/diag-parlabel.d b/ld/testsuite/ld-ctf/diag-parlabel.d index bba161cf309..9d2c0860997 100644 --- a/ld/testsuite/ld-ctf/diag-parlabel.d +++ b/ld/testsuite/ld-ctf/diag-parlabel.d @@ -13,8 +13,8 @@ Contents of CTF section .ctf: Version: 4 \(CTF_VERSION_3\) #... Compilation unit name: .*A.c - Data object section: 0x0 -- 0x3 \(0x4 bytes\) - Type section: 0x4 -- 0x33 \(0x30 bytes\) + Data object section: .* \(0x[1-9a-f][0-9a-f]* bytes\) + Type section: .* \(0x30 bytes\) String section: .* Labels: diff --git a/ld/testsuite/ld-ctf/function.d b/ld/testsuite/ld-ctf/function.d index 1584bdbedfd..e6cb20951f4 100644 --- a/ld/testsuite/ld-ctf/function.d +++ b/ld/testsuite/ld-ctf/function.d @@ -13,7 +13,7 @@ Contents of CTF section .ctf: Version: 4 \(CTF_VERSION_3\) #... Compilation unit name: .*function.c - Function info section: .* \(0x4 bytes\) + Function info section: .* \(0x[1-9a-f][0-9a-f]* bytes\) Type section: .* \(0x90 bytes\) String section: .* #... diff --git a/ld/testsuite/ld-ctf/slice.d b/ld/testsuite/ld-ctf/slice.d index 72a7074f9ef..efe32a87523 100644 --- a/ld/testsuite/ld-ctf/slice.d +++ b/ld/testsuite/ld-ctf/slice.d @@ -14,7 +14,7 @@ Contents of CTF section .ctf: #... Compilation unit name: .*slice.c #... - Data object section: .* \(0x4 bytes\) + Data object section: .* \(0x[1-9a-f][0-9a-f]* bytes\) Type section: .* \(0xd0 bytes\) String section: .* #... -- 2.30.2