diff options
| author |   Andrii Nakryiko <andrii@kernel.org> | 2025-02-19 16:28:21 -0800 |
|---|---|---|
| committer |   Alexei Starovoitov <ast@kernel.org> | 2025-02-20 18:42:16 -0800 |
| commit | e0525cd72b5979d8089fe524a071ea93fd011dc9 (patch) | |
| tree | 29c9460cbec5397611a605f26bd661ce0895e1c4 /tools/lib/bpf/linker.c | |
| parent | bpf: Use preempt_count() directly in bpf_send_signal_common() (diff) | |
| download | wireguard-linux-e0525cd72b5979d8089fe524a071ea93fd011dc9.tar.xz wireguard-linux-e0525cd72b5979d8089fe524a071ea93fd011dc9.zip | |
libbpf: Fix hypothetical STT_SECTION extern NULL deref case
Fix theoretical NULL dereference in linker when resolving *extern*
STT_SECTION symbol against not-yet-existing ELF section. Not sure if
it's possible in practice for valid ELF object files (this would require
embedded assembly manipulations, at which point BTF will be missing),
but fix the s/dst_sym/dst_sec/ typo guarding this condition anyways.
Fixes: faf6ed321cf6 ("libbpf: Add BPF static linker APIs")
Fixes: a46349227cd8 ("libbpf: Add linker extern resolution support for functions and global variables")
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Link: https://lore.kernel.org/r/20250220002821.834400-1-andrii@kernel.org
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'tools/lib/bpf/linker.c')
| -rw-r--r-- | tools/lib/bpf/linker.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/lib/bpf/linker.c b/tools/lib/bpf/linker.c index b52f71c59616..800e0ef09c37 100644 --- a/tools/lib/bpf/linker.c +++ b/tools/lib/bpf/linker.c @@ -2163,7 +2163,7 @@ add_sym: obj->sym_map[src_sym_idx] = dst_sym_idx; - if (sym_type == STT_SECTION && dst_sym) { + if (sym_type == STT_SECTION && dst_sec) { dst_sec->sec_sym_idx = dst_sym_idx; dst_sym->st_value = 0; } |