diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2020-09-02 18:30:48 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2020-09-09 20:03:09 +0300 |
| commit | c5c6e00f6cc5d3ed0d6464b14e33f2f5c8505888 (patch) | |
| tree | 748f81f628a9da73ad87658856acd855ca41f188 /fs/eventpoll.c | |
| parent | cfb5e057b7179767d6f3cef3381633417e4e3afb (diff) | |
| download | linux-c5c6e00f6cc5d3ed0d6464b14e33f2f5c8505888.tar.xz | |
fix regression in "epoll: Keep a reference on files added to the check list"
[ Upstream commit 77f4689de17c0887775bb77896f4cc11a39bf848 ]
epoll_loop_check_proc() can run into a file already committed to destruction;
we can't grab a reference on those and don't need to add them to the set for
reverse path check anyway.
Tested-by: Marc Zyngier <maz@kernel.org>
Fixes: a9ed4a6560b8 ("epoll: Keep a reference on files added to the check list")
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'fs/eventpoll.c')
| -rw-r--r-- | fs/eventpoll.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/eventpoll.c b/fs/eventpoll.c index 00f0902e27e8..af9dfa494b1f 100644 --- a/fs/eventpoll.c +++ b/fs/eventpoll.c @@ -1901,9 +1901,9 @@ static int ep_loop_check_proc(void *priv, void *cookie, int call_nests) * during ep_insert(). */ if (list_empty(&epi->ffd.file->f_tfile_llink)) { - get_file(epi->ffd.file); - list_add(&epi->ffd.file->f_tfile_llink, - &tfile_check_list); + if (get_file_rcu(epi->ffd.file)) + list_add(&epi->ffd.file->f_tfile_llink, + &tfile_check_list); } } } |