Messages in this thread Patch in this message |  | | Date | Fri, 6 Mar 2026 10:50:22 -0800 | | Subject | [PATCH v1] perf ftrace: Proper error clean up in print_profile_result | | From | Ian Rogers <> |
| |
The ftrace profile_hash is cleaned up on success but not on memory allocation and empty paths. Fix by running the clean up for all. Free the profile too, so make it NULL, just for clarity that this is all clean up code.
Signed-off-by: Ian Rogers <irogers@google.com> --- tools/perf/builtin-ftrace.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/tools/perf/builtin-ftrace.c b/tools/perf/builtin-ftrace.c index 6b6eec65f93f..19e796e58fb0 100644 --- a/tools/perf/builtin-ftrace.c +++ b/tools/perf/builtin-ftrace.c @@ -18,6 +18,7 @@ #include <poll.h> #include <ctype.h> #include <linux/capability.h> +#include <linux/err.h> #include <linux/string.h> #include <sys/stat.h> @@ -1369,17 +1370,17 @@ static int cmp_profile_data(const void *a, const void *b) static void print_profile_result(struct perf_ftrace *ftrace) { - struct hashmap_entry *entry, **profile; + struct hashmap_entry *entry, **profile = NULL; size_t i, nr, bkt; nr = hashmap__size(ftrace->profile_hash); if (nr == 0) - return; + goto out; profile = calloc(nr, sizeof(*profile)); if (profile == NULL) { pr_err("failed to allocate memory for the result\n"); - return; + goto out; } i = 0; @@ -1403,6 +1404,7 @@ static void print_profile_result(struct perf_ftrace *ftrace) p->st.max / 1000, p->st.max % 1000, p->st.n, name); } +out: free(profile); hashmap__for_each_entry(ftrace->profile_hash, entry, bkt) { -- 2.53.0.473.g4a7958ca14-goog
|  |