HomeFreeBSD

netsmb: Add bounds checking to smb_t2_placedata

Description

netsmb: Add bounds checking to smb_t2_placedata

Verify that the requested region of the mbuf chain is not beyond the
end of the chain before trimming it from the end. If it is out of
bounds, fail with an error (EPROTO).

While here, properly handle the case that the amount of data at the
end of the chain might span more than one mbuf by using m_adj to drop
the extra bytes rather than assuming m_len of the last mbuf can be
adjusted directly.

PR: 258504
Reported by: Robert Morris <rtm@lcs.mit.edu>
Co-authored-by: Robert Morris <rtm@lcs.mit.edu>
MFC after: 1 week
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D41229

(cherry picked from commit aca3d65fedffbbe71399a88d33ea8ecf550177eb)

Details

Provenance
jhbAuthored on Aug 4 2023, 11:42 PM
Differential Revision
D41229: netsmb: Add bounds checking to smb_t2_placedata
Parents
rG16f142865cb8: cd9660: Reject volumes with small logical block sizes
Branches
Unknown
Tags
Unknown