bpf: Resolve fext program type when checking map compatibility
authorToke Høiland-Jørgensen <toke@redhat.com>
Wed, 14 Dec 2022 23:02:53 +0000 (00:02 +0100)
committerMartin KaFai Lau <martin.lau@kernel.org>
Thu, 15 Dec 2022 05:30:27 +0000 (21:30 -0800)
commit1c123c567fb138ebd187480b7fc0610fcb0851f5
tree3d2be7bf2881d780405ed73b2db99eaff7c55828
parent4121d4481b72501aa4d22680be4ea1096d69d133
bpf: Resolve fext program type when checking map compatibility

The bpf_prog_map_compatible() check makes sure that BPF program types are
not mixed inside BPF map types that can contain programs (tail call maps,
cpumaps and devmaps). It does this by setting the fields of the map->owner
struct to the values of the first program being checked against, and
rejecting any subsequent programs if the values don't match.

One of the values being set in the map owner struct is the program type,
and since the code did not resolve the prog type for fext programs, the map
owner type would be set to PROG_TYPE_EXT and subsequent loading of programs
of the target type into the map would fail.

This bug is seen in particular for XDP programs that are loaded as
PROG_TYPE_EXT using libxdp; these cannot insert programs into devmaps and
cpumaps because the check fails as described above.

Fix the bug by resolving the fext program type to its target program type
as elsewhere in the verifier.

v3:
- Add Yonghong's ACK

Fixes: f45d5b6ce2e8 ("bpf: generalise tail call map compatibility check")
Acked-by: Yonghong Song <yhs@fb.com>
Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com>
Link: https://lore.kernel.org/r/20221214230254.790066-1-toke@redhat.com
Signed-off-by: Martin KaFai Lau <martin.lau@kernel.org>
kernel/bpf/core.c