io_uring: set @poll->file after @poll init
authorPavel Begunkov <asml.silence@gmail.com>
Sun, 21 Jun 2020 10:09:52 +0000 (13:09 +0300)
committerJens Axboe <axboe@kernel.dk>
Mon, 22 Jun 2020 02:46:05 +0000 (20:46 -0600)
It's a good practice to modify fields of a struct after but not before
it was initialised. Even though io_init_poll_iocb() doesn't touch
poll->file, call it first.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
fs/io_uring.c

index 93af915..cc1f2f3 100644 (file)
@@ -4596,8 +4596,8 @@ static __poll_t __io_arm_poll_handler(struct io_kiocb *req,
        struct io_ring_ctx *ctx = req->ctx;
        bool cancel = false;
 
-       poll->file = req->file;
        io_init_poll_iocb(poll, mask, wake_func);
+       poll->file = req->file;
        poll->wait.private = req;
 
        ipt->pt._key = mask;