From: Oleg Nesterov <oleg@redhat.com>
Date: Sun, 24 Mar 2013 17:45:44 +0000 (+0100)
Subject: uprobes: Kill the unnecesary filp != NULL check in __copy_insn()
X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=98763a1bb1515f8a8d7f1d9ae42604e19872364b;p=sagit-ice-cold%2Fkernel_xiaomi_msm8998.git

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>
---

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;
 	/*