uprobes: Kill the unnecesary filp != NULL check in __copy_insn()
authorOleg Nesterov <oleg@redhat.com>
Sun, 24 Mar 2013 17:45:44 +0000 (18:45 +0100)
committerOleg Nesterov <oleg@redhat.com>
Thu, 4 Apr 2013 11:57:05 +0000 (13:57 +0200)
__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>
kernel/events/uprobes.c

index ba6acfe7c0de557d2fb8c29e4815afc483cf3691..093866547fe338e93bf19b2c7448af6b42cf92da 100644 (file)
@@ -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;
        /*