Page MenuHomeFreeBSD

netsmb: Add bounds checking to smb_t2_placedata
ClosedPublic

Authored by jhb on Jul 28 2023, 10:24 PM.
Tags
None
Referenced Files
F108572959: D41229.diff
Sun, Jan 26, 12:24 PM
Unknown Object (File)
Fri, Dec 27, 9:52 PM
Unknown Object (File)
Dec 2 2024, 9:49 AM
Unknown Object (File)
Dec 2 2024, 9:49 AM
Unknown Object (File)
Dec 2 2024, 8:58 AM
Unknown Object (File)
Dec 2 2024, 8:43 AM
Unknown Object (File)
Nov 20 2024, 9:36 PM
Unknown Object (File)
Nov 18 2024, 2:44 PM

Details

Summary

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>
Sponsored by: The FreeBSD Foundation

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable