Page MenuHomeFreeBSD

cd9660: Add length checks to Rock Ridge parser
ClosedPublic

Authored by des on Wed, Apr 1, 1:26 PM.
Tags
None
Referenced Files
F152616978: D56215.diff
Thu, Apr 16, 1:01 AM
Unknown Object (File)
Mon, Apr 13, 5:15 PM
Unknown Object (File)
Sat, Apr 11, 12:54 AM
Unknown Object (File)
Wed, Apr 8, 8:12 AM
Unknown Object (File)
Wed, Apr 8, 8:10 AM
Unknown Object (File)
Tue, Apr 7, 5:36 AM
Unknown Object (File)
Tue, Apr 7, 3:55 AM
Unknown Object (File)
Mon, Apr 6, 11:58 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