]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
freebsd32: Fix a double copyin in sendmsg() and recvmsg()
authorMark Johnston <markj@FreeBSD.org>
Sun, 19 Sep 2021 17:45:09 +0000 (13:45 -0400)
committerMark Johnston <markj@FreeBSD.org>
Sun, 19 Sep 2021 17:54:16 +0000 (13:54 -0400)
commitfea1a98ead918b39280b586773a923e76194400b
treeef81adaf2242fd78acc8cf0ed80f8ae585c88348
parent4bda16ff184bfca5ee4bf9709a06323d9cf5945b
freebsd32: Fix a double copyin in sendmsg() and recvmsg()

freebsd32_sendmsg() and freebsd32_recvmsg() both copyin the message
header twice, once directly and once in freebsd32_copyinmsghdr().  The
iovec length from the former is used when copying in msg_iov, but the
rest of the kernel uses the iovec length from the latter.  When
kern_sendit() and kern_recvit() iterate over the iovec to compute the
residual for I/O, they can therefore end up walking past the end of the
copied in iovec, either resulting in a system call error, userspace
memory corruption from uiomove() with invalid iovecs, or a kernel page
fault if the copied-in iovec is followed by an unmapped KVA region.

Reported by: syzbot+7cc64cd0c49605acd421@syzkaller.appspotmail.com
Reviewed by: kib, emaste
MFC after: 1 week
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D32010
sys/compat/freebsd32/freebsd32_misc.c