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)
Thu, Nov 14, 6:37 PM
Unknown Object (File)
Sep 30 2024, 7:32 PM
Unknown Object (File)
Sep 23 2024, 12:43 AM
Unknown Object (File)
Sep 12 2024, 4:00 PM
Unknown Object (File)
Sep 5 2024, 7:10 PM
Unknown Object (File)
Sep 3 2024, 4:05 PM
Unknown Object (File)
Jul 25 2024, 12:29 AM
Unknown Object (File)
Jul 16 2024, 11:57 PM
View All 35 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...