summaryrefslogtreecommitdiff
path: root/tools
diff options
context:
space:
mode:
authorTom Ryan <ryan36005@gmail.com>2026-03-09 22:20:02 -0700
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2026-03-19 16:14:57 +0100
commit1f794f9bed3e5cf7250a3b4daf112a72ed1513e9 (patch)
treee886de49b06298eed3c94241fb3e31046867638d /tools
parent0cee68fb7f4cf1562e067c5a82d25062a973b0d0 (diff)
io_uring: fix physical SQE bounds check for SQE_MIXED 128-byte ops
[ Upstream commit 6f02c6b196036dbb6defb4647d8707d29b7fe95b ] When IORING_SETUP_SQE_MIXED is used without IORING_SETUP_NO_SQARRAY, the boundary check for 128-byte SQE operations in io_init_req() validated the logical SQ head position rather than the physical SQE index. The existing check: !(ctx->cached_sq_head & (ctx->sq_entries - 1)) ensures the logical position isn't at the end of the ring, which is correct for NO_SQARRAY rings where physical == logical. However, when sq_array is present, an unprivileged user can remap any logical position to an arbitrary physical index via sq_array. Setting sq_array[N] = sq_entries - 1 places a 128-byte operation at the last physical SQE slot, causing the 128-byte memcpy in io_uring_cmd_sqe_copy() to read 64 bytes past the end of the SQE array. Replace the cached_sq_head alignment check with a direct validation of the physical SQE index, which correctly handles both sq_array and NO_SQARRAY cases. Fixes: 1cba30bf9fdd ("io_uring: add support for IORING_SETUP_SQE_MIXED") Signed-off-by: Tom Ryan <ryan36005@gmail.com> Link: https://patch.msgid.link/20260310052003.72871-1-ryan36005@gmail.com Signed-off-by: Jens Axboe <axboe@kernel.dk> Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions