Page MenuHomeFreeBSD

cd9660: Add length checks to Rock Ridge parser
ClosedPublic

Authored by des on Apr 1 2026, 1:26 PM.
Tags
None
Referenced Files
F157721001: D56215.diff
Sun, May 24, 11:11 AM
Unknown Object (File)
Thu, May 21, 9:53 AM
Unknown Object (File)
Thu, May 21, 9:53 AM
Unknown Object (File)
Wed, May 20, 10:00 PM
Unknown Object (File)
Tue, May 19, 5:19 PM
Unknown Object (File)
Tue, May 19, 1:15 PM
Unknown Object (File)
Tue, May 19, 1:10 PM
Unknown Object (File)
Mon, May 18, 8:18 AM
Subscribers

Details

Summary
  • cd9660_rrip_slink() did not check that the lengths of individual entries do not exceed the length of the overall record.
  • cd9660_rrip_altname() did not check that the length of the record was at least 5 before subtracting 5 from it.

Note that in both cases, a better solution would be to check the length
of the data before calling the handler, or immediately upon entry of
the handler, but this would require significant refactoring.

MFC after: 1 week
Reported by: Calif.io in collaboration with Claude and Anthropic Research
Reported by: Adam Crosser, Praetorian

Diff Detail

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