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

io_uring/kbuf: use mem_is_zero() 



Make use of mem_is_zero() for reserved fields checking.

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


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

io_uring/kbuf.c

+4 −10
Original line number Diff line number Diff line
@@ -602,8 +602,7 @@ int io_register_pbuf_ring(struct io_ring_ctx *ctx, void __user *arg) 


	if (copy_from_user(&reg, arg, sizeof(reg)))

		return -EFAULT;



	if (reg.resv[0] || reg.resv[1] || reg.resv[2])

	if (!mem_is_zero(reg.resv, sizeof(reg.resv)))

		return -EINVAL;

	if (reg.flags & ~(IOU_PBUF_RING_MMAP | IOU_PBUF_RING_INC))

		return -EINVAL;
@@ -679,9 +678,7 @@ int io_unregister_pbuf_ring(struct io_ring_ctx *ctx, void __user *arg) 


	if (copy_from_user(&reg, arg, sizeof(reg)))

		return -EFAULT;

	if (reg.resv[0] || reg.resv[1] || reg.resv[2])

		return -EINVAL;

	if (reg.flags)

	if (!mem_is_zero(reg.resv, sizeof(reg.resv)) || reg.flags)

		return -EINVAL;



	bl = io_buffer_get_list(ctx, reg.bgid);
@@ -701,13 +698,10 @@ int io_register_pbuf_status(struct io_ring_ctx *ctx, void __user *arg) 
{

	struct io_uring_buf_status buf_status;

	struct io_buffer_list *bl;

	int i;



	if (copy_from_user(&buf_status, arg, sizeof(buf_status)))

		return -EFAULT;



	for (i = 0; i < ARRAY_SIZE(buf_status.resv); i++)

		if (buf_status.resv[i])

	if (!mem_is_zero(buf_status.resv, sizeof(buf_status.resv)))

		return -EINVAL;



	bl = io_buffer_get_list(ctx, buf_status.buf_group);