libbpf: Fix null pointer dereference in btf_dump__free on allocation failure
[ Upstream commit aa485e8789d56a4573f7c8d000a182b749eaa64d ]
When btf_dump__new() fails to allocate memory for the internal hashmap
(btf_dump->type_names), it returns an error code. However, the cleanup
function btf_dump__free() does not check if btf_dump->type_names is NULL
before attempting to free it. This leads to a null pointer dereference
when btf_dump__free() is called on a btf_dump object.
Fixes: 351131b51c
("libbpf: add btf_dump API for BTF-to-C conversion")
Signed-off-by: Yuan Chen <chenyuan@kylinos.cn>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Link: https://lore.kernel.org/bpf/20250618011933.11423-1-chenyuan_fl@163.com
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
591f796257
commit
9b9a87f0bd
@@ -225,6 +225,9 @@ static void btf_dump_free_names(struct hashmap *map)
|
|||||||
size_t bkt;
|
size_t bkt;
|
||||||
struct hashmap_entry *cur;
|
struct hashmap_entry *cur;
|
||||||
|
|
||||||
|
if (!map)
|
||||||
|
return;
|
||||||
|
|
||||||
hashmap__for_each_entry(map, cur, bkt)
|
hashmap__for_each_entry(map, cur, bkt)
|
||||||
free((void *)cur->pkey);
|
free((void *)cur->pkey);
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user