Closed Bug 1182092 Opened 9 years ago Closed 9 years ago

Assertion failure: !isIndex(&dummy), at c:\users\mozilla\debug-builds\mozilla-central\js\src\vm/String.h:1409

Categories

(Core :: JavaScript Engine, defect)

Unspecified
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: cbook, Assigned: bhackett1024)

Details

(Keywords: assertion)

Attachments

(3 files)

Attached file bughunter stack
found via bughunter  and reproduced on windows 7 trunk debug build

For the url please ask me 

--> Assertion failure: !isIndex(&dummy), at c:\users\mozilla\debug-builds\mozilla-central\js\src\vm/String.h:1409
This seems related to unboxed objects.
Flags: needinfo?(bhackett1024)
Attached patch patchSplinter Review
I can't get the page to load, but this should fix the assertion.  newPlainObject() tries to filter out objects that have strings which are indexes, by checking IdIsIndex on each property in objects it creates.  You might think this would be enough to catch index properties, but IdIsIndex actually calls StringIsArrayIndex, which does something totally different.
Assignee: nobody → bhackett1024
Flags: needinfo?(bhackett1024)
Attachment #8632946 - Flags: review?(jdemooij)
This assertion failure shouldn't lead to any bad behavior.
Group: core-security
Attachment #8632946 - Flags: review?(jdemooij) → review+
https://hg.mozilla.org/mozilla-central/rev/40227d8273cc
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: