diff options
| author | Oleg Nesterov <oleg@redhat.com> | 2013-03-24 21:45:44 +0400 |
|---|---|---|
| committer | Oleg Nesterov <oleg@redhat.com> | 2013-04-04 15:57:05 +0400 |
| commit | 98763a1bb1515f8a8d7f1d9ae42604e19872364b (patch) | |
| tree | 9ba213a12f30a94507a9bfdf68d8ef184f0ee056 /kernel/events/uprobes.c | |
| parent | 2edb7b5574d447354202ba365fb8ca6a1bac1d1c (diff) | |
| download | linux-98763a1bb1515f8a8d7f1d9ae42604e19872364b.tar.xz | |
uprobes: Kill the unnecesary filp != NULL check in __copy_insn()
__copy_insn(filp) can only be called after valid_vma() returns T,
vma->vm_file passed as "filp" can not be NULL.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Acked-by: Anton Arapov <anton@redhat.com>
Acked-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
Diffstat (limited to 'kernel/events/uprobes.c')
| -rw-r--r-- | kernel/events/uprobes.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/kernel/events/uprobes.c b/kernel/events/uprobes.c index ba6acfe7c0de..093866547fe3 100644 --- a/kernel/events/uprobes.c +++ b/kernel/events/uprobes.c @@ -501,9 +501,6 @@ __copy_insn(struct address_space *mapping, struct file *filp, char *insn, { struct page *page; - if (!filp) - return -EINVAL; - if (!mapping->a_ops->readpage) return -EIO; /* |