diff options
| author | Jens Axboe <[email protected]> | 2019-12-15 10:57:46 -0700 |
|---|---|---|
| committer | Jens Axboe <[email protected]> | 2019-12-15 22:12:47 -0700 |
| commit | 0b416c3e1345fd696db4c422643468d844410877 (patch) | |
| tree | 0084779addbc4e9e6a5d3c190062fbebab41720a /lib/string_helpers.c | |
| parent | d195a66e367b3d24fdd3c3565f37ab7c6882b9d2 (diff) | |
io_uring: fix sporadic -EFAULT from IORING_OP_RECVMSG
If we have to punt the recvmsg to async context, we copy all the
context. But since the iovec used can be either on-stack (if small) or
dynamically allocated, if it's on-stack, then we need to ensure we reset
the iov pointer. If we don't, then we're reusing old stack data, and
that can lead to -EFAULTs if things get overwritten.
Ensure we retain the right pointers for the iov, and free it as well if
we end up having to go beyond UIO_FASTIOV number of vectors.
Fixes: 03b1230ca12a ("io_uring: ensure async punted sendmsg/recvmsg requests copy data")
Reported-by: 李通洲 <[email protected]>
Signed-off-by: Jens Axboe <[email protected]>
Diffstat (limited to 'lib/string_helpers.c')
0 files changed, 0 insertions, 0 deletions