Closed Bug 361558 Opened 18 years ago Closed 18 years ago

"Assertion failure: sprop->setter != js_watch_set" with watch on __proto__

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: jruderman, Assigned: brendan)

Details

(Keywords: crash, testcase)

Attachments

(1 file)

js> ({}.__proto__.watch('x', print)); ({}.watch('x', print));
Assertion failure: sprop->setter != js_watch_set, at jsdbgapi.c:557

Seems harmless in opt.
Attached patch fixSplinter Review
Assignee: general → brendan
Status: NEW → ASSIGNED
Attachment #246343 - Flags: review?(mrbkap)
Attachment #246343 - Flags: review?(mrbkap) → review+
Fixed on trunk:

Checking in jsdbgapi.c;
/cvsroot/mozilla/js/src/jsdbgapi.c,v  <--  jsdbgapi.c
new revision: 3.76; previous revision: 3.75
done

This patch should be folded into the branch patch for bug 361552, to save another nomination/approval cycle. It's just an assertion fix.

/be
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
RCS file: /cvsroot/mozilla/js/tests/js1_5/Regress/regress-361558.js,v
done
Checking in regress-361558.js;
/cvsroot/mozilla/js/tests/js1_5/Regress/regress-361558.js,v  <--  regress-361558.js
initial revision: 1.1
done
Flags: in-testsuite+
not mac only.

verified fixed 20061125 1.9 windows/linux
Status: RESOLVED → VERIFIED
OS: Mac OS X 10.4 → All
Hardware: Macintosh → All
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: