Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

perf/core: Prevent VMA split of buffer mappings

The perf mmap code is careful about mmap()'ing the user page with the
ringbuffer and additionally the auxiliary buffer, when the event supports
it. Once the first mapping is established, subsequent mapping have to use
the same offset and the same size in both cases. The reference counting for
the ringbuffer and the auxiliary buffer depends on this being correct.

Though perf does not prevent that a related mapping is split via mmap(2),
munmap(2) or mremap(2). A split of a VMA results in perf_mmap_open() calls,
which take reference counts, but then the subsequent perf_mmap_close()
calls are not longer fulfilling the offset and size checks. This leads to
reference count leaks.

As perf already has the requirement for subsequent mappings to match the
initial mapping, the obvious consequence is that VMA splits, caused by
resizing of a mapping or partial unmapping, have to be prevented.

Implement the vm_operations_struct::may_split() callback and return
unconditionally -EINVAL.

That ensures that the mapping offsets and sizes cannot be changed after the
fact. Remapping to a different fixed address with the same size is still
possible as it takes the references for the new mapping and drops those of
the old mapping.

Fixes: 45bfb2e50471 ("perf: Add AUX area to ring buffer for raw data streams")
Reported-by: zdi-disclosures@trendmicro.com # ZDI-CAN-27504
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
Acked-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Acked-by: Vlastimil Babka <vbabka@suse.cz>
Cc: stable@vger.kernel.org

+10
+10
kernel/events/core.c
··· 6842 6842 return vmf->pgoff == 0 ? 0 : VM_FAULT_SIGBUS; 6843 6843 } 6844 6844 6845 + static int perf_mmap_may_split(struct vm_area_struct *vma, unsigned long addr) 6846 + { 6847 + /* 6848 + * Forbid splitting perf mappings to prevent refcount leaks due to 6849 + * the resulting non-matching offsets and sizes. See open()/close(). 6850 + */ 6851 + return -EINVAL; 6852 + } 6853 + 6845 6854 static const struct vm_operations_struct perf_mmap_vmops = { 6846 6855 .open = perf_mmap_open, 6847 6856 .close = perf_mmap_close, /* non mergeable */ 6848 6857 .pfn_mkwrite = perf_mmap_pfn_mkwrite, 6858 + .may_split = perf_mmap_may_split, 6849 6859 }; 6850 6860 6851 6861 static int map_range(struct perf_buffer *rb, struct vm_area_struct *vma)