From: Greg Kroah-Hartman Date: Fri, 24 Mar 2017 08:42:19 +0000 (+0100) Subject: 4.4-stable patches X-Git-Tag: v4.4.57~7 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=97d55e7e5c677eb4f84e97f67fc43fbccfd16dbb;p=thirdparty%2Fkernel%2Fstable-queue.git 4.4-stable patches added patches: perf-core-fix-event-inheritance-on-fork.patch --- diff --git a/queue-4.4/perf-core-fix-event-inheritance-on-fork.patch b/queue-4.4/perf-core-fix-event-inheritance-on-fork.patch new file mode 100644 index 00000000000..9f73af4873e --- /dev/null +++ b/queue-4.4/perf-core-fix-event-inheritance-on-fork.patch @@ -0,0 +1,66 @@ +From e7cc4865f0f31698ef2f7aac01a50e78968985b7 Mon Sep 17 00:00:00 2001 +From: Peter Zijlstra +Date: Thu, 16 Mar 2017 13:47:49 +0100 +Subject: perf/core: Fix event inheritance on fork() + +From: Peter Zijlstra + +commit e7cc4865f0f31698ef2f7aac01a50e78968985b7 upstream. + +While hunting for clues to a use-after-free, Oleg spotted that +perf_event_init_context() can loose an error value with the result +that fork() can succeed even though we did not fully inherit the perf +event context. + +Spotted-by: Oleg Nesterov +Signed-off-by: Peter Zijlstra (Intel) +Cc: Alexander Shishkin +Cc: Arnaldo Carvalho de Melo +Cc: Arnaldo Carvalho de Melo +Cc: Dmitry Vyukov +Cc: Frederic Weisbecker +Cc: Jiri Olsa +Cc: Linus Torvalds +Cc: Mathieu Desnoyers +Cc: Peter Zijlstra +Cc: Stephane Eranian +Cc: Thomas Gleixner +Cc: Vince Weaver +Cc: oleg@redhat.com +Fixes: 889ff0150661 ("perf/core: Split context's event group list into pinned and non-pinned lists") +Link: http://lkml.kernel.org/r/20170316125823.190342547@infradead.org +Signed-off-by: Ingo Molnar +Signed-off-by: Greg Kroah-Hartman + +--- + kernel/events/core.c | 5 +++-- + 1 file changed, 3 insertions(+), 2 deletions(-) + +--- a/kernel/events/core.c ++++ b/kernel/events/core.c +@@ -9230,7 +9230,7 @@ static int perf_event_init_context(struc + ret = inherit_task_group(event, parent, parent_ctx, + child, ctxn, &inherited_all); + if (ret) +- break; ++ goto out_unlock; + } + + /* +@@ -9246,7 +9246,7 @@ static int perf_event_init_context(struc + ret = inherit_task_group(event, parent, parent_ctx, + child, ctxn, &inherited_all); + if (ret) +- break; ++ goto out_unlock; + } + + raw_spin_lock_irqsave(&parent_ctx->lock, flags); +@@ -9274,6 +9274,7 @@ static int perf_event_init_context(struc + } + + raw_spin_unlock_irqrestore(&parent_ctx->lock, flags); ++out_unlock: + mutex_unlock(&parent_ctx->mutex); + + perf_unpin_context(parent_ctx); diff --git a/queue-4.4/series b/queue-4.4/series index 2ebd9d49a66..ac57cd242a9 100644 --- a/queue-4.4/series +++ b/queue-4.4/series @@ -16,3 +16,4 @@ tpm_tis-use-devm_free_irq-not-free_irq.patch hv_netvsc-use-skb_get_hash-instead-of-a-homegrown-implementation.patch kernek-fork.c-allocate-idle-task-for-a-cpu-always-on-its-local-node.patch give-up-on-gcc-ilog2-constant-optimizations.patch +perf-core-fix-event-inheritance-on-fork.patch