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)
Fri, Dec 13, 5:56 AM
Unknown Object (File)
Wed, Dec 11, 8:14 PM
Unknown Object (File)
Thu, Dec 5, 6:25 PM
Unknown Object (File)
Oct 23 2024, 10:58 PM
Unknown Object (File)
Oct 3 2024, 10:43 PM
Unknown Object (File)
Oct 1 2024, 9:04 PM
Unknown Object (File)
Oct 1 2024, 6:54 PM
Unknown Object (File)
Sep 27 2024, 9:29 PM
View All 57 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...