Skip to content

Commit dbaa118

Browse files
keeskrazey
authored andcommitted
FROMLIST: rename missed uaccess .fixup section
When the uaccess .fixup section was renamed to .text.fixup, one case was missed. Under ld.bfd, the orphaned section was moved close to .text (since they share the "ax" bits), so things would work normally on uaccess faults. Under ld.lld, the orphaned section was placed outside the .text section, making it unreachable. Fixes: c4a84ae ("ARM: 8322/1: keep .text and .fixup regions closer together") Link: ClangBuiltLinux/linux#282 Link: https://bugs.chromium.org/p/chromium/issues/detail?id=1020633#c44 Link: https://lore.kernel.org/r/[email protected] Link: https://lore.kernel.org/lkml/202002071754.F5F073F1D@keescook/ Cc: [email protected] Signed-off-by: Kees Cook <[email protected]> Reviewed-by: Ard Biesheuvel <[email protected]> Reviewed-by: Nick Desaulniers <[email protected]> Bug: 149493767 (am from https://www.armlinux.org.uk/developer/patches/viewpatch.php?id=8958/1) Signed-off-by: Nick Desaulniers <[email protected]> Change-Id: I39a2d2e7bfee7ff9854782c06547c277895e8ec1
1 parent ee696fa commit dbaa118

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

arch/arm/lib/copy_from_user.S

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ ENTRY(arm_copy_from_user)
100100

101101
ENDPROC(arm_copy_from_user)
102102

103-
.pushsection .fixup,"ax"
103+
.pushsection .text.fixup,"ax"
104104
.align 0
105105
copy_abort_preamble
106106
ldmfd sp!, {r1, r2, r3}

0 commit comments

Comments
 (0)