Re: [PATCH bpf-next 2/4] LoongArch: BPF: Fix memory leak in bpf_jit_free

[Huacai Chen]

On Fri, Jul 24, 2026 at 4:05 PM Tiezhu Yang <yangtiezhu@xxxxxxxxxxx> wrote:
>
> Cc: Huacai Chen <chenhuacai@xxxxxxxxxx> loongarch@xxxxxxxxxxxxxxx
>
> On 2026/7/22 下午7:27, Pu Lehui wrote:
> > From: Pu Lehui <pulehui@xxxxxxxxxx>
> >
> > When bpf_int_jit_compile() is called for subprograms, it returns early
> > during the first pass (!prog->is_func || extra_pass is false), keeping
> > ctx->offset alive for the subsequent extra pass.
> >
> > If JIT compilation fails for a later subprogram, the BPF core aborts and
> > calls bpf_jit_free() to clean up the first subprogram. However,
> > bpf_jit_free() fails to free jit_data->ctx.offset, which causes a memory
> > leak of the JIT context offsets array.
> >
> > Fix this by adding the missing kvfree(jit_data->ctx.offset) in
> > bpf_jit_free().
> >
> > Fixes: 4ab17e762b34 ("LoongArch: BPF: Use BPF prog pack allocator")
> > Reported-by: Sashiko <sashiko-bot@xxxxxxxxxx>
> > Signed-off-by: Pu Lehui <pulehui@xxxxxxxxxx>
> > ---
> >   arch/loongarch/net/bpf_jit.c | 1 +
> >   1 file changed, 1 insertion(+)
> >
> > diff --git a/arch/loongarch/net/bpf_jit.c b/arch/loongarch/net/bpf_jit.c
> > index 2738b4db1165..3d3ed1677d01 100644
> > --- a/arch/loongarch/net/bpf_jit.c
> > +++ b/arch/loongarch/net/bpf_jit.c
> > @@ -2361,6 +2361,7 @@ void bpf_jit_free(struct bpf_prog *prog)
> >                */
> >               if (jit_data) {
> >                       bpf_jit_binary_pack_finalize(jit_data->ro_header, jit_data->header);
> > +                     kvfree(jit_data->ctx.offset);
> >                       kfree(jit_data);
> >               }
> >               hdr = bpf_jit_binary_pack_hdr(prog);
> >
>
> Here is the link:
> https://lore.kernel.org/bpf/20260722112754.1615201-3-pulehui@xxxxxxxxxxxxxxx/
>
> Acked-by: Tiezhu Yang <yangtiezhu@xxxxxxxxxxx>
Applied, thanks.

Huacai
>
> Thanks,
> Tiezhu
>
>