From: Rik van Riel riel@surriel.com
mainline inclusion from mainline-v5.13-rc1 commit f0362a253606e2031f8d61c74195d4d6556e12a4 category: bugfix bugzilla: https://gitee.com/openeuler/kernel/issues/I960AB CVE: NA
Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?i...
--------------------------------
The code calling ima_free_kexec_buffer runs long after the memblock allocator has already been torn down, potentially resulting in a use after free in memblock_isolate_range.
With KASAN or KFENCE, this use after free will result in a BUG from the idle task, and a subsequent kernel panic.
Switch ima_free_kexec_buffer over to memblock_free_late to avoid that issue.
Fixes: fee3ff99bc67 ("powerpc: Move arch independent ima kexec functions to drivers/of/kexec.c") Cc: stable@kernel.org Signed-off-by: Rik van Riel riel@surriel.com Suggested-by: Mike Rappoport rppt@kernel.org Link: https://lore.kernel.org/r/20230817135759.0888e5ef@imladris.surriel.com Signed-off-by: Rob Herring robh@kernel.org Conflicts: drivers/of/kexec.c Signed-off-by: GUO Zihua guozihua@huawei.com --- drivers/of/kexec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/of/kexec.c b/drivers/of/kexec.c index d8231c34e873..2d1fd5587123 100644 --- a/drivers/of/kexec.c +++ b/drivers/of/kexec.c @@ -178,7 +178,8 @@ int ima_free_kexec_buffer(void) if (ret) return ret;
- return memblock_free(addr, size); + memblock_free_late(addr, size); + return 0;
}