Commit 3273c440 authored by Pavel Begunkov's avatar Pavel Begunkov Committed by Jens Axboe
Browse files

io_uring: don't check file ops of registered rings 



Registered rings are per definitions io_uring files, so we don't need to
additionally verify them.

Signed-off-by: default avatarPavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/425cd64fd885b8e329a46c205ee811987691baaf.1656153286.git.asml.silence@gmail.com


Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent ad8b261d

io_uring/io_uring.c

+8 −8
Original line number Diff line number Diff line
@@ -3036,21 +3036,21 @@ SYSCALL_DEFINE6(io_uring_enter, unsigned int, fd, u32, to_submit, 
	if (flags & IORING_ENTER_REGISTERED_RING) {

		struct io_uring_task *tctx = current->io_uring;



		if (!tctx || fd >= IO_RINGFD_REG_MAX)

		if (unlikely(!tctx || fd >= IO_RINGFD_REG_MAX))

			return -EINVAL;

		fd = array_index_nospec(fd, IO_RINGFD_REG_MAX);

		f.file = tctx->registered_rings[fd];

		f.flags = 0;

		if (unlikely(!f.file))

			return -EBADF;

	} else {

		f = fdget(fd);

	}



		if (unlikely(!f.file))

			return -EBADF;



		ret = -EOPNOTSUPP;

		if (unlikely(!io_is_uring_fops(f.file)))

			goto out_fput;

	}



	ret = -ENXIO;

	ctx = f.file->private_data;