InlineWriteBarrierGuardedTrap need to assert more aggressively that container is a GCObject

RESOLVED FIXED in Q3 11 - Serrano

Status

P3
normal
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: treilly, Assigned: pnkfelix)

Tracking

unspecified
Q3 11 - Serrano

Details

(Whiteboard: fixed-in-tr)

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
Having the asserts buried under BarrierActive() isn't good.
(swipe)

This should be easy to implement; the "hard" part is doing the appropriate sandbox builds (including player land).

(I'll unassign myself if I discover that I shouldn't have put sarcasm-quotes around the word "hard" above. :)
Assignee: nobody → fklockii
Created attachment 516607 [details] [diff] [review]
unbury barrier asserts.

Updated

8 years ago
Priority: -- → P3
Target Milestone: --- → Q3 11 - Serrano

Updated

8 years ago
Priority: P3 → P4

Comment 3

8 years ago
Fix it or move it to Anza.
Priority: P4 → P3
Comment on attachment 516607 [details] [diff] [review]
unbury barrier asserts.

This looks like it will survive ATS runs.  Lets get it in there!
Attachment #516607 - Flags: superreview?(lhansen)
Attachment #516607 - Flags: review?(treilly)
(Reporter)

Updated

8 years ago
Attachment #516607 - Flags: review?(treilly) → review+

Updated

8 years ago
Attachment #516607 - Flags: superreview?(lhansen) → superreview+

Comment 5

8 years ago
changeset: 6341:60ea629f807a
user:      Felix S Klock II <fklockii@adobe.com>
summary:   Bug 636744: unbury barrier asserts (r=treilly, sr=lhansen).

http://hg.mozilla.org/tamarin-redux/rev/60ea629f807a
(Assignee)

Updated

8 years ago
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
(whoops, still need to push to tr-serrano)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Updated

8 years ago
Whiteboard: fixed-in-tr
(Assignee)

Updated

8 years ago
Status: REOPENED → RESOLVED
Last Resolved: 8 years ago8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.