Chen Zhongjin
8b355d6b1f
profiling: fix shift too large makes kernel panic
[ Upstream commit 0fe6ee8f123a4dfb529a5aff07536bb481f34043 ]
2d186afd04d6 ("profiling: fix shift-out-of-bounds bugs") limits shift
value by [0, BITS_PER_LONG -1], which means [0, 63].
However, syzbot found that the max shift value should be the bit number of
(_etext - _stext). If shift is outside of this, the "buffer_bytes" will
be zero and will cause kzalloc(0). Then the kernel panics due to
dereferencing the returned pointer 16.
This can be easily reproduced by passing a large number like 60 to enable
profiling and then run readprofile.
LOGS:
BUG: kernel NULL pointer dereference, address: 0000000000000010
#PF: supervisor write access in kernel mode
#PF: error_code(0x0002) - not-present page
PGD 6148067 P4D 6148067 PUD 6142067 PMD 0
PREEMPT SMP
CPU: 4 PID: 184 Comm: readprofile Not tainted 5.18.0+ #162
Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS rel-1.15.0-0-g2dd4b9b3f840-prebuilt.qemu.org 04/01/2014
RIP: 0010:read_profile+0x104/0x220
RSP: 0018:ffffc900006fbe80 EFLAGS: 00000202
RAX: 0000000000000000 RBX: 0000000000000000 RCX: 0000000000000000
RDX: ffff888006150000 RSI: 0000000000000001 RDI: ffffffff82aba4a0
RBP: 000000000188bb60 R08: 0000000000000010 R09: ffff888006151000
R10: 0000000000000000 R11: 0000000000000000 R12: ffffffff82aba4a0
R13: 0000000000000000 R14: ffffc900006fbf08 R15: 0000000000020c30
FS: 000000000188a8c0(0000) GS:ffff88803ed00000(0000) knlGS:0000000000000000
CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
CR2: 0000000000000010 CR3: 0000000006144000 CR4: 00000000000006e0
Call Trace:
<TASK>
proc_reg_read+0x56/0x70
vfs_read+0x9a/0x1b0
ksys_read+0xa1/0xe0
? fpregs_assert_state_consistent+0x1e/0x40
do_syscall_64+0x3a/0x80
entry_SYSCALL_64_after_hwframe+0x46/0xb0
RIP: 0033:0x4d4b4e
RSP: 002b:00007ffebb668d58 EFLAGS: 00000246 ORIG_RAX: 0000000000000000
RAX: ffffffffffffffda RBX: 000000000188a8a0 RCX: 00000000004d4b4e
RDX: 0000000000000400 RSI: 000000000188bb60 RDI: 0000000000000003
RBP: 0000000000000003 R08: 000000000000006e R09: 0000000000000000
R10: 0000000000000041 R11: 0000000000000246 R12: 000000000188bb60
R13: 0000000000000400 R14: 0000000000000000 R15: 000000000188bb60
</TASK>
Modules linked in:
CR2: 0000000000000010
Killed
---[ end trace 0000000000000000 ]---
Check prof_len in profile_init() to prevent it be zero.
Link: https://lkml.kernel.org/r/20220531012854.229439-1-chenzhongjin@huawei.com
Fixes: 1da177e4c3 ("Linux-2.6.12-rc2")
Signed-off-by: Chen Zhongjin <chenzhongjin@huawei.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
|
2022-08-25 11:18:02 +02:00 |
| .. |
|
bpf
|
bpf: Verifer, adjust_scalar_min_max_vals to always call update_reg_bounds()
|
2022-08-11 12:57:51 +02:00 |
|
cgroup
|
cgroup: Use separate src/dst nodes when preloading css_sets for migration
|
2022-07-21 20:59:17 +02:00 |
|
configs
|
|
|
|
debug
|
lockdown: also lock down previous kgdb use
|
2022-06-06 08:33:48 +02:00 |
|
dma
|
dma-debug: make things less spammy under memory pressure
|
2022-06-22 14:11:19 +02:00 |
|
events
|
perf/core: Fix data race between perf_event_set_output() and perf_mmap_close()
|
2022-07-29 17:14:10 +02:00 |
|
gcov
|
gcov: re-fix clang-11+ support
|
2021-04-14 08:24:10 +02:00 |
|
irq
|
genirq: Don't return error on missing optional irq_request_resources()
|
2022-08-25 11:17:29 +02:00 |
|
livepatch
|
|
|
|
locking
|
locking/lockdep: Avoid RCU-induced noinstr fail
|
2021-11-17 09:48:28 +01:00 |
|
power
|
PM: hibernate: defer device probing when resuming from hibernation
|
2022-08-25 11:17:32 +02:00 |
|
printk
|
printk: fix return value of printk.devkmsg __setup handler
|
2022-04-15 14:18:08 +02:00 |
|
rcu
|
rcu: Don't deboost before reporting expedited quiescent state
|
2022-03-28 08:46:48 +02:00 |
|
sched
|
nohz/full, sched/rt: Fix missed tick-reenabling bug in dequeue_task_rt()
|
2022-08-25 11:17:36 +02:00 |
|
time
|
timekeeping: Add raw clock fallback for random_get_entropy()
|
2022-06-22 14:11:15 +02:00 |
|
trace
|
tracing/histograms: Fix memory leak problem
|
2022-07-21 20:59:16 +02:00 |
|
.gitignore
|
kbuild: update config_data.gz only when the content of .config is changed
|
2021-05-11 14:04:16 +02:00 |
|
acct.c
|
|
|
|
async.c
|
Revert "module, async: async_synchronize_full() on module init iff async is used"
|
2022-02-23 11:59:56 +01:00 |
|
audit.c
|
audit: improve audit queue handling when "audit=1" on cmdline
|
2022-02-08 18:24:26 +01:00 |
|
audit.h
|
audit: log AUDIT_TIME_* records only from rules
|
2022-04-15 14:18:04 +02:00 |
|
audit_fsnotify.c
|
|
|
|
audit_tree.c
|
audit: move put_tree() to avoid trim_trees refcount underflow and UAF
|
2021-09-03 10:08:16 +02:00 |
|
audit_watch.c
|
audit: CONFIG_CHANGE don't log internal bookkeeping as an event
|
2020-10-01 13:17:32 +02:00 |
|
auditfilter.c
|
|
|
|
auditsc.c
|
audit: log AUDIT_TIME_* records only from rules
|
2022-04-15 14:18:04 +02:00 |
|
backtracetest.c
|
|
|
|
bounds.c
|
|
|
|
capability.c
|
|
|
|
compat.c
|
|
|
|
configs.c
|
|
|
|
context_tracking.c
|
|
|
|
cpu.c
|
random: clear fast pool, crng, and batches in cpuhp bring up
|
2022-06-22 14:11:12 +02:00 |
|
cpu_pm.c
|
|
|
|
crash_core.c
|
|
|
|
crash_dump.c
|
|
|
|
cred.c
|
|
|
|
delayacct.c
|
|
|
|
dma.c
|
|
|
|
exec_domain.c
|
|
|
|
exit.c
|
don't dump the threads that had been already exiting when zapped.
|
2020-11-18 19:20:31 +01:00 |
|
extable.c
|
|
|
|
fail_function.c
|
fail_function: Remove a redundant mutex unlock
|
2020-11-24 13:29:18 +01:00 |
|
fork.c
|
copy_process(): Move fd_install() out of sighand->siglock critical section
|
2022-02-23 12:00:00 +01:00 |
|
freezer.c
|
|
|
|
futex.c
|
mm, futex: fix shared futex pgoff on shmem huge page
|
2021-06-30 08:47:55 -04:00 |
|
gen_kheaders.sh
|
kbuild: add variables for compression tools
|
2020-09-03 11:27:10 +02:00 |
|
groups.c
|
|
|
|
hung_task.c
|
|
|
|
iomem.c
|
|
|
|
irq_work.c
|
|
|
|
jump_label.c
|
|
|
|
kallsyms.c
|
|
|
|
kcmp.c
|
exec: Transform exec_update_mutex into a rw_semaphore
|
2021-01-09 13:44:55 +01:00 |
|
Kconfig.freezer
|
|
|
|
Kconfig.hz
|
|
|
|
Kconfig.locks
|
|
|
|
Kconfig.preempt
|
|
|
|
kcov.c
|
|
|
|
kexec.c
|
|
|
|
kexec_core.c
|
kernel: kexec: remove the lock operation of system_transition_mutex
|
2021-02-03 23:25:56 +01:00 |
|
kexec_elf.c
|
|
|
|
kexec_file.c
|
kexec_file: drop weak attribute from arch_kexec_apply_relocations[_add]
|
2022-07-02 16:28:50 +02:00 |
|
kexec_internal.h
|
|
|
|
kheaders.c
|
|
|
|
kmod.c
|
|
|
|
kprobes.c
|
kprobes: Limit max data_size of the kretprobe instances
|
2021-12-08 09:01:10 +01:00 |
|
ksysfs.c
|
|
|
|
kthread.c
|
kthread: Fix PF_KTHREAD vs to_kthread() race
|
2021-09-12 08:56:39 +02:00 |
|
latencytop.c
|
|
|
|
Makefile
|
kbuild: update config_data.gz only when the content of .config is changed
|
2021-05-11 14:04:16 +02:00 |
|
module-internal.h
|
|
|
|
module.c
|
module/ftrace: handle patchable-function-entry
|
2022-02-23 11:59:56 +01:00 |
|
module_signature.c
|
module: harden ELF info handling
|
2021-04-07 14:47:38 +02:00 |
|
module_signing.c
|
module: harden ELF info handling
|
2021-04-07 14:47:38 +02:00 |
|
notifier.c
|
kernel/notifier.c: intercept duplicate registrations to avoid infinite loops
|
2020-10-01 13:17:23 +02:00 |
|
nsproxy.c
|
|
|
|
padata.c
|
|
|
|
panic.c
|
|
|
|
params.c
|
|
|
|
pid.c
|
|
|
|
pid_namespace.c
|
memcg: enable accounting for pids in nested pid namespaces
|
2021-09-22 12:26:37 +02:00 |
|
profile.c
|
profiling: fix shift too large makes kernel panic
|
2022-08-25 11:18:02 +02:00 |
|
ptrace.c
|
ptrace: Reimplement PTRACE_KILL by always sending SIGKILL
|
2022-06-14 18:11:24 +02:00 |
|
range.c
|
|
|
|
reboot.c
|
reboot: fix overflow parsing reboot cpu number
|
2020-11-18 19:20:30 +01:00 |
|
relay.c
|
kernel/relay.c: fix memleak on destroy relay channel
|
2020-08-26 10:40:51 +02:00 |
|
resource.c
|
|
|
|
rseq.c
|
|
|
|
seccomp.c
|
seccomp: Invalidate seccomp mode to catch death failures
|
2022-02-16 12:52:53 +01:00 |
|
signal.c
|
signal handling: don't use BUG_ON() for debugging
|
2022-07-21 20:59:27 +02:00 |
|
smp.c
|
smp: Fix offline cpu check in flush_smp_call_function_queue()
|
2022-04-20 09:19:39 +02:00 |
|
smpboot.c
|
kthread: Extract KTHREAD_IS_PER_CPU
|
2021-02-07 15:35:49 +01:00 |
|
smpboot.h
|
|
|
|
softirq.c
|
|
|
|
stackleak.c
|
|
|
|
stacktrace.c
|
|
|
|
stop_machine.c
|
|
|
|
sys.c
|
prctl: allow to setup brk for et_dyn executables
|
2021-09-26 14:07:08 +02:00 |
|
sys_ni.c
|
|
|
|
sysctl-test.c
|
kernel/sysctl-test: Add null pointer test for sysctl.c:proc_dointvec()
|
2020-10-01 13:17:10 +02:00 |
|
sysctl.c
|
mm: sysctl: fix missing numa_stat when !CONFIG_HUGETLB_PAGE
|
2022-07-21 20:59:24 +02:00 |
|
sysctl_binary.c
|
|
|
|
task_work.c
|
|
|
|
taskstats.c
|
|
|
|
test_kprobes.c
|
|
|
|
torture.c
|
|
|
|
tracepoint.c
|
tracepoint: Add tracepoint_probe_register_may_exist() for BPF tracing
|
2021-07-14 16:53:08 +02:00 |
|
tsacct.c
|
taskstats: Cleanup the use of task->exit_code
|
2022-02-23 11:59:57 +01:00 |
|
ucount.c
|
|
|
|
uid16.c
|
|
|
|
uid16.h
|
|
|
|
umh.c
|
usermodehelper: reset umask to default before executing user process
|
2020-10-14 10:32:58 +02:00 |
|
up.c
|
smp: Fix smp_call_function_single_async prototype
|
2021-05-14 09:44:33 +02:00 |
|
user-return-notifier.c
|
|
|
|
user.c
|
|
|
|
user_namespace.c
|
|
|
|
utsname.c
|
|
|
|
utsname_sysctl.c
|
|
|
|
watchdog.c
|
|
|
|
watchdog_hld.c
|
|
|
|
workqueue.c
|
workqueue: Fix unbind_workers() VS wq_worker_running() race
|
2022-01-16 09:15:38 +01:00 |
|
workqueue_internal.h
|
|
|