Assertion failure: JS::StringIsASCII(message.get()) with parseBin shell function

RESOLVED FIXED in Firefox 63

Status

()

defect
RESOLVED FIXED
11 months ago
11 months ago

People

(Reporter: anba, Assigned: Waldo)

Tracking

Trunk
mozilla63
Points:
---

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

Test case:
---
parseBin(new ArrayBuffer(0), {format: "\u{101}"})
---

Asserts with:
---
Assertion failure: JS::StringIsASCII(message.get()), at /home/andre/git/mozilla-central/js/src/vm/JSContext.cpp:413
---
I imagine I'm racing you to land this, now that I see what's buggy.  Oh well.
Attachment #9002946 - Flags: review?(andrebargull)
Assignee: nobody → jwalden+bmo
Status: NEW → ASSIGNED
Comment on attachment 9002946 [details] [diff] [review]
Don't pass the result of ValueToPrintableUTF8 to a function called JS_ReportErrorASCII

Review of attachment 9002946 [details] [diff] [review]:
-----------------------------------------------------------------

(In reply to Jeff Walden [:Waldo] from comment #1)
> I imagine I'm racing you to land this, now that I see what's buggy.  Oh well.

I didn't have immediate plans to work on this bug. The only thing this patch does is that it'll create a small merge conflict with bug 1485066 for me. :-)
Attachment #9002946 - Flags: review?(andrebargull) → review+
Pushed by jwalden@mit.edu:
https://hg.mozilla.org/integration/mozilla-inbound/rev/03ad53dbb0ae
Don't pass the result of ValueToPrintableUTF8 to a function called JS_ReportErrorASCII.  r=anba
https://hg.mozilla.org/mozilla-central/rev/03ad53dbb0ae
Status: ASSIGNED → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.