diff options
| author | Hao Ge <gehao@kylinos.cn> | 2024-11-13 11:05:37 +0800 |
|---|---|---|
| committer | Namhyung Kim <namhyung@kernel.org> | 2024-11-22 13:36:00 -0800 |
| commit | bd077a53ad87cb111632e564cdfe8dfbe96786de (patch) | |
| tree | a0e0769bfdc7535013264f70b7e4db586ce55141 /tools/perf | |
| parent | fc26637d70d3f1ba9df8cd0f7c22b8b331c564e7 (diff) | |
| download | lwn-bd077a53ad87cb111632e564cdfe8dfbe96786de.tar.gz lwn-bd077a53ad87cb111632e564cdfe8dfbe96786de.zip | |
perf bpf-filter: Return -ENOMEM directly when pfi allocation fails
Directly return -ENOMEM when pfi allocation fails,
instead of performing other operations on pfi.
Fixes: 0fe2b18ddc40 ("perf bpf-filter: Support multiple events properly")
Signed-off-by: Hao Ge <gehao@kylinos.cn>
Acked-by: Namhyung Kim <namhyung@kernel.org>
Cc: hao.ge@linux.dev
Cc: bpf@vger.kernel.org
Link: https://lore.kernel.org/r/20241113030537.26732-1-hao.ge@linux.dev
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Diffstat (limited to 'tools/perf')
| -rw-r--r-- | tools/perf/util/bpf-filter.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/util/bpf-filter.c b/tools/perf/util/bpf-filter.c index e87b6789eb9e..a4fdf6911ec1 100644 --- a/tools/perf/util/bpf-filter.c +++ b/tools/perf/util/bpf-filter.c @@ -375,7 +375,7 @@ static int create_idx_hash(struct evsel *evsel, struct perf_bpf_filter_entry *en pfi = zalloc(sizeof(*pfi)); if (pfi == NULL) { pr_err("Cannot save pinned filter index\n"); - goto err; + return -ENOMEM; } pfi->evsel = evsel; |