All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] perf machine: Initialize srcline string member in add_location struct
@ 2021-07-19 14:53 Michael Petlan
  2021-09-06 14:21 ` Michael Petlan
  2021-09-09 13:14 ` Arnaldo Carvalho de Melo
  0 siblings, 2 replies; 5+ messages in thread
From: Michael Petlan @ 2021-07-19 14:53 UTC (permalink / raw)
  To: acme, linux-perf-users; +Cc: jolsa, jlelli, milian.wolff

It's later supposed to be either a correct address or NULL. Without the
initialization, it may contain an undefined value which results in the
following segmentation fault:

  # perf top --sort comm -g --ignore-callees=do_idle

terminates with

  perf: Segmentation fault
  -------- backtrace --------
  perf(+0x417b26)[0x557794f1fb26]
  /lib64/libc.so.6(+0x37400)[0x7f62a0194400]
  /lib64/libc.so.6(+0x15d685)[0x7f62a02ba685]
  /lib64/libc.so.6(__strdup+0x12)[0x7f62a01e6802]
  perf(+0x3769d9)[0x557794e7e9d9]
  perf(+0x376d3a)[0x557794e7ed3a]
  perf(+0x377284)[0x557794e7f284]
  perf(+0x3773e0)[0x557794e7f3e0]
  perf(hist_entry_iter__add+0xc4)[0x557794e80114]
  perf(+0x2799aa)[0x557794d819aa]
  perf(+0x354ec8)[0x557794e5cec8]
  perf(+0x279341)[0x557794d81341]
  /lib64/libpthread.so.0(+0x814a)[0x7f62a27b514a]
  /lib64/libc.so.6(clone+0x43)[0x7f62a0259dc3]

Fixes: 1fb7d06a509e ("perf report: Use srcline from callchain for hist entries")

Signed-off-by: Michael Petlan <mpetlan@redhat.com>
Reported-by: Juri Lelli <jlelli@redhat.com>
CC: Milian Wolff <milian.wolff@kdab.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/perf/util/machine.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tools/perf/util/machine.c b/tools/perf/util/machine.c
index da19be7da284..44e40bad0e33 100644
--- a/tools/perf/util/machine.c
+++ b/tools/perf/util/machine.c
@@ -2149,6 +2149,7 @@ static int add_callchain_ip(struct thread *thread,
 
 	al.filtered = 0;
 	al.sym = NULL;
+	al.srcline = NULL;
 	if (!cpumode) {
 		thread__find_cpumode_addr_location(thread, ip, &al);
 	} else {
-- 
2.18.4


^ permalink raw reply related	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2021-09-14 21:21 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-07-19 14:53 [PATCH] perf machine: Initialize srcline string member in add_location struct Michael Petlan
2021-09-06 14:21 ` Michael Petlan
2021-09-09 13:14 ` Arnaldo Carvalho de Melo
2021-09-13 15:13   ` Michael Petlan
2021-09-14 21:21     ` Arnaldo Carvalho de Melo

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.