On 2/7/26 03:32, Andrii Nakryiko wrote: > On Mon, Jun 29, 2026 at 8:25 AM Leon Hwang <[email protected]> wrote: [...] >> @@ -1975,7 +1981,7 @@ bpf_object__init_internal_map(struct bpf_object *obj, >> enum libbpf_map_type type, >> map->sec_idx = sec_idx; >> map->sec_offset = 0; >> map->real_name = strdup(real_name); >> - map->name = internal_map_name(obj, real_name); >> + map->name = is_percpu ? strdup(real_name) : internal_map_name(obj, >> real_name); > > nit: I'd probably pass type into internal_map_name() and let it handle > all this in one place, consider that for follow up > hmm, it is a true issue that 'map->name' must have the tail '\0'. And strdup(real_name) does not guarantee the tail '\0'.
Will pass type into internal_map_name() instead. Thanks, Leon

