diff options
author | Peter Zijlstra <peterz@infradead.org> | 2014-05-05 13:12:20 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2014-05-07 05:33:15 -0400 |
commit | ffb4ef21ac4308c2e738e6f83b6741bbc9b4fa3b (patch) | |
tree | 8fac0ed6e911159faa232d149a5d3113a2415ae2 /kernel/events | |
parent | 46ce0fe97a6be7532ce6126bb26ce89fed81528c (diff) |
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>
Diffstat (limited to 'kernel/events')
-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) | |||
7729 | * swapped under us. | 7729 | * swapped under us. |
7730 | */ | 7730 | */ |
7731 | parent_ctx = perf_pin_task_context(parent, ctxn); | 7731 | parent_ctx = perf_pin_task_context(parent, ctxn); |
7732 | if (!parent_ctx) | ||
7733 | return 0; | ||
7732 | 7734 | ||
7733 | /* | 7735 | /* |
7734 | * No need to check if parent_ctx != NULL here; since we saw | 7736 | * No need to check if parent_ctx != NULL here; since we saw |