[PATCH] io_uring/net: initialize mshot_len for send

[PATCH] io_uring/net: initialize mshot_len for send

[Jens Axboe]

Commit:

6a8afb9fff64 ("io_uring/net: allow multishot receive per-invocation cap")

changed how io_mshot_prep_retry() set sr->len, and added the same
initialization in io_mshot_prep_retry(). But it neglected to touch the
send path, which may also uses the mshot retry path. Ensure that
sr->mshot_len always gets initialized correctly.

Fixes: 6a8afb9fff64 ("io_uring/net: allow multishot receive per-invocation cap")
Cc: stable@vger.kernel.org
Reported-by: Sung Keum <kambodi127@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

---

diff --git a/io_uring/net.c b/io_uring/net.c
index 00a7df803b99..a74d15f7b7d2 100644
--- a/io_uring/net.c
+++ b/io_uring/net.c
@@ -445,6 +445,7 @@ int io_sendmsg_prep(struct io_kiocb *req, const struct io_uring_sqe *sqe)
 		req->flags |= REQ_F_NOWAIT;
 	if (req->flags & REQ_F_BUFFER_SELECT)
 		sr->buf_group = req->buf_index;
+	sr->mshot_total_len = sr->mshot_len = 0;
 	if (sr->flags & IORING_RECVSEND_BUNDLE) {
 		if (req->opcode == IORING_OP_SENDMSG)
 			return -EINVAL;
-- 
Jens Axboe

Re: [PATCH] io_uring/net: initialize mshot_len for send

[Gabriel Krisman Bertazi]

Jens Axboe <axboe@kernel.dk> writes:

> Commit:
>
> 6a8afb9fff64 ("io_uring/net: allow multishot receive per-invocation cap")
>
> changed how io_mshot_prep_retry() set sr->len, and added the same
> initialization in io_mshot_prep_retry(). But it neglected to touch the
> send path, which may also uses the mshot retry path. Ensure that
> sr->mshot_len always gets initialized correctly.
>
> Fixes: 6a8afb9fff64 ("io_uring/net: allow multishot receive per-invocation cap")
> Cc: stable@vger.kernel.org
> Reported-by: Sung Keum <kambodi127@gmail.com>
> Signed-off-by: Jens Axboe <axboe@kernel.dk>
>

Reviewed-by: Gabriel Krisman Bertazi <krisman@suse.de>

-- 
Gabriel Krisman Bertazi