diff options
| author | Peter Zijlstra <peterz@infradead.org> | 2014-05-05 19:12:20 +0200 |
|---|---|---|
| committer | Ingo Molnar <mingo@kernel.org> | 2014-05-07 11:33:15 +0200 |
| commit | ffb4ef21ac4308c2e738e6f83b6741bbc9b4fa3b (patch) | |
| tree | 8fac0ed6e911159faa232d149a5d3113a2415ae2 | |
| parent | 46ce0fe97a6be7532ce6126bb26ce89fed81528c (diff) | |
| download | linux-ffb4ef21ac4308c2e738e6f83b6741bbc9b4fa3b.tar.bz2 | |
perf: Fix perf_event_init_context()
perf_pin_task_context() can return NULL but perf_event_init_context()
assumes it will not, correct this.
Reported-by: Vince Weaver <vincent.weaver@maine.edu>
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Cc: Arnaldo Carvalho de Melo <acme@kernel.org>
Link: http://lkml.kernel.org/r/20140505171428.GU26782@laptop.programming.kicks-ass.net
Signed-off-by: Ingo Molnar <mingo@kernel.org>
| -rw-r--r-- | kernel/events/core.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/kernel/events/core.c b/kernel/events/core.c index ea899e2b5593..71232844f235 100644 --- a/kernel/events/core.c +++ b/kernel/events/core.c @@ -7729,6 +7729,8 @@ int perf_event_init_context(struct task_struct *child, int ctxn) * swapped under us. */ parent_ctx = perf_pin_task_context(parent, ctxn); + if (!parent_ctx) + return 0; /* * No need to check if parent_ctx != NULL here; since we saw |