Skip to content

Commit d05ea35

Browse files
rafaelmsoaresmhiramat
authored andcommitted
fprobe: Check rethook_alloc() return in rethook initialization
Check if fp->rethook succeeded to be allocated. Otherwise, if rethook_alloc() fails, then we end up dereferencing a NULL pointer in rethook_add_node(). Link: https://lore.kernel.org/all/20221025031209.954836-1-rafaelmendsr@gmail.com/ Fixes: 5b0ab78 ("fprobe: Add exit_handler support") Cc: stable@vger.kernel.org Signed-off-by: Rafael Mendonca <rafaelmendsr@gmail.com> Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org> Acked-by: Masami Hiramatsu (Google) <mhiramat@kernel.org> Signed-off-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
1 parent 4a6f316 commit d05ea35

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

kernel/trace/fprobe.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,6 +141,8 @@ static int fprobe_init_rethook(struct fprobe *fp, int num)
141141
return -E2BIG;
142142

143143
fp->rethook = rethook_alloc((void *)fp, fprobe_exit_handler);
144+
if (!fp->rethook)
145+
return -ENOMEM;
144146
for (i = 0; i < size; i++) {
145147
struct fprobe_rethook_node *node;
146148

0 commit comments

Comments
 (0)