vm_phys_avail_count(): Fix out-of-bounds accesses
Needs ReviewPublic
Actions

Authored by olce on Thu, Jan 23, 5:15 PM.

Details

Reviewers

markj
dougm
alc

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.Thu, Jan 23, 5:15 PM

Herald added a subscriber: imp. · View Herald TranscriptThu, Jan 23, 5:15 PM

olce requested review of this revision.Thu, Jan 23, 5:15 PM

Harbormaster completed remote builds in B61928: Diff 149785.Thu, Jan 23, 5:15 PM

olce added a parent revision: D48628: vm_phys: Check for overlap when adding a segment.Thu, Jan 23, 5:21 PM

olce added a child revision: D48630: vm_phys_avail_split(): Tolerate split requests at boundaries.

Revision Contents
Changeset List

Path

Size

sys/

vm/

vm_phys.c

10 lines

Diff 149785

View Options

vm_phys_avail_count(): Fix out-of-bounds accessesNeeds ReviewPublicActions