diff options
| author | Lizhi Xu <lizhi.xu@windriver.com> | 2024-12-16 15:32:38 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2025-01-09 13:28:42 +0100 |
| commit | f60172b447317cb6c5e74b5601a151866269baf6 (patch) | |
| tree | b9ce0383ea7e3d12e7b4d2c22d19858d2c502ee6 /kernel | |
| parent | c5a28af7855586d3da4132687643e681b6ec1e79 (diff) | |
| download | linux-f60172b447317cb6c5e74b5601a151866269baf6.tar.gz linux-f60172b447317cb6c5e74b5601a151866269baf6.tar.bz2 linux-f60172b447317cb6c5e74b5601a151866269baf6.zip | |
tracing: Prevent bad count for tracing_cpumask_write
commit 98feccbf32cfdde8c722bc4587aaa60ee5ac33f0 upstream.
If a large count is provided, it will trigger a warning in bitmap_parse_user.
Also check zero for it.
Cc: stable@vger.kernel.org
Fixes: 9e01c1b74c953 ("cpumask: convert kernel trace functions")
Link: https://lore.kernel.org/20241216073238.2573704-1-lizhi.xu@windriver.com
Reported-by: syzbot+0aecfd34fb878546f3fd@syzkaller.appspotmail.com
Closes: https://syzkaller.appspot.com/bug?extid=0aecfd34fb878546f3fd
Tested-by: syzbot+0aecfd34fb878546f3fd@syzkaller.appspotmail.com
Signed-off-by: Lizhi Xu <lizhi.xu@windriver.com>
Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/trace/trace.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index b199b0c7cba0..3727a926b7fa 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -5250,6 +5250,9 @@ tracing_cpumask_write(struct file *filp, const char __user *ubuf, cpumask_var_t tracing_cpumask_new; int err; + if (count == 0 || count > KMALLOC_MAX_SIZE) + return -EINVAL; + if (!zalloc_cpumask_var(&tracing_cpumask_new, GFP_KERNEL)) return -ENOMEM; |