1170355 - Assertion failure: INT_FITS_IN_JSID(i), at dist/include/js/Id.h

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

x = Array(4294967295);
x[1] = 0;
Array.prototype.shift.call(x);

asserts js debug shell on m-c changeset f8d21278244b with --fuzzing-safe --no-threads --no-ion at Assertion failure: INT_FITS_IN_JSID(i), at dist/include/js/Id.h.

Configure options:

CC="clang -Qunused-arguments" CXX="clang++ -Qunused-arguments" AR=ar AUTOCONF=/usr/local/Cellar/autoconf213/2.13/bin/autoconf213 sh /Users/skywalker/trees/mozilla-central/js/src/configure --target=x86_64-apple-darwin12.5.0 --enable-debug --enable-nspr-build --enable-more-deterministic --with-ccache --enable-gczeal --enable-debug-symbols --disable-tests

python -u ~/fuzzing/js/compileShell.py -b "--enable-debug --enable-more-deterministic --enable-nspr-build" -r f8d21278244b

=== Treeherder Build Bisection Results by autoBisect ===

The "good" changeset has the timestamp "20150519065123" and the hash "f7054968c36b".
The "bad" changeset has the timestamp "20150519065830" and the hash "1410ca139039".

Likely regression window: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=f7054968c36b&tochange=1410ca139039

Brian, is bug 1163091 a likely regressor?

Comment 1

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

Attachment: stack

(lldb) bt 5
* thread #1: tid = 0x52348, 0x000000010006f7e0 js-dbg-64-dm-nsprBuild-darwin-f8d21278244b`js::array_shift(JSContext*, unsigned int, JS::Value*) + 52 at Id.h:103, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
  * frame #0: 0x000000010006f7e0 js-dbg-64-dm-nsprBuild-darwin-f8d21278244b`js::array_shift(JSContext*, unsigned int, JS::Value*) + 52 at Id.h:103
    frame #1: 0x000000010006f7ac js-dbg-64-dm-nsprBuild-darwin-f8d21278244b`js::array_shift(cx=<unavailable>, argc=<unavailable>, vp=<unavailable>) + 908 at jsarray.cpp:2172
    frame #2: 0x00000001001f23cf js-dbg-64-dm-nsprBuild-darwin-f8d21278244b`js::Invoke(JSContext*, JS::CallArgs, js::MaybeConstruct) [inlined] js::CallJSNative(cx=0x00000001028a5180, native=0x000000010006f420)(JSContext*, unsigned int, JS::Value*), JS::CallArgs const&) + 176 at jscntxtinlines.h:235
    frame #3: 0x00000001001f231f js-dbg-64-dm-nsprBuild-darwin-f8d21278244b`js::Invoke(cx=0x00000001028a5180, args=CallArgs at 0x00007fff5fbfdfa0, construct=<unavailable>) + 447 at Interpreter.cpp:727
    frame #4: 0x00000001007af5a3 js-dbg-64-dm-nsprBuild-darwin-f8d21278244b`js::fun_call(cx=0x00000001028a5180, argc=<unavailable>, vp=<unavailable>) + 339 at jsfun.cpp:1216
(lldb)

Comment 2

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

The top line of the testcase in comment 0 came from:

js/src/tests/js1_5/Array/regress-157652.js

Comment 3

[Brian Hackett [Laid off!]]

Attachment: patch

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b476ead4954b

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b476ead4954b