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
F164852056: D56215.diff
Tue, Aug 4, 8:03 AM
Unknown Object (File)
Tue, Aug 4, 1:57 AM
Unknown Object (File)
Mon, Aug 3, 4:08 AM
Unknown Object (File)
Sun, Aug 2, 3:52 PM
Unknown Object (File)
Sun, Aug 2, 12:19 AM
Unknown Object (File)
Fri, Jul 24, 10:28 PM
Unknown Object (File)
Thu, Jul 23, 3:23 AM
Unknown Object (File)
Sat, Jul 18, 1:21 PM
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