perf: intel-tpebs: Fix incorrect usage of zfree()
authorJames Clark <james.clark@linaro.org>
Wed, 19 Mar 2025 10:16:10 +0000 (10:16 +0000)
committerNamhyung Kim <namhyung@kernel.org>
Wed, 19 Mar 2025 23:56:56 +0000 (16:56 -0700)
zfree() requires an address otherwise it frees what's in name, rather
than name itself. Pass the address of name to fix it.

This was the only incorrect occurrence in Perf found using a search.

Fixes: 8db5cabcf1b6 ("perf stat: Fork and launch 'perf record' when 'perf stat' needs to get retire latency value for a metric.")
Signed-off-by: James Clark <james.clark@linaro.org>
Link: https://lore.kernel.org/r/20250319101614.190922-1-james.clark@linaro.org
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
tools/perf/util/intel-tpebs.c

index 50a3c3e0716065f8207d361e79d1b24eadc7331c..2c421b475b3b8bda1febfc74c2708d20e04d756e 100644 (file)
@@ -254,7 +254,7 @@ int tpebs_start(struct evlist *evsel_list)
                new = zalloc(sizeof(*new));
                if (!new) {
                        ret = -1;
-                       zfree(name);
+                       zfree(&name);
                        goto err;
                }
                new->name = name;