Page MenuHomeFreeBSD
Differential D14812

Correctly mark last section header in El Torito catalog
ClosedPublic
Actions

Authored by benno on Mar 23 2018, 9:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 22 2024, 4:35 AM
Unknown Object (File)
Feb 14 2024, 1:48 AM
Unknown Object (File)
Jan 9 2024, 7:09 AM
Unknown Object (File)
Jan 2 2024, 7:56 PM
Unknown Object (File)
Dec 22 2023, 11:50 PM
Unknown Object (File)
Dec 21 2023, 6:36 PM
Unknown Object (File)
Dec 7 2023, 5:03 AM
Unknown Object (File)
Nov 12 2023, 7:23 PM
View All 31 Files
Subscribers
None

Details

Reviewers
emaste
imp
Commits
rS331467: Correctly mark the last El Torito section header.
Summary

The final section header in the El Torito catalog should be marked as such. We had a comment saying we'd do that but we weren't.

Test Plan

Test-booted ISO image under both UEFI and BIOS.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

benno created this revision.Mar 23 2018, 9:31 PM
Harbormaster completed remote builds in B15749: Diff 40655.Mar 23 2018, 9:31 PM
imp accepted this revision.Mar 23 2018, 9:36 PM

Does what it says...

Warner

This revision is now accepted and ready to land.Mar 23 2018, 9:36 PM
emaste added a comment.Mar 23 2018, 9:39 PM

Worth comparing with NetBSD's?

benno added a comment.Mar 23 2018, 9:44 PM

NetBSD seem to attempt this but it looks like they're actually modifying the last ET_ENTRY_SE entry, not ET_ENTRY_SH. I'm not sure how correct that is.

Closed by commit rS331467: Correctly mark the last El Torito section header. (authored by benno). · Explain WhyMar 23 2018, 10:52 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
usr.sbin/
makefs/
cd9660/
12 lines

Diff 40658

View Options

head/usr.sbin/makefs/cd9660/cd9660_eltorito.c

Loading...