diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2023-04-12 09:50:08 -0300 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2023-04-12 10:07:06 -0300 |
commit | 25feb605fe3b2674142d084379e0737a99945aba (patch) | |
tree | 18e1633bd8d896cdc0205a7bd8863c0a7a88ad16 /tools/perf/util | |
parent | a77f8184a07cbe81cdee30582640ed1b412705fc (diff) | |
download | linux-stable-25feb605fe3b2674142d084379e0737a99945aba.tar.gz linux-stable-25feb605fe3b2674142d084379e0737a99945aba.tar.bz2 linux-stable-25feb605fe3b2674142d084379e0737a99945aba.zip |
perf parse-events: Use zfree() to reduce chances of use after free
Do defensive programming by using zfree() to initialize freed pointers
to NULL, so that eventual use after free result in a NULL pointer deref
instead of more subtle behaviour.
Also remove one NULL test before free(), as it accepts a NULL arg and we
get one line shaved not doing it explicitely.
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/util')
-rw-r--r-- | tools/perf/util/parse-events.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/util/parse-events.c b/tools/perf/util/parse-events.c index f341995cb04e..d71019dcd614 100644 --- a/tools/perf/util/parse-events.c +++ b/tools/perf/util/parse-events.c @@ -2020,7 +2020,7 @@ int perf_pmu__test_parse_init(void) err_free: for (j = 0, tmp = list; j < i; j++, tmp++) - free(tmp->symbol); + zfree(&tmp->symbol); free(list); return -ENOMEM; } |