Page MenuHomeFreeBSD

aio: Fix up the opcode in aiocb32_copyin()
ClosedPublic

Authored by markj on Sep 11 2021, 4:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jul 12, 4:41 PM
Unknown Object (File)
Tue, Jul 7, 5:09 PM
Unknown Object (File)
Sat, Jul 4, 3:52 AM
Unknown Object (File)
Fri, Jul 3, 12:29 PM
Unknown Object (File)
Fri, Jul 3, 1:08 AM
Unknown Object (File)
Thu, Jul 2, 5:34 AM
Unknown Object (File)
Wed, Jul 1, 2:32 PM
Unknown Object (File)
Tue, Jun 30, 6:37 PM
Subscribers

Details

Summary

With lio_listio(2), the opcode is specified by userspace rather than
being hard-coded by the system call (e.g., aio_readv() -> LIO_READV).
kern_lio_listio() calls aio_aqueue() with an opcode of LIO_NOP, which
gets fixed up when the aiocb is copied in.

When copying in a job request, we need to dynamically allocate a uio to
wrap the iovec. So aiocb_copyin() needs to get the opcode from the
aiocb and then decide whether an allocation is required. We failed to
do this in the COMPAT_FREEBSD32 case. Fix it.

Reported by: syzbot+27eab6f2c2162f2885ee@syzkaller.appspotmail.com
Fixes: f30a1ae8d529 ("lio_listio(2): Allow LIO_READV and LIO_WRITEV.")

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 41468
Build 38357: arc lint + arc unit