643847 - "Assertion failure: !entry->vindex,"

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

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

function f(o) {
  o.constructor = function() {};
}
__proto__.__defineSetter__('constructor',
function(v) {});
f({});
Object.defineProperty(__proto__, 'constructor', {
  writable: true,
});
f({});

asserts js debug shell on TM changeset c811be25eaad without -m nor -j at Assertion failure: entry->vcapTag() == 0

Not the smallest regression window:

http://hg.mozilla.org/tracemonkey/pushloghtml?fromchange=71c0268fb372&tochange=301b97a20042

Comment 1

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

Tested on 64-bit shell.

Comment 2

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

Testing on 32-bit shell in 64-bit Ubuntu Linux 10.04,

autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   54587:7ef107ab081e
user:        Brendan Eich
date:        Thu Sep 16 11:56:54 2010 -0700
summary:     Fix shape vs. slot management under putProperty, plus related layering and error reporting fixes (596805, r=jorendorff).

This means the regression window in comment #0 is incorrect.

Comment 3

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

Still occurs in m-c changeset 5c8405e6226e which has now morphed to:

Assertion failure: !entry->vindex,

Comment 4

[Christian Holler (:decoder)]

Could be a duplicate of bug 637202.

Comment 5

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

Fixed by bug 713944.

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

Comment 6

[Christian Holler (:decoder)]

Automatically extracted testcase for this bug was committed:

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