]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
perf pmu: Switch FILENAME_MAX to NAME_MAX
authorIan Rogers <irogers@google.com>
Thu, 17 Jul 2025 15:08:54 +0000 (08:08 -0700)
committerNamhyung Kim <namhyung@kernel.org>
Wed, 23 Jul 2025 01:17:53 +0000 (18:17 -0700)
FILENAME_MAX is the same as PATH_MAX (4kb) in glibc rather than
NAME_MAX's 255. Switch to using NAME_MAX and ensure the '\0' is
accounted for in the path's buffer size.

Fixes: 754baf426e09 ("perf pmu: Change aliases from list to hashmap")
Signed-off-by: Ian Rogers <irogers@google.com>
Link: https://lore.kernel.org/r/20250717150855.1032526-2-irogers@google.com
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
tools/perf/util/pmu.c

index b09b2ea2407ab1a4b0f6c35a6649e6d36c34d898..f3da6e27bfcb7772f07159d1dfbbe57cb07e8170 100644 (file)
@@ -453,7 +453,7 @@ static struct perf_pmu_alias *perf_pmu__find_alias(struct perf_pmu *pmu,
 {
        struct perf_pmu_alias *alias;
        bool has_sysfs_event;
-       char event_file_name[FILENAME_MAX + 8];
+       char event_file_name[NAME_MAX + 8];
 
        if (hashmap__find(pmu->aliases, name, &alias))
                return alias;
@@ -752,7 +752,7 @@ static int pmu_aliases_parse(struct perf_pmu *pmu)
 
 static int pmu_aliases_parse_eager(struct perf_pmu *pmu, int sysfs_fd)
 {
-       char path[FILENAME_MAX + 7];
+       char path[NAME_MAX + 8];
        int ret, events_dir_fd;
 
        scnprintf(path, sizeof(path), "%s/events", pmu->name);