Revision e0fa7ab42232e742dcb3de9f3c1f6127b5adc019 authored by Riccardo Mancini on 15 July 2021, 16:07:25 UTC, committed by Arnaldo Carvalho de Melo on 18 July 2021, 12:27:37 UTC
ASan reports some memory leaks when running:

  # perf test "42: BPF filter"

This second leak is caused by a strlist not being dellocated on error
inside probe_file__del_events.

This patch adds a goto label before the deallocation and makes the error
path jump to it.

Signed-off-by: Riccardo Mancini <rickyman7@gmail.com>
Fixes: e7895e422e4da63d ("perf probe: Split del_perf_probe_events()")
Cc: Ian Rogers <irogers@google.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lore.kernel.org/lkml/174963c587ae77fa108af794669998e4ae558338.1626343282.git.rickyman7@gmail.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
1 parent 937654c
History
File Mode Size
Kconfig -rw-r--r-- 529 bytes
Makefile -rw-r--r-- 378 bytes
ast.c -rw-r--r-- 8.1 KB
ast.h -rw-r--r-- 1.1 KB
config.c -rw-r--r-- 23.2 KB
config.h -rw-r--r-- 1.4 KB
debug_fs.c -rw-r--r-- 19.5 KB
dir.c -rw-r--r-- 7.3 KB
dir.h -rw-r--r-- 889 bytes
dlm_internal.h -rw-r--r-- 20.8 KB
lock.c -rw-r--r-- 157.3 KB
lock.h -rw-r--r-- 2.9 KB
lockspace.c -rw-r--r-- 20.5 KB
lockspace.h -rw-r--r-- 902 bytes
lowcomms.c -rw-r--r-- 47.9 KB
lowcomms.h -rw-r--r-- 1.7 KB
lvb_table.h -rw-r--r-- 541 bytes
main.c -rw-r--r-- 1.8 KB
member.c -rw-r--r-- 16.5 KB
member.h -rw-r--r-- 1.3 KB
memory.c -rw-r--r-- 1.9 KB
memory.h -rw-r--r-- 904 bytes
midcomms.c -rw-r--r-- 35.5 KB
midcomms.h -rw-r--r-- 1.3 KB
netlink.c -rw-r--r-- 2.8 KB
plock.c -rw-r--r-- 11.5 KB
rcom.c -rw-r--r-- 16.6 KB
rcom.h -rw-r--r-- 987 bytes
recover.c -rw-r--r-- 22.8 KB
recover.h -rw-r--r-- 1.3 KB
recoverd.c -rw-r--r-- 8.1 KB
recoverd.h -rw-r--r-- 781 bytes
requestqueue.c -rw-r--r-- 4.7 KB
requestqueue.h -rw-r--r-- 736 bytes
user.c -rw-r--r-- 24.5 KB
user.h -rw-r--r-- 440 bytes
util.c -rw-r--r-- 4.4 KB
util.h -rw-r--r-- 738 bytes

back to top