libbpf: Fix potential NULL dereference when parsing ELF

Fix if condition filtering empty ELF sections to prevent NULL
dereference.

Fixes: 47ea7417b074 ("libbpf: Skip empty sections in bpf_object__init_global_data_maps")
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Hao Luo <haoluo@google.com>
Link: https://lore.kernel.org/bpf/20220816001929.369487-2-andrii@kernel.org

diff --git a/tools/lib/bpf/libbpf.c b/tools/lib/bpf/libbpf.c
index 0159a43c7efd3f41fa18909fbc5ab5e43c96f96a..146d35526b87e6f9e015d36155f41b71c05ddaf9 100644 (file)
--- a/tools/lib/bpf/libbpf.c
+++ b/tools/lib/bpf/libbpf.c
@@ -1649,7 +1649,7 @@ static int bpf_object__init_global_data_maps(struct bpf_object *obj)
                sec_desc = &obj->efile.secs[sec_idx];
 
                /* Skip recognized sections with size 0. */
-               if (sec_desc->data && sec_desc->data->d_size == 0)
+               if (!sec_desc->data || sec_desc->data->d_size == 0)
                        continue;
 
                switch (sec_desc->sec_type) {