Page MenuHomeFreeBSD
Differential D26339

imgact_elf.c: unify check for phdr fitting into the first page
ClosedPublic
Actions

Authored by kib on Sep 5 2020, 7:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 5, 11:22 PM
Unknown Object (File)
Tue, Nov 4, 2:20 AM
Unknown Object (File)
Thu, Oct 30, 12:18 PM
Unknown Object (File)
Mon, Oct 27, 7:00 AM
Unknown Object (File)
Thu, Oct 23, 11:58 PM
Unknown Object (File)
Thu, Oct 23, 11:58 PM
Unknown Object (File)
Thu, Oct 23, 3:28 PM
Unknown Object (File)
Thu, Oct 23, 3:28 PM
View All 82 Files
Subscribers
imp

Details

Reviewers
emaste
dim
Commits
rS365433: imgact_elf.c: unify check for phdr fitting into the first page.
Summary

Similar to the userspace rtld check.

Diff Detail

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

Event Timeline

kib created this revision.Sep 5 2020, 7:05 PM
Herald added a subscriber: imp. · View Herald TranscriptSep 5 2020, 7:05 PM
kib requested review of this revision.Sep 5 2020, 7:05 PM
Harbormaster completed remote builds in B33414: Diff 76694.Sep 5 2020, 7:05 PM
dim accepted this revision.Sep 5 2020, 7:16 PM

LGTM, nice catch!

This revision is now accepted and ready to land.Sep 5 2020, 7:16 PM
emaste added inline comments.Sep 5 2020, 7:18 PM
sys/kern/imgact_elf.c
455 ↗(On Diff #76694)

hrm, actually shouldn't this be <= PAGE_SIZE - hdr->e_phoff)

kib updated this revision to Diff 76700.Sep 5 2020, 8:56 PM
kib edited the summary of this revision. (Show Details)

Unfix off by one.

P.S. Then rtld-elf/map_object.c needs to be unfixed as well.

This revision now requires review to proceed.Sep 5 2020, 8:56 PM
Harbormaster completed remote builds in B33415: Diff 76700.Sep 5 2020, 8:56 PM
emaste added a comment.Sep 5 2020, 9:08 PM

P.S. Then rtld-elf/map_object.c needs to be unfixed as well.

Yes, in that review I thought it was fixing an off-by-one, but now that I look at it again here I think it's correct this way (now).

dim accepted this revision.Sep 7 2020, 7:06 PM
This revision is now accepted and ready to land.Sep 7 2020, 7:06 PM
emaste accepted this revision.Sep 7 2020, 7:18 PM
Closed by commit rS365433: imgact_elf.c: unify check for phdr fitting into the first page. (authored by kib). · Explain WhySep 7 2020, 9:37 PM
This revision was automatically updated to reflect the committed changes.
kib added a commit: rS365433: imgact_elf.c: unify check for phdr fitting into the first page..

Revision Contents
Changeset List

PathSize
head/
sys/
kern/
14 lines

Diff 76743

View Options

head/sys/kern/imgact_elf.c

Loading...