From: Al Viro <viro@zeniv.linux.org.uk>
Date: Thu, 27 Jun 2019 00:43:53 +0000 (-0400)
Subject: perf_event_get(): don't bother with fget_raw()
X-Git-Tag: v5.3-rc1~29^2
X-Git-Url: https://repo.jachan.dev/linux.git/commitdiff_plain/02e5ad973883c36c0868b301b8357d9c455bb91c

perf_event_get(): don't bother with fget_raw()

... since we immediately follow that with check that it *is* an
opened perf file, with O_PATH ones ending with with the same
-EBADF we'd get for descriptor that isn't opened at all.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
---

diff --git a/kernel/events/core.c b/kernel/events/core.c
index abbd4b3b96c2..f9ff04c8d084 100644
--- a/kernel/events/core.c
+++ b/kernel/events/core.c
@@ -11554,9 +11554,7 @@ void perf_event_delayed_put(struct task_struct *task)
 
 struct file *perf_event_get(unsigned int fd)
 {
-	struct file *file;
-
-	file = fget_raw(fd);
+	struct file *file = fget(fd);
 	if (!file)
 		return ERR_PTR(-EBADF);