perf tools: Robustify dynamic sample content fetch
authorFrederic Weisbecker <fweisbec@gmail.com>
Sat, 21 May 2011 18:08:15 +0000 (20:08 +0200)
committerFrederic Weisbecker <fweisbec@gmail.com>
Sun, 22 May 2011 01:38:48 +0000 (03:38 +0200)
Ensure the size of the dynamic fields such as callchains
or raw events don't overlap the whole event boundaries.

This prevents from dereferencing junk if the given size of
the callchain goes too eager.

Reported-by: Linus Torvalds <torvalds@linux-foundation.org>
Reported-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Stephane Eranian <eranian@google.com>
tools/perf/util/evsel.c

index bfce8bf642fae5943ade4dffb1a8743f55beaf25..ee0fe0dffa7122e1936925b91fcbcf47de7739a1 100644 (file)
@@ -303,6 +303,17 @@ static int perf_event__parse_id_sample(const union perf_event *event, u64 type,
        return 0;
 }
 
+static bool sample_overlap(const union perf_event *event,
+                          const void *offset, u64 size)
+{
+       const void *base = event;
+
+       if (offset + size > base + event->header.size)
+               return true;
+
+       return false;
+}
+
 int perf_event__parse_sample(const union perf_event *event, u64 type,
                             int sample_size, bool sample_id_all,
                             struct perf_sample *data)
@@ -373,14 +384,29 @@ int perf_event__parse_sample(const union perf_event *event, u64 type,
        }
 
        if (type & PERF_SAMPLE_CALLCHAIN) {
+               if (sample_overlap(event, array, sizeof(data->callchain->nr)))
+                       return -EFAULT;
+
                data->callchain = (struct ip_callchain *)array;
+
+               if (sample_overlap(event, array, data->callchain->nr))
+                       return -EFAULT;
+
                array += 1 + data->callchain->nr;
        }
 
        if (type & PERF_SAMPLE_RAW) {
                u32 *p = (u32 *)array;
+
+               if (sample_overlap(event, array, sizeof(u32)))
+                       return -EFAULT;
+
                data->raw_size = *p;
                p++;
+
+               if (sample_overlap(event, p, data->raw_size))
+                       return -EFAULT;
+
                data->raw_data = p;
        }