RecordAllocationSample doesn't check if RecordAllocationSample succeeded.

VERIFIED FIXED

Status

Tamarin
Virtual Machine
VERIFIED FIXED
9 years ago
8 years ago

People

(Reporter: Erik Tierney, Assigned: Erik Tierney)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

9 years ago
Created attachment 383981 [details] [diff] [review]
Check success of recordAllocationSample

recordAllocationSample could fail because the buffer was exhausted, but recordAllocationInfo did not check for failure.  

This patch changes recordAllocationInfo to check for failure, and bail if there is no space left.

Patch from achicu@adobe.com.
Attachment #383981 - Flags: superreview?(lhansen)
Attachment #383981 - Flags: review+

Updated

9 years ago
Attachment #383981 - Flags: superreview?(lhansen) → superreview+
(Assignee)

Comment 1

9 years ago
pushed: http://hg.mozilla.org/tamarin-redux/rev/605e1010d655
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED

Comment 2

8 years ago
Resolved fixed engineering / work item that has been pushed.  Setting status to verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.