diff options
| author | Masahiro Yamada <masahiroy@kernel.org> | 2023-06-01 21:10:01 +0900 |
|---|---|---|
| committer | Masahiro Yamada <masahiroy@kernel.org> | 2023-06-04 01:37:41 +0900 |
| commit | 2cb749466d179e3ccfe83eb8a52dc002d07b08af (patch) | |
| tree | 4b80e1815c2b9679621b0b4a2363883ae0540638 /scripts | |
| parent | 3310bae805250aec227eb056e8e61a246678f28a (diff) | |
| download | linux-2cb749466d179e3ccfe83eb8a52dc002d07b08af.tar.gz linux-2cb749466d179e3ccfe83eb8a52dc002d07b08af.tar.bz2 linux-2cb749466d179e3ccfe83eb8a52dc002d07b08af.zip | |
modpost: detect section mismatch for R_ARM_REL32
For ARM, modpost fails to detect some types of section mismatches.
[test code]
.section .init.data,"aw"
bar:
.long 0
.section .data,"aw"
.globl foo
foo:
.long bar - .
It is apparently a bad reference, but modpost does not report anything.
The test code above produces the following relocations.
Relocation section '.rel.data' at offset 0xe8 contains 1 entry:
Offset Info Type Sym.Value Sym. Name
00000000 00000403 R_ARM_REL32 00000000 .init.data
Currently, R_ARM_REL32 is just skipped.
Handle it like R_ARM_ABS32.
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Diffstat (limited to 'scripts')
| -rw-r--r-- | scripts/mod/modpost.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index 4e911ab711d4..d10f5bdcb753 100644 --- a/scripts/mod/modpost.c +++ b/scripts/mod/modpost.c @@ -1281,6 +1281,7 @@ static int addend_arm_rel(struct elf_info *elf, Elf_Shdr *sechdr, Elf_Rela *r) switch (r_typ) { case R_ARM_ABS32: + case R_ARM_REL32: inst = TO_NATIVE(*(uint32_t *)loc); r->r_addend = inst + sym->st_value; break; |