io_uring: fix complete_post use ctx after free
authorPavel Begunkov <asml.silence@gmail.com>
Sun, 14 Mar 2021 20:57:09 +0000 (20:57 +0000)
committerJens Axboe <axboe@kernel.dk>
Mon, 15 Mar 2021 15:32:24 +0000 (09:32 -0600)
commit180f829fe4026bd192447d261e712b6cb84f6202
tree0e3dd92a7f5d7ae9951ea247c8320be12e9d7c5e
parentefe814a471e0e58f28f1efaf430c8784a4f36626
io_uring: fix complete_post use ctx after free

If io_req_complete_post() put not a final ref, we can't rely on the
request's ctx ref, and so ctx may potentially be freed while
complete_post() is in io_cqring_ev_posted()/etc.

In that case get an additional ctx reference, and put it in the end, so
protecting following io_cqring_ev_posted(). And also prolong ctx
lifetime until spin_unlock happens, as we do with mutexes, so added
percpu_ref_get() doesn't race with ctx free.

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