mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-08-31 09:20:13 -04:00
fuse: wait for FR_FINISHED on abort_on_kill to prevent use-after-free
The abort_on_kill path in request_wait_answer() calls fuse_abort_conn()
and returns without waiting for FR_FINISHED. If fuse_dev_do_write() is
concurrently processing the same request (FR_LOCKED set), the caller
frees req->args while it is still being accessed, causing a
use-after-free.
Fix this by jumping to the existing wait_event(FR_FINISHED) instead of
returning early. The wait will not hang because fuse_abort_conn()
ensures all requests are ended.
Reported-by: syzbot+d6540a3fa1626e11360d@syzkaller.appspotmail.com
Fixes: 204aa22a68 ("fuse: abort on fatal signal during sync init")
Cc: stable@vger.kernel.org
Assisted-by: Claude:claude-opus-4-6
Signed-off-by: Rochan Avlur <rochan.avlur@gmail.com>
Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
This commit is contained in:
committed by
Miklos Szeredi
parent
6e64df0f73
commit
64b0b5cacb
@@ -724,7 +724,7 @@ static void request_wait_answer(struct fuse_req *req)
|
||||
|
||||
if (req->args->abort_on_kill) {
|
||||
fuse_chan_abort(fch, false);
|
||||
return;
|
||||
goto wait_for_finish;
|
||||
}
|
||||
|
||||
if (test_bit(FR_URING, &req->flags))
|
||||
@@ -735,6 +735,7 @@ static void request_wait_answer(struct fuse_req *req)
|
||||
return;
|
||||
}
|
||||
|
||||
wait_for_finish:
|
||||
/*
|
||||
* Either request is already in userspace, or it was forced.
|
||||
* Wait it out.
|
||||
|
||||
Reference in New Issue
Block a user