libbpf: handle nulled-out program in struct_ops correctly
authorAndrii Nakryiko <andrii@kernel.org>
Sun, 28 Apr 2024 03:09:53 +0000 (20:09 -0700)
committerMartin KaFai Lau <martin.lau@kernel.org>
Mon, 29 Apr 2024 23:46:06 +0000 (16:46 -0700)
commitf973fccd43d34b096077d5d21d051ef75b22a7ea
tree9cd7d6ace7c93aedc960a429e91ed40d45e8bcf4
parentcfd3bfe9507b4aa39f7e86772e60b50b799e490e
libbpf: handle nulled-out program in struct_ops correctly

If struct_ops has one of program callbacks set declaratively and host
kernel is old and doesn't support this callback, libbpf will allow to
load such struct_ops as long as that callback was explicitly nulled-out
(presumably through skeleton). This is all working correctly, except we
won't reset corresponding program slot to NULL before bailing out, which
will lead to libbpf not detecting that BPF program has to be not
auto-loaded. Fix this by unconditionally resetting corresponding program
slot to NULL.

Fixes: c911fc61a7ce ("libbpf: Skip zeroed or null fields if not found in the kernel type.")
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Link: https://lore.kernel.org/r/20240428030954.3918764-1-andrii@kernel.org
Signed-off-by: Martin KaFai Lau <martin.lau@kernel.org>
tools/lib/bpf/libbpf.c