[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20251105112204.4b0ab3d9@gandalf.local.home>
Date: Wed, 5 Nov 2025 11:22:04 -0500
From: Steven Rostedt <rostedt@...dmis.org>
To: Petr Mladek <pmladek@...e.com>
Cc: Petr Pavlu <petr.pavlu@...e.com>, Alexei Starovoitov <ast@...nel.org>,
Andrew Morton <akpm@...ux-foundation.org>, Kees Cook <kees@...nel.org>,
Daniel Borkmann <daniel@...earbox.net>, John Fastabend
<john.fastabend@...il.com>, Masami Hiramatsu <mhiramat@...nel.org>, Mark
Rutland <mark.rutland@....com>, Luis Chamberlain <mcgrof@...nel.org>,
Daniel Gomez <da.gomez@...nel.org>, Sami Tolvanen
<samitolvanen@...gle.com>, linux-kernel@...r.kernel.org,
bpf@...r.kernel.org, linux-modules@...r.kernel.org,
linux-trace-kernel@...r.kernel.org
Subject: Re: [PATCH 4/6] kallsyms/ftrace: Set module buildid in
ftrace_mod_address_lookup()
On Wed, 5 Nov 2025 15:23:16 +0100
Petr Mladek <pmladek@...e.com> wrote:
> __sprint_symbol() might access an invalid pointer when
> kallsyms_lookup_buildid() returns a symbol found by
> ftrace_mod_address_lookup().
>
> The ftrace lookup function must set both @modname and @modbuildid
> the same way as module_address_lookup().
>
> Fixes: 9294523e3768 ("module: add printk formats to add module build ID to stacktraces")
> Signed-off-by: Petr Mladek <pmladek@...e.com>
> ---
> include/linux/ftrace.h | 6 ++++--
> kernel/kallsyms.c | 4 ++--
> kernel/trace/ftrace.c | 5 ++++-
> 3 files changed, 10 insertions(+), 5 deletions(-)
Acked-by: Steven Rostedt (Google) <rostedt@...dmis.org>
-- Steve
Powered by blists - more mailing lists