Page MenuHomeFreeBSD
Differential D48629

vm_phys_avail_count(): Fix out-of-bounds accesses
ClosedPublic
Actions

Authored by olce on Jan 23 2025, 5:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 23, 9:17 PM
Unknown Object (File)
Tue, Nov 18, 9:20 AM
Unknown Object (File)
Sun, Nov 16, 4:54 PM
Unknown Object (File)
Fri, Nov 14, 11:32 PM
Unknown Object (File)
Fri, Nov 14, 10:18 PM
Unknown Object (File)
Fri, Nov 14, 8:30 PM
Unknown Object (File)
Fri, Nov 14, 8:27 PM
Unknown Object (File)
Thu, Nov 13, 1:11 AM
View All 72 Files
Subscribers
emaste
imp

Details

Reviewers
markj
dougm
alc
Commits
rG7658c4f8c727: vm_phys_avail_count(): Fix out-of-bounds accesses
rG291b7bf071e8: vm_phys_avail_count(): Fix out-of-bounds accesses
Summary

On improper termination of phys_avail[] (two consecutive 0 starting at
an even index), this function would (unnecessarily) continue searching
for the termination markers even if the index was out of bounds.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 61928
Build 58812: arc lint + arc unit

Event Timeline

olce created this revision.Jan 23 2025, 5:15 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 23 2025, 5:15 PM
olce requested review of this revision.Jan 23 2025, 5:15 PM
Harbormaster completed remote builds in B61928: Diff 149785.Jan 23 2025, 5:15 PM
olce added a parent revision: D48628: vm_phys: Check for overlap when adding a segment.Jan 23 2025, 5:21 PM
olce added a child revision: D48630: vm_phys_avail_split(): Tolerate split requests at boundaries.
markj added inline comments.Jan 24 2025, 2:47 PM
sys/vm/vm_phys.c
1784
olce updated this revision to Diff 150690.Feb 7 2025, 5:20 PM
olce marked an inline comment as done.

Fix spacing.

Harbormaster completed remote builds in B62296: Diff 150690.Feb 7 2025, 5:20 PM
olce added a comment.Feb 7 2025, 5:20 PM

Spacing fixed.

markj accepted this revision.Feb 18 2025, 8:05 PM
This revision is now accepted and ready to land.Feb 18 2025, 8:05 PM
Closed by commit rG291b7bf071e8: vm_phys_avail_count(): Fix out-of-bounds accesses (authored by olce). · Explain WhyFeb 19 2025, 2:14 PM
This revision was automatically updated to reflect the committed changes.
olce added a commit: rG291b7bf071e8: vm_phys_avail_count(): Fix out-of-bounds accesses.
olce added a commit: rG7658c4f8c727: vm_phys_avail_count(): Fix out-of-bounds accesses.Apr 8 2025, 1:41 PM

Revision Contents
Changeset List

PathSize
sys/
vm/
10 lines

Diff 149785

View Options

sys/vm/vm_phys.c

Loading...