Page MenuHomeFreeBSD
Differential D19213

cd9660: dirmatch fails to unmatch when name is prefix for directory record
ClosedPublic
Actions

Authored by tsoome on Feb 16 2019, 6:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jul 26, 5:17 PM
Unknown Object (File)
Sun, Jul 26, 1:45 PM
Unknown Object (File)
Fri, Jul 24, 8:27 PM
Unknown Object (File)
Thu, Jul 23, 2:24 PM
Unknown Object (File)
Thu, Jul 23, 5:53 AM
Unknown Object (File)
Fri, Jul 17, 1:15 AM
Unknown Object (File)
Wed, Jul 15, 10:27 AM
Unknown Object (File)
Sun, Jul 5, 12:57 PM
View All Files
Subscribers
allanjude
imp

Details

Reviewers
allanjude
Commits
rS344248: cd9660: dirmatch fails to unmatch when name is prefix for directory record
Summary

Loader does fail to properly match the file name in directory record and
does open file based on prefix match.

For fix, we check the name lengths first.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 22529
Build 21672: arc lint + arc unit

Event Timeline

tsoome created this revision.Feb 16 2019, 6:12 PM
Herald added a subscriber: imp. · View Herald TranscriptFeb 16 2019, 6:12 PM
Harbormaster completed remote builds in B22529: Diff 53990.Feb 16 2019, 6:12 PM
allanjude accepted this revision.Feb 16 2019, 7:19 PM
allanjude added a subscriber: allanjude.

Seems like a good fix, and even an optimization.

This revision is now accepted and ready to land.Feb 16 2019, 7:19 PM
Closed by commit rS344248: cd9660: dirmatch fails to unmatch when name is prefix for directory record (authored by tsoome). · Explain WhyFeb 18 2019, 8:26 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
stand/
libsa/
4 lines

Diff 53990

View Options

stand/libsa/cd9660.c

Loading...