mirror of
https://github.com/linux-msm/laptops-kernel.git
synced 2026-08-13 14:19:53 -07:00
tracing/probes: Reject $arg0 in meta argument expansion
traceprobe_expand_meta_args() parses $argN with simple_strtoul() and
calls sprint_nth_btf_arg(n - 1, ...). For $arg0, n is 0 so the index is
-1. Because ctx->nr_params is signed, the "idx >= nr_params" guard in
sprint_nth_btf_arg() does not catch the negative index, and
ctx->params[-1].name_off is read out of bounds.
The normal per-argument path (parse_probe_vars()) already rejects
$arg0 via its argument-number check, but meta-argument expansion runs
before per-argument parsing and substitutes the value first, bypassing
that check.
Reject $arg0 explicitly during expansion.
Link: https://lore.kernel.org/all/20260724054435.146279-1-raushan.jhon@gmail.com/
Fixes: 18b1e870a4 ("tracing/probes: Add $arg* meta argument for all function args")
Cc: stable@vger.kernel.org
Signed-off-by: Raushan Patel <raushan.jhon@gmail.com>
Signed-off-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
This commit is contained in:
committed by
Masami Hiramatsu (Google)
parent
f5098b6bae
commit
00a8ce2a2a
@@ -1901,7 +1901,11 @@ const char **traceprobe_expand_meta_args(int argc, const char *argv[],
|
||||
trace_probe_log_err(0, BAD_VAR);
|
||||
return ERR_PTR(-ENOENT);
|
||||
}
|
||||
/* Note: $argN starts from $arg1 */
|
||||
/* Note: $argN starts from $arg1, so $arg0 is invalid. */
|
||||
if (n == 0) {
|
||||
trace_probe_log_err(0, BAD_ARG_NUM);
|
||||
return ERR_PTR(-EINVAL);
|
||||
}
|
||||
ret = sprint_nth_btf_arg(n - 1, type, buf + used,
|
||||
bufsize - used, ctx);
|
||||
if (ret < 0)
|
||||
|
||||
Reference in New Issue
Block a user