From: Hao Luo Date: Tue, 22 Aug 2023 19:38:40 +0000 (-0700) Subject: libbpf: Free btf_vmlinux when closing bpf_object X-Git-Tag: v6.1.61~939 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d5feaef143b6217657f07e4d9f13a2a2da6a4ee1;p=platform%2Fkernel%2Flinux-starfive.git libbpf: Free btf_vmlinux when closing bpf_object [ Upstream commit 29d67fdebc42af6466d1909c60fdd1ef4f3e5240 ] I hit a memory leak when testing bpf_program__set_attach_target(). Basically, set_attach_target() may allocate btf_vmlinux, for example, when setting attach target for bpf_iter programs. But btf_vmlinux is freed only in bpf_object_load(), which means if we only open bpf object but not load it, setting attach target may leak btf_vmlinux. So let's free btf_vmlinux in bpf_object__close() anyway. Signed-off-by: Hao Luo Signed-off-by: Andrii Nakryiko Link: https://lore.kernel.org/bpf/20230822193840.1509809-1-haoluo@google.com Signed-off-by: Sasha Levin --- diff --git a/tools/lib/bpf/libbpf.c b/tools/lib/bpf/libbpf.c index eeb2693..10f15a3 100644 --- a/tools/lib/bpf/libbpf.c +++ b/tools/lib/bpf/libbpf.c @@ -8173,6 +8173,7 @@ void bpf_object__close(struct bpf_object *obj) bpf_object__elf_finish(obj); bpf_object_unload(obj); btf__free(obj->btf); + btf__free(obj->btf_vmlinux); btf_ext__free(obj->btf_ext); for (i = 0; i < obj->nr_maps; i++)