This is a note to let you know that I've just added the patch titled
arm64/sve: Use correct size when reinitialising SVE state
to the 5.4-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
The filename of the patch is:
arm64-sve-use-correct-size-when-reinitialising-sve-state.patch
and it can be found in the queue-5.4 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.
>From e35ac9d0b56e9efefaeeb84b635ea26c2839ea86 Mon Sep 17 00:00:00 2001
From: Mark Brown <broonie@xxxxxxxxxx>
Date: Thu, 9 Sep 2021 17:53:56 +0100
Subject: arm64/sve: Use correct size when reinitialising SVE state
From: Mark Brown <broonie@xxxxxxxxxx>
commit e35ac9d0b56e9efefaeeb84b635ea26c2839ea86 upstream.
When we need a buffer for SVE register state we call sve_alloc() to make
sure that one is there. In order to avoid repeated allocations and frees
we keep the buffer around unless we change vector length and just memset()
it to ensure a clean register state. The function that deals with this
takes the task to operate on as an argument, however in the case where we
do a memset() we initialise using the SVE state size for the current task
rather than the task passed as an argument.
This is only an issue in the case where we are setting the register state
for a task via ptrace and the task being configured has a different vector
length to the task tracing it. In the case where the buffer is larger in
the traced process we will leak old state from the traced process to
itself, in the case where the buffer is smaller in the traced process we
will overflow the buffer and corrupt memory.
Fixes: bc0ee4760364 ("arm64/sve: Core task context handling")
Cc: <stable@xxxxxxxxxxxxxxx> # 4.15.x
Signed-off-by: Mark Brown <broonie@xxxxxxxxxx>
Link: https://lore.kernel.org/r/20210909165356.10675-1-broonie@xxxxxxxxxx
Signed-off-by: Catalin Marinas <catalin.marinas@xxxxxxx>
Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
---
arch/arm64/kernel/fpsimd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- a/arch/arm64/kernel/fpsimd.c
+++ b/arch/arm64/kernel/fpsimd.c
@@ -498,7 +498,7 @@ size_t sve_state_size(struct task_struct
void sve_alloc(struct task_struct *task)
{
if (task->thread.sve_state) {
- memset(task->thread.sve_state, 0, sve_state_size(current));
+ memset(task->thread.sve_state, 0, sve_state_size(task));
return;
}
Patches currently in stable-queue which might be from broonie@xxxxxxxxxx are
queue-5.4/asoc-atmel-atmel-drivers-don-t-need-has_dma.patch
queue-5.4/asoc-intel-bytcr_rt5640-move-platform-clock-routes-t.patch
queue-5.4/asoc-rockchip-i2s-fix-regmap_ops-hang.patch
queue-5.4/asoc-rockchip-i2s-fixup-config-for-daifmt_dsp_a-b.patch
queue-5.4/asoc-intel-skylake-fix-passing-loadable-flag-for-mod.patch
queue-5.4/arm64-sve-use-correct-size-when-reinitialising-sve-state.patch
queue-5.4/asoc-intel-skylake-fix-module-configuration-for-kpb-.patch
