io_uring/cmd_net: use READ_ONCE() for ->addr3 read

Any SQE read should use READ_ONCE(), to ensure the result is read once
and only once. Doesn't really matter for this case, but it's better to
keep these 100% consistent and always use READ_ONCE() for the prep side
of SQE handling.

Fixes: 5d24321e4c15 ("io_uring: Introduce getsockname io_uring cmd")
Signed-off-by: Jens Axboe <axboe@kernel.dk>

diff --git a/io_uring/cmd_net.c b/io_uring/cmd_net.c
index 57ddaf87461173da718714ded0cbe40c69955f8c..125a81c520a6cb7b71af677bba5c8de0ff64f44d 100644 (file)
--- a/io_uring/cmd_net.c
+++ b/io_uring/cmd_net.c
@@ -146,7 +146,7 @@ static int io_uring_cmd_getsockname(struct socket *sock,
                return -EINVAL;
 
        uaddr = u64_to_user_ptr(READ_ONCE(sqe->addr));
-       ulen = u64_to_user_ptr(sqe->addr3);
+       ulen = u64_to_user_ptr(READ_ONCE(sqe->addr3));
        peer = READ_ONCE(sqe->optlen);
        if (peer > 1)
                return -EINVAL;