bpf: Fix re-attachment branch in bpf_tracing_prog_attach
commit 715d82ba upstream. The following case can cause a crash due to missing attach_btf: 1) load rawtp program 2) load fentry program with rawtp as target_fd 3) create tracing link for fentry program with target_fd = 0 4) repeat 3 In the end we have: - prog->aux->dst_trampoline == NULL - tgt_prog == NULL (because we did not provide target_fd to link_create) - prog->aux->attach_btf == NULL (the program was loaded with attach_prog_fd=X) - the program was loaded for tgt_prog but we have no way to find out which one BUG: kernel NULL pointer dereference, address: 0000000000000058 Call Trace: <TASK> ? __die+0x20/0x70 ? page_fault_oops+0x15b/0x430 ? fixup_exception+0x22/0x330 ? exc_page_fault+0x6f/0x170 ? asm_exc_page_fault+0x22/0x30 ? bpf_tracing_prog_attach+0x279/0x560 ? btf_obj_id+0x5/0x10 bpf_tracing_prog_attach+0x439/0x560 __sys_bpf+0x1cf4/0x2de0 __x64_sys_bpf+0x1c/0x30 do_syscall_64+0x41/0xf0 entry_SYSCALL_64_after_hwframe+0x6e/0x76 Return -EINVAL in this situation. Fixes: f3a95075 ("bpf: Allow trampoline re-attach for tracing and lsm programs") Cc: stable@vger.kernel.org Signed-off-by:Jiri Olsa <olsajiri@gmail.com> Acked-by:
Jiri Olsa <olsajiri@gmail.com> Acked-by:
Song Liu <song@kernel.org> Signed-off-by:
Dmitrii Dolgov <9erthalion6@gmail.com> Link: https://lore.kernel.org/r/20240103190559.14750-4-9erthalion6@gmail.com Signed-off-by:
Alexei Starovoitov <ast@kernel.org> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Loading
-
mentioned in commit 0a636b1a
-
mentioned in commit 5f12c91a
-
mentioned in commit 5f12c91a
-
mentioned in commit 5f12c91a
-
mentioned in commit e7de7e89
-
mentioned in commit d2e96977
-
mentioned in commit d66cd9d4
-
mentioned in commit d7fac609
-
mentioned in commit 70422d08
-
mentioned in commit cdb444a3
-
mentioned in commit 79167160
-
mentioned in commit 41a429ab
-
mentioned in commit 98d73821
-
mentioned in commit 47f30bf7
-
mentioned in commit 90bc8306
-
mentioned in commit 0a4b3a76
-
mentioned in commit a71e93db
-
mentioned in commit fa7914b2
-
mentioned in commit 8bb8a52b
Please sign in to comment