diff options
author | Arve Hjønnevåg <arve@android.com> | 2012-01-20 22:56:21 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-02-08 17:14:13 -0500 |
commit | 3c1b86f17068cf6476fb2d022b9c8b44dedea2e5 (patch) | |
tree | 9288383705799820d62f412a859a2dac99047bd9 | |
parent | 1793bf1deddc8ce25dc41925d5dbe64536c841b6 (diff) |
Staging: android: binder: Don't call dump_stack in binder_vma_open
If user-space partially unmaps the driver, binder_vma_open
would dump the kernel stack. This is not a kernel bug however
and will be treated as if the whole area was unmapped once
binder_vma_close gets called.
Signed-off-by: Arve Hjønnevåg <arve@android.com>
Cc: stable <stable@vger.kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r-- | drivers/staging/android/binder.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/staging/android/binder.c b/drivers/staging/android/binder.c index 7491801a661c..48cf27cdb51a 100644 --- a/drivers/staging/android/binder.c +++ b/drivers/staging/android/binder.c | |||
@@ -2759,7 +2759,6 @@ static void binder_vma_open(struct vm_area_struct *vma) | |||
2759 | proc->pid, vma->vm_start, vma->vm_end, | 2759 | proc->pid, vma->vm_start, vma->vm_end, |
2760 | (vma->vm_end - vma->vm_start) / SZ_1K, vma->vm_flags, | 2760 | (vma->vm_end - vma->vm_start) / SZ_1K, vma->vm_flags, |
2761 | (unsigned long)pgprot_val(vma->vm_page_prot)); | 2761 | (unsigned long)pgprot_val(vma->vm_page_prot)); |
2762 | dump_stack(); | ||
2763 | } | 2762 | } |
2764 | 2763 | ||
2765 | static void binder_vma_close(struct vm_area_struct *vma) | 2764 | static void binder_vma_close(struct vm_area_struct *vma) |