diff options
| author | Tianyou Li <tianyou.li@intel.com> | 2025-10-20 15:30:05 +0800 |
|---|---|---|
| committer | Namhyung Kim <namhyung@kernel.org> | 2025-10-21 05:42:34 -0700 |
| commit | f1204e5846d22fb2fffbd1164eeb19535f306797 (patch) | |
| tree | 3c316e54450a0a9bd52a7cc4731c05a7059f1502 /tools/perf/util/annotate.c | |
| parent | 262c61435c0348f3c4652bb200268b2d1bcefbb0 (diff) | |
perf annotate: Check return value of evsel__get_arch() properly
Check the error code of evsel__get_arch() in the symbol__annotate().
Previously it checked non-zero value but after the refactoring it does
only for negative values.
Fixes: 0669729eb0afb0cf ("perf annotate: Factor out evsel__get_arch()")
Suggested-by: James Clark <james.clark@linaro.org>
Acked-by: Namhyung Kim <namhyung@kernel.org>
Signed-off-by: Tianyou Li <tianyou.li@intel.com>
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Diffstat (limited to 'tools/perf/util/annotate.c')
| -rw-r--r-- | tools/perf/util/annotate.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/util/annotate.c b/tools/perf/util/annotate.c index 39d6594850f1..859e802a1e5e 100644 --- a/tools/perf/util/annotate.c +++ b/tools/perf/util/annotate.c @@ -1021,7 +1021,7 @@ int symbol__annotate(struct map_symbol *ms, struct evsel *evsel, int err, nr; err = evsel__get_arch(evsel, &arch); - if (err < 0) + if (err) return err; if (parch) |