io_uring/net: correct the type of variable

musamaanjum · axboe · commit 86bcacc957fc · 2024-03-04T16:33:11.000-07:00
The namelen is of type int. It shouldn't be made size_t which is unsigned. The signed number is needed for error checking before use. Fixes: c559780 ("io_uring/net: move receive multishot out of the generic msghdr path") Signed-off-by: Muhammad Usama Anjum <usama.anjum@collabora.com> Link: https://lore.kernel.org/r/20240301144349.2807544-1-usama.anjum@collabora.com Signed-off-by: Jens Axboe <axboe@kernel.dk>
diff --git a/io_uring/net.c b/io_uring/net.c
@@ -551,7 +551,7 @@ int io_send(struct io_kiocb *req, unsigned int issue_flags)
 
 static int io_recvmsg_mshot_prep(struct io_kiocb *req,
 				 struct io_async_msghdr *iomsg,
-				 size_t namelen, size_t controllen)
+				 int namelen, size_t controllen)
 {
 	if ((req->flags & (REQ_F_APOLL_MULTISHOT|REQ_F_BUFFER_SELECT)) ==
 			  (REQ_F_APOLL_MULTISHOT|REQ_F_BUFFER_SELECT)) {

Original file line number	Diff line number	Diff line change
`@@ -551,7 +551,7 @@ int io_send(struct io_kiocb *req, unsigned int issue_flags)`
`551`	`551`
`552`	`552`	`static int io_recvmsg_mshot_prep(struct io_kiocb *req,`
`553`	`553`	`struct io_async_msghdr *iomsg,`
`554`		`- size_t namelen, size_t controllen)`
	`554`	`+ int namelen, size_t controllen)`
`555`	`555`	`{`
`556`	`556`	`if ((req->flags & (REQ_F_APOLL_MULTISHOT\|REQ_F_BUFFER_SELECT)) ==`
`557`	`557`	`(REQ_F_APOLL_MULTISHOT\|REQ_F_BUFFER_SELECT)) {`