Closed Bug 1291124 Opened 8 years ago Closed 8 years ago

Intermittent SUMMARY: AddressSanitizer: SEGV /home/worker/workspace/build/src/js/src/vm/ObjectGroup.h:127:49 in compartment

Categories

(Core :: JavaScript Engine, defect, P3)

defect

Tracking

()

RESOLVED DUPLICATE of bug 1294747

People

(Reporter: intermittent-bug-filer, Unassigned)

Details

Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
Terrence, does this look like the ASAN version of any of the failures you're looking at?
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1291124&endday=2016-09-20&startday=2016-07-01&tree=trunk
Flags: needinfo?(terrence.d.cole)
Yup, this is definitely the same gray issue!

It looks like ReparentWrapper uses JSAPI on the, presumably gray, wrapper. I guess this is a case where we may actually want to keep the wrapper gray? Need to look deeper still.
Flags: needinfo?(terrence.d.cole)
The gray thing must be:

JS::Rooted<JSObject*> copyFrom(aCx, isProxy ? expandoObject : aObj);

I thought we had already solved this instance? Maybe I'm thinking of a different one?
This looks like the ASAN version of bug 1294747 to me.  There's nothing gray here; we're just crashing in ObjectGroup.h because we're trying to compartment() a swept object.

Ryan, has this happened after bug 1294747 landed?
Flags: needinfo?(ryanvm)
Ah, thanks for refreshing my memory!

Ryan pointed out on IRC that the last instance of this was on 7 Sept. The fix for bug 1294747 landed on 17 Sept, but the instances are rare enough that it's not so surprising if we didn't see any in the meantime. So I think this fixed.
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(ryanvm)
Resolution: --- → FIXED
Resolution: FIXED → DUPLICATE
You need to log in before you can comment on or make changes to this bug.