Closed Bug 1176430 Opened 9 years ago Closed 9 years ago

Disable js1_8_5/extensions/recursion.js on ASAN

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: shu, Unassigned)

References

Details

Attachments

(1 file)

A test testing blowing the stack in ASAN seems like a bad mix. My patches over in bug 1165486 somehow caused this test to intermittently crash on ASAN with a null deref.

I can't reproduce it on my normal Linux machine or a VM with similar RAM constraints as the infra testers. I can't successfully get a debug build built on the tester machines to debug.

I give up and am going to disable the test on ASAN.
Attached patch rollup.patchSplinter Review
Christian, see if you can get recursion.js to fail with this patch set.
Attachment #8624959 - Flags: feedback?(choller)
Blocks: 1165486
Talked with decoder on IRC. He'd rather not have it disabled. In that case, somebody else can land bug 1165486 and debug this.
I managed to fix this after much suffering. See patch over in bug 1165486.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Attachment #8624959 - Flags: feedback?(choller)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: