Closed Bug 538165 Opened 15 years ago Closed 15 years ago

Harden GCHeap against new virtual memory API behavior

Categories

(Tamarin Graveyard :: Garbage Collection (mmGC), defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: treilly, Assigned: treilly)

Details

Attachments

(1 file)

Desktop platforms reserve API's have always returned either NULL or the address you asked for but on wince and linux we're seeing results that are sometimes close to the asked for address and the GCHeap needs to handle this.  This also is what our VMPI docs say can happen ironically.
Looking for quick review, this is a player/tr rectification fix (although player needs clean up too)
Assignee: nobody → treilly
Status: NEW → ASSIGNED
Attachment #420332 - Flags: superreview?(lhansen)
Attachment #420332 - Attachment is patch: true
Attachment #420332 - Attachment mime type: application/octet-stream → text/plain
Comment on attachment 420332 [details] [diff] [review]
handles VMPI_reserveMemory returning different address than asked for

Presumably in the test

newRegionAddr != NULL && newRegionAddr == region->reserveTop

the first conjunct is redundant because reserveTop is never NULL?
Attachment #420332 - Flags: superreview?(lhansen) → superreview+
changeset 3459	5bcf95ed4a0a
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Engineering work item.  Marking as verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: