This is indeed a bug, I've tweaked your patch slightly to clear the pending exception after we've rooted it in the allocated stack, but other than that, it looks good.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P3
Target Milestone: --- → mozilla1.8.1
Created attachment 237060 [details] [diff] [review] Tweaked patch
Attachment #237060 - Flags: review?(brendan)
Comment on attachment 237060 [details] [diff] [review] Tweaked patch Great -- thanks for the patch, and thanks to mrbkap for helping get it in. /be
Attachment #237060 - Flags: review?(brendan) → review+
It looks like this bug is a duplicate of bug 345734
*** Bug 345734 has been marked as a duplicate of this bug. ***
(In reply to comment #4) > It looks like this bug is a duplicate of bug 345734 Thanks for naming that bug, I knew you knew it. Can you vouch for the patch here? Thanks again, /be
>Can you vouch for the patch here? I know my original problem involved an object too. Not sure if it was **exactly** the same. My ability to reproduce this was not 100% due to the other weird crashes and corruptions I kept bumping into. There was another entry in the js-engine newsgroup on 8/25 that referred to this problem specifically. I gave the guy a hack suggestion that worked for me. Maybe take a look at what I wrote and see if you can reconcile the two?
Flags: blocking1.8.1? → blocking1.8.1+
I just checked this into the trunk.
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Attachment #237060 - Flags: approval1.8.1?
Comment on attachment 237060 [details] [diff] [review] Tweaked patch a=schrep for drivers.
Attachment #237060 - Flags: approval1.8.1? → approval1.8.1+
Checking in regress-350650-n.js; /cvsroot/mozilla/js/tests/js1_5/Exceptions/regress-350650-n.js,v <-- regress-350650-n.js initial revision: 1.1
verified fixed 1.9 20060909 windows/mac*/linux
Status: RESOLVED → VERIFIED
can we get this landed on 1.8.1 branch and mark fixed1.8.1.
I am a CVS branch monkey. /be
verified fixed 1.8 20060914 windows/linux 1.9 20060914 windows/mac*/linux
Keywords: fixed1.8.1 → verified1.8.1
You need to log in before you can comment on or make changes to this bug.