On Thu, Dec 04, 2025 at 03:04:23PM +0100, Jiri Olsa wrote: > On Wed, Dec 03, 2025 at 07:32:15PM -0800, Josh Poimboeuf wrote: > > EMIT1(0xC9); /* leave */ > > + if (im) > > + im->ksym.fp_end = prog - (u8 *)rw_image; > > is the null check needed? there are other places in the function that > use 'im' without that
That was a NULL pointer dereference found by BPF CI. bpf_struct_ops_prepare_trampoline() calls arch_prepare_bpf_trampoline() with NULL im. -- Josh
