bpf: Avoid dummy bpf_offload_netdev in __bpf_prog_dev_bound_init
authorEduard Zingerman <eddyz87@gmail.com>
Tue, 12 Sep 2023 00:55:37 +0000 (03:55 +0300)
committerMartin KaFai Lau <martin.lau@kernel.org>
Tue, 12 Sep 2023 05:06:06 +0000 (22:06 -0700)
commit1a49f4195d3498fe458a7f5ff7ec5385da70d92e
treeaf192f1718f0ecc30b641990d98ac6c2182e8e96
parenta34a9f1a19afe9c60ca0ea61dfeee63a1c2baac8
bpf: Avoid dummy bpf_offload_netdev in __bpf_prog_dev_bound_init

Fix for a bug observable under the following sequence of events:
1. Create a network device that does not support XDP offload.
2. Load a device bound XDP program with BPF_F_XDP_DEV_BOUND_ONLY flag
   (such programs are not offloaded).
3. Load a device bound XDP program with zero flags
   (such programs are offloaded).

At step (2) __bpf_prog_dev_bound_init() associates with device (1)
a dummy bpf_offload_netdev struct with .offdev field set to NULL.
At step (3) __bpf_prog_dev_bound_init() would reuse dummy struct
allocated at step (2).
However, downstream usage of the bpf_offload_netdev assumes that
.offdev field can't be NULL, e.g. in bpf_prog_offload_verifier_prep().

Adjust __bpf_prog_dev_bound_init() to require bpf_offload_netdev
with non-NULL .offdev for offloaded BPF programs.

Fixes: 2b3486bc2d23 ("bpf: Introduce device-bound XDP programs")
Reported-by: syzbot+291100dcb32190ec02a8@syzkaller.appspotmail.com
Closes: https://lore.kernel.org/bpf/000000000000d97f3c060479c4f8@google.com/
Signed-off-by: Eduard Zingerman <eddyz87@gmail.com>
Link: https://lore.kernel.org/r/20230912005539.2248244-2-eddyz87@gmail.com
Signed-off-by: Martin KaFai Lau <martin.lau@kernel.org>
kernel/bpf/offload.c