diff options
| author | Thomas Gleixner <tglx@linutronix.de> | 2025-08-02 12:49:48 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2025-08-15 12:14:09 +0200 |
| commit | 92043120a2e992800580855498ab8507e1b22db9 (patch) | |
| tree | dc596ce149f87093d0941b8d4cbd595e8bbb654f /kernel | |
| parent | 8c67899a52955c1a753a007b890e26aca9d1a209 (diff) | |
| download | linux-92043120a2e992800580855498ab8507e1b22db9.tar.gz linux-92043120a2e992800580855498ab8507e1b22db9.tar.bz2 linux-92043120a2e992800580855498ab8507e1b22db9.zip | |
perf/core: Exit early on perf_mmap() fail
commit 07091aade394f690e7b655578140ef84d0e8d7b0 upstream.
When perf_mmap() fails to allocate a buffer, it still invokes the
event_mapped() callback of the related event. On X86 this might increase
the perf_rdpmc_allowed reference counter. But nothing undoes this as
perf_mmap_close() is never called in this case, which causes another
reference count leak.
Return early on failure to prevent that.
Fixes: 1e0fb9ec679c ("perf/core: Add pmu callbacks to track event mapping and unmapping")
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>Cc: stable@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/events/core.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/kernel/events/core.c b/kernel/events/core.c index b606680370ca..92ed4b5b0b8b 100644 --- a/kernel/events/core.c +++ b/kernel/events/core.c @@ -6836,6 +6836,9 @@ aux_unlock: mutex_unlock(aux_mutex); mutex_unlock(&event->mmap_mutex); + if (ret) + return ret; + /* * Since pinned accounting is per vm we cannot allow fork() to copy our * vma. |