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
F81626542: D19213.id53990.diff
Fri, Apr 19, 4:57 AM
Unknown Object (File)
Tue, Apr 16, 11:18 PM
Unknown Object (File)
Dec 30 2023, 4:23 AM
Unknown Object (File)
Dec 22 2023, 10:49 PM
Unknown Object (File)
Nov 23 2023, 11:29 AM
Unknown Object (File)
Sep 30 2023, 1:56 AM
Unknown Object (File)
Aug 9 2023, 12:33 AM
Unknown Object (File)
Jul 21 2023, 8:18 PM
View All 22 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...