mirror of
https://github.com/torvalds/linux.git
synced 2026-03-08 06:04:44 +01:00
Commitbf454ec31a("kexec_file: allow to place kexec_buf randomly") introduced the kexec_buf->random field to enable random placement of kexec_buf. However, this field was never properly initialized for kexec images that do not need to be placed randomly, leading to the following UBSAN warning: [ +0.364528] ------------[ cut here ]------------ [ +0.000019] UBSAN: invalid-load in ./include/linux/kexec.h:210:12 [ +0.000131] load of value 2 is not a valid value for type 'bool' (aka '_Bool') [ +0.000003] CPU: 4 UID: 0 PID: 927 Comm: kexec Not tainted 6.18.0-rc7+ #3 PREEMPT(full) [ +0.000002] Hardware name: QEMU QEMU Virtual Machine, BIOS 0.0.0 02/06/2015 [ +0.000000] Call trace: [ +0.000001] show_stack+0x24/0x40 (C) [ +0.000006] __dump_stack+0x28/0x48 [ +0.000002] dump_stack_lvl+0x7c/0xb0 [ +0.000002] dump_stack+0x18/0x34 [ +0.000001] ubsan_epilogue+0x10/0x50 [ +0.000002] __ubsan_handle_load_invalid_value+0xc8/0xd0 [ +0.000003] locate_mem_hole_callback+0x28c/0x2a0 [ +0.000003] kexec_locate_mem_hole+0xf4/0x2f0 [ +0.000001] kexec_add_buffer+0xa8/0x178 [ +0.000002] image_load+0xf0/0x258 [ +0.000001] __arm64_sys_kexec_file_load+0x510/0x718 [ +0.000002] invoke_syscall+0x68/0xe8 [ +0.000001] el0_svc_common+0xb0/0xf8 [ +0.000002] do_el0_svc+0x28/0x48 [ +0.000001] el0_svc+0x40/0xe8 [ +0.000002] el0t_64_sync_handler+0x84/0x140 [ +0.000002] el0t_64_sync+0x1bc/0x1c0 To address this, initialise kexec_buf->random field properly. Fixes:bf454ec31a("kexec_file: allow to place kexec_buf randomly") Suggested-by: Breno Leitao <leitao@debian.org> Cc: stable@vger.kernel.org Signed-off-by: Yeoreum Yun <yeoreum.yun@arm.com> Reviewed-by: Breno Leitao <leitao@debian.org> Link: https://lore.kernel.org/all/oninomspajhxp4omtdapxnckxydbk2nzmrix7rggmpukpnzadw@c67o7njgdgm3/ [1] Link: https://lore.kernel.org/all/20250825180531.94bfb86a26a43127c0a1296f@linux-foundation.org/ [2] Link: https://lkml.kernel.org/r/20250826-akpm-v1-1-3c831f0e3799@debian.org Signed-off-by: Breno Leitao <leitao@debian.org> Suggested-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Reviewed-by: Pratyush Yadav <pratyush@kernel.org> Signed-off-by: Will Deacon <will@kernel.org>
138 lines
3.8 KiB
C
138 lines
3.8 KiB
C
// SPDX-License-Identifier: GPL-2.0
|
|
/*
|
|
* Kexec image loader
|
|
|
|
* Copyright (C) 2018 Linaro Limited
|
|
* Author: AKASHI Takahiro <takahiro.akashi@linaro.org>
|
|
*/
|
|
|
|
#define pr_fmt(fmt) "kexec_file(Image): " fmt
|
|
|
|
#include <linux/err.h>
|
|
#include <linux/errno.h>
|
|
#include <linux/kernel.h>
|
|
#include <linux/kexec.h>
|
|
#include <linux/pe.h>
|
|
#include <linux/string.h>
|
|
#include <asm/byteorder.h>
|
|
#include <asm/cpufeature.h>
|
|
#include <asm/image.h>
|
|
#include <asm/memory.h>
|
|
|
|
static int image_probe(const char *kernel_buf, unsigned long kernel_len)
|
|
{
|
|
const struct arm64_image_header *h =
|
|
(const struct arm64_image_header *)(kernel_buf);
|
|
|
|
if (!h || (kernel_len < sizeof(*h)))
|
|
return -EINVAL;
|
|
|
|
if (memcmp(&h->magic, ARM64_IMAGE_MAGIC, sizeof(h->magic)))
|
|
return -EINVAL;
|
|
|
|
return 0;
|
|
}
|
|
|
|
static void *image_load(struct kimage *image,
|
|
char *kernel, unsigned long kernel_len,
|
|
char *initrd, unsigned long initrd_len,
|
|
char *cmdline, unsigned long cmdline_len)
|
|
{
|
|
struct arm64_image_header *h;
|
|
u64 flags, value;
|
|
bool be_image, be_kernel;
|
|
struct kexec_buf kbuf = {};
|
|
unsigned long text_offset, kernel_segment_number;
|
|
struct kexec_segment *kernel_segment;
|
|
int ret;
|
|
|
|
/*
|
|
* We require a kernel with an unambiguous Image header. Per
|
|
* Documentation/arch/arm64/booting.rst, this is the case when image_size
|
|
* is non-zero (practically speaking, since v3.17).
|
|
*/
|
|
h = (struct arm64_image_header *)kernel;
|
|
if (!h->image_size)
|
|
return ERR_PTR(-EINVAL);
|
|
|
|
/* Check cpu features */
|
|
flags = le64_to_cpu(h->flags);
|
|
be_image = arm64_image_flag_field(flags, ARM64_IMAGE_FLAG_BE);
|
|
be_kernel = IS_ENABLED(CONFIG_CPU_BIG_ENDIAN);
|
|
if ((be_image != be_kernel) && !system_supports_mixed_endian())
|
|
return ERR_PTR(-EINVAL);
|
|
|
|
value = arm64_image_flag_field(flags, ARM64_IMAGE_FLAG_PAGE_SIZE);
|
|
if (((value == ARM64_IMAGE_FLAG_PAGE_SIZE_4K) &&
|
|
!system_supports_4kb_granule()) ||
|
|
((value == ARM64_IMAGE_FLAG_PAGE_SIZE_64K) &&
|
|
!system_supports_64kb_granule()) ||
|
|
((value == ARM64_IMAGE_FLAG_PAGE_SIZE_16K) &&
|
|
!system_supports_16kb_granule()))
|
|
return ERR_PTR(-EINVAL);
|
|
|
|
/* Load the kernel */
|
|
kbuf.image = image;
|
|
kbuf.buf_min = 0;
|
|
kbuf.buf_max = ULONG_MAX;
|
|
kbuf.top_down = false;
|
|
|
|
kbuf.buffer = kernel;
|
|
kbuf.bufsz = kernel_len;
|
|
kbuf.mem = KEXEC_BUF_MEM_UNKNOWN;
|
|
kbuf.memsz = le64_to_cpu(h->image_size);
|
|
text_offset = le64_to_cpu(h->text_offset);
|
|
kbuf.buf_align = MIN_KIMG_ALIGN;
|
|
|
|
/* Adjust kernel segment with TEXT_OFFSET */
|
|
kbuf.memsz += text_offset;
|
|
|
|
kernel_segment_number = image->nr_segments;
|
|
|
|
/*
|
|
* The location of the kernel segment may make it impossible to satisfy
|
|
* the other segment requirements, so we try repeatedly to find a
|
|
* location that will work.
|
|
*/
|
|
while ((ret = kexec_add_buffer(&kbuf)) == 0) {
|
|
/* Try to load additional data */
|
|
kernel_segment = &image->segment[kernel_segment_number];
|
|
ret = load_other_segments(image, kernel_segment->mem,
|
|
kernel_segment->memsz, initrd,
|
|
initrd_len, cmdline);
|
|
if (!ret)
|
|
break;
|
|
|
|
/*
|
|
* We couldn't find space for the other segments; erase the
|
|
* kernel segment and try the next available hole.
|
|
*/
|
|
image->nr_segments -= 1;
|
|
kbuf.buf_min = kernel_segment->mem + kernel_segment->memsz;
|
|
kbuf.mem = KEXEC_BUF_MEM_UNKNOWN;
|
|
}
|
|
|
|
if (ret) {
|
|
pr_err("Could not find any suitable kernel location!");
|
|
return ERR_PTR(ret);
|
|
}
|
|
|
|
kernel_segment = &image->segment[kernel_segment_number];
|
|
kernel_segment->mem += text_offset;
|
|
kernel_segment->memsz -= text_offset;
|
|
image->start = kernel_segment->mem;
|
|
|
|
kexec_dprintk("Loaded kernel at 0x%lx bufsz=0x%lx memsz=0x%lx\n",
|
|
kernel_segment->mem, kbuf.bufsz,
|
|
kernel_segment->memsz);
|
|
|
|
return NULL;
|
|
}
|
|
|
|
const struct kexec_file_ops kexec_image_ops = {
|
|
.probe = image_probe,
|
|
.load = image_load,
|
|
#ifdef CONFIG_KEXEC_IMAGE_VERIFY_SIG
|
|
.verify_sig = kexec_kernel_verify_pe_sig,
|
|
#endif
|
|
};
|