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
Unknown Object (File)
Mon, Apr 27, 6:54 AM
Unknown Object (File)
Sat, Apr 25, 5:04 AM
Unknown Object (File)
Fri, Apr 24, 1:22 AM
Unknown Object (File)
Mon, Apr 20, 12:50 PM
Unknown Object (File)
Sat, Apr 18, 8:58 PM
Unknown Object (File)
Sat, Apr 18, 7:59 PM
Unknown Object (File)
Sat, Apr 18, 1:02 PM
Unknown Object (File)
Sat, Apr 18, 12:40 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