diff options
author | Pavel Begunkov <asml.silence@gmail.com> | 2022-04-06 12:43:58 +0100 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2022-04-07 11:17:47 -0600 |
commit | a07211e3001435fe8591b992464cd8d5e3c98c5a (patch) | |
tree | 8f1c5092bae72a51afe02cc5b5dc2295d2105de3 /fs/io_uring.c | |
parent | 34bb77184123ae401100a4d156584f12fa630e5c (diff) | |
download | linux-stable-a07211e3001435fe8591b992464cd8d5e3c98c5a.tar.gz linux-stable-a07211e3001435fe8591b992464cd8d5e3c98c5a.tar.bz2 linux-stable-a07211e3001435fe8591b992464cd8d5e3c98c5a.zip |
io_uring: don't touch scm_fp_list after queueing skb
It's safer to not touch scm_fp_list after we queued an skb to which it
was assigned, there might be races lurking if we screw subtle sync
guarantees on the io_uring side.
Fixes: 6b06314c47e14 ("io_uring: add file set registration")
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'fs/io_uring.c')
-rw-r--r-- | fs/io_uring.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/fs/io_uring.c b/fs/io_uring.c index b517fd9c3f60..7e672464dcb3 100644 --- a/fs/io_uring.c +++ b/fs/io_uring.c @@ -8631,8 +8631,12 @@ static int __io_sqe_files_scm(struct io_ring_ctx *ctx, int nr, int offset) refcount_add(skb->truesize, &sk->sk_wmem_alloc); skb_queue_head(&sk->sk_receive_queue, skb); - for (i = 0; i < nr_files; i++) - fput(fpl->fp[i]); + for (i = 0; i < nr; i++) { + struct file *file = io_file_from_index(ctx, i + offset); + + if (file) + fput(file); + } } else { kfree_skb(skb); free_uid(fpl->user); |