browser only, regressed after 2007073104 build. global.foo = eval; global.a = 'global'; expect = 'global indirect'; => actual = String(['a+" indirect"'].map(global.foo)); uncaught exception: [Exception... "Illegal value" nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)" location: "JS frame :: http://test.mozilla.com/tests/mozilla.org/js/js1_6/Regress/regress-382509.js :: test :: line 61" data: no] bug 389878 ?
Moving to a better component and removing the accidentally-set security flag.
Assignee: general → nobody
QA Contact: general → xpconnect
Created attachment 274871 [details] [diff] [review] Proposed fix I'd written this exact patch at some point, but must have decided I didn't need it for some (wrong) reason.
Comment on attachment 274871 [details] [diff] [review] Proposed fix You might want to complete the first sentence in the comment, other than that r+sr=jst
Comment on attachment 274871 [details] [diff] [review] Proposed fix This allows someone to pull a function off of a XOW and call it with a different 'this' object. The patch only affects this case, and should be very safe.
Attachment #274871 - Flags: approval1.9?
Attachment #274871 - Flags: approval1.9? → approval1.9+
Fix checked into trunk.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.