linux-kernel - [PATCH v3 1/1] ARM: kprobes: Explicitly reserve r7 for local variables

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <20231120032909.19186-1-quic_aiquny@quicinc.com>
Date:   Mon, 20 Nov 2023 11:29:09 +0800
From:   Maria Yu <quic_aiquny@...cinc.com>
To:     Nathan Chancellor <nathan@...nel.org>, <linux@...linux.org.uk>,
        <ardb@...nel.org>, <mhiramat@...nel.org>
CC:     Maria Yu <quic_aiquny@...cinc.com>, <kernel@...cinc.com>,
        <linux-arm-kernel@...ts.infradead.org>,
        <linux-arm-msm@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <quic_lijuang@...cinc.com>, <stable@...r.kernel.org>
Subject: [PATCH v3 1/1] ARM: kprobes: Explicitly reserve r7 for local variables

Registers r7 is removed in clobber list, so compiler may choose r7 for
local variables usage, while r7 will be actually updated by the inline asm
code. This caused the runtime behavior wrong.
While those kind of reserved registers cannot be set to clobber list
because of error like "inline asm clobber list contains reserved
registers".
Explicitly reserve r7 by adding attribute no-omit-frame-pointer for this
file, then in T32 asm code r7 is used as a frame pointer and is not
available for use as a general-purpose register.
Note that "no-omit-frame-pointer" will make the code size a little bigger
to store the stack frame pointer.

Fixes: dd12e97f3c72 ("ARM: kprobes: treat R7 as the frame pointer register in Thumb2 builds")
Suggested-by: Ard Biesheuvel <ardb@...nel.org>
Signed-off-by: Maria Yu <quic_aiquny@...cinc.com>
Cc: stable@...r.kernel.org
---
 arch/arm/probes/kprobes/Makefile | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/probes/kprobes/Makefile b/arch/arm/probes/kprobes/Makefile
index 6159010dac4a..b1f21e78950b 100644
--- a/arch/arm/probes/kprobes/Makefile
+++ b/arch/arm/probes/kprobes/Makefile
@@ -8,6 +8,7 @@ test-kprobes-objs		:= test-core.o

 ifdef CONFIG_THUMB2_KERNEL
 obj-$(CONFIG_KPROBES)		+= actions-thumb.o checkers-thumb.o
+CFLAGS_actions-thumb.o		+= -fno-omit-frame-pointer
 test-kprobes-objs		+= test-thumb.o
 else
 obj-$(CONFIG_KPROBES)		+= actions-arm.o checkers-arm.o

base-commit: 98b1cc82c4affc16f5598d4fa14b1858671b2263
-- 
2.17.1