dtrace: Fix DOF section bounds validation
ClosedPublic
Actions

Authored by markj on Jun 30 2026, 9:13 PM.

Details

Reviewers

gnn
christos

Group Reviewers

DTrace

Commits

rGa8954ff89caa: dtrace: Fix DOF section bounds validation
rG0c829bb55a86: dtrace: Fix DOF section bounds validation
rG8dc98f4d25a3: dtrace: Fix DOF section bounds validation

Summary

We must ensure that each DOF section does not overlap with the DOF
header or section table. Otherwise the relocations processed in the
second pass over sections can manipulate DOF metadata, leading to OOB
writes.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Jun 30 2026, 9:13 PM

Herald added a reviewer: gnn. · View Herald TranscriptJun 30 2026, 9:13 PM

Herald added a subscriber: imp. · View Herald Transcript

markj requested review of this revision.Jun 30 2026, 9:13 PM

markj added a parent revision: D57975: dtrace: Improve DOF section size validation.

markj added a child revision: D57977: dtrace: Improve DOF string table validation.

Harbormaster completed remote builds in B74433: Diff 181116.Jun 30 2026, 9:13 PM

christos accepted this revision.Thu, Jul 2, 3:37 PM

This revision is now accepted and ready to land.Thu, Jul 2, 3:37 PM

Closed by commit rG8dc98f4d25a3: dtrace: Fix DOF section bounds validation (authored by markj). · Explain WhyMon, Jul 6, 1:28 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG8dc98f4d25a3: dtrace: Fix DOF section bounds validation.

markj added a commit: rG0c829bb55a86: dtrace: Fix DOF section bounds validation.Mon, Jul 27, 7:15 PM

markj added a commit: rGa8954ff89caa: dtrace: Fix DOF section bounds validation.Mon, Jul 27, 10:33 PM

Revision Contents
Changeset List

Path

Size

sys/

cddl/

contrib/

opensolaris/

uts/

common/

dtrace/

dtrace.c

12 lines

Diff 181352

View Options

sys/cddl/contrib/opensolaris/uts/common/dtrace/dtrace.c

dtrace: Fix DOF section bounds validationClosedPublicActions