diff options
| author |  Theodore Ts'o <tytso@mit.edu>
| 2023-03-06 13:54:50 -0500 |
|---|---|---|
| committer |  Al Viro <viro@zeniv.linux.org.uk>
| 2023-03-09 22:46:21 -0500 |
| commit | 609d54441493c99f21c1823dfd66fa7f4c512ff4 (patch) | |
| tree | 197db6ee1e1f8d89d60b9e0db4ce4634a53a2784 /fs/file.c | |
| parent | alpha: fix lazy-FPU mis(merged/applied/whatnot) (diff) | |
| download | linux-609d54441493c99f21c1823dfd66fa7f4c512ff4.tar.gz linux-609d54441493c99f21c1823dfd66fa7f4c512ff4.tar.bz2 linux-609d54441493c99f21c1823dfd66fa7f4c512ff4.zip | |
fs: prevent out-of-bounds array speculation when closing a file descriptor
Google-Bug-Id: 114199369
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/file.c')
| -rw-r--r-- | fs/file.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/file.c b/fs/file.c index c942c89ca4cd..7893ea161d77 100644 --- a/fs/file.c +++ b/fs/file.c @@ -642,6 +642,7 @@ static struct file *pick_file(struct files_struct *files, unsigned fd) if (fd >= fdt->max_fds) return NULL; + fd = array_index_nospec(fd, fdt->max_fds); file = fdt->fd[fd]; if (file) { rcu_assign_pointer(fdt->fd[fd], NULL); |