projects / cascardo / linux.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 75ed82e) | patch
uprobes: Change valid_vma() to demand VM_MAYEXEC rather than VM_EXEC
authorOleg Nesterov <oleg@redhat.com>
Sun, 16 Sep 2012 17:07:41 +0000 (19:07 +0200)
committerOleg Nesterov <oleg@redhat.com>
Sat, 29 Sep 2012 19:21:53 +0000 (21:21 +0200)
commit78a320542e6cdb2800cd736b2d136e4261d34f43
treeeb55bec46370f7890fdec4c44450e52aba043885tree | snapshot
parent75ed82ea53bd0d2d8083261123576250f7ba851ecommit | diff
uprobes: Change valid_vma() to demand VM_MAYEXEC rather than VM_EXEC

uprobe_register() or uprobe_mmap() requires VM_READ | VM_EXEC, this
is not right. An apllication can do mprotect(PROT_EXEC) later and
execute this code.

Change valid_vma(is_register => true) to check VM_MAYEXEC instead.
No need to check VM_MAYREAD, it is always set.

Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Acked-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
kernel/events/uprobes.c diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.