339685 - "TypeError: d has invalid __iterator__ value undefined" after setting d.__proto__ to null

Status: VERIFIED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jesse Ruderman]

Comment 1

[Jesse Ruderman]

Attachment: testcase

Comment 2

[Jesse Ruderman]

Regressed between the May 17 nightly and the May 21 nightly.  Guessing the fix for bug 326466 (pythonic generators and iteration) is to blame.

Comment 3

[James Ross]

Just for my notes, this is also occurring in Venkman when trying to enumerate the properties of the Call scope object during debugging.

Comment 4

[Blake Kaplan (:mrbkap) (inactive)]

I think this is an important bug. The problem is that we look up d.__iterator__, execept that __iterator__ lives on Object.prototype, and because d.__proto__ = null, we don't find it and are unable to iterate over it. This will bite anybody trying to use objects as dictionaries or otherwise wants to enumerate an object whose prototype chain doesn't either define __iterator__ seperately or lead to Object.prototype.

Comment 5

[Brendan Eich [:brendan]]

Attachment: fix

mrbkap says r=him.

/be

Comment 6

[Brendan Eich [:brendan]]

Fixed on JS_1_7_ALPHA_BRANCH and trunk.

/be

Comment 7

[Bob Clary [:bc] (inactive)]

Checking in regress-339685.js;
/cvsroot/mozilla/js/tests/js1_5/Regress/regress-339685.js,v  <--  regress-339685.js
initial revision: 1.1

Comment 8

[Bob Clary [:bc] (inactive)]

verified fixed 1.9 windows/mac(ppc|tel)/linux 20060810