Re: [PATCH] io_uring/nop: fix file reference leak with IOSQE_FIXED_FILE
From: Jens Axboe
Date: Mon Jun 15 2026 - 12:15:49 EST
On Mon, 15 Jun 2026 16:45:57 +0200, Vasileios Almpanis wrote:
> NOP file-acquisition support choses between a fixed (registered) file and
> a normal fget()'d file based on its own IORING_NOP_FIXED_FILE flag in
> sqe->nop_flags. However, a request's REQ_F_FIXED_FILE is set
> independently from the generic IOSQE_FIXED_FILE sqe flag during request
> init, before the issue handler runs.
>
> If a NOP is submitted with IOSQE_FIXED_FILE set (so REQ_F_FIXED_FILE is
> set) but without IORING_NOP_FIXED_FILE, io_nop() takes the normal path
> and grabs a real reference via io_file_get_normal(). On completion,
> io_put_file() only drops the reference when REQ_F_FIXED_FILE is clear,
> so the fget()'d file is never released and leaks:
>
> [...]
Applied, thanks!
[1/1] io_uring/nop: fix file reference leak with IOSQE_FIXED_FILE
commit: 571999ce5d657440f22e7f97c21ba86886fa987b
Best regards,
--
Jens Axboe