Closed
Bug 308395
Opened 19 years ago
Closed 19 years ago
JS_Assert is uncatchable, even in a debugger
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mrbkap, Assigned: mrbkap)
Details
Attachments
(1 file)
(deleted),
patch
|
brendan
:
review+
|
Details | Diff | Splinter Review |
Currently, JS_Assert currently calls abort() unconditionally if the current platform is not WIN32. This hurts on Linux, since by the time gdb catches the SIGABRT the stack has been totally corrupted and it's impossible to go back and inspect what happened to cause the assertion. OS2 uses asm("int $3") currently, and Linux can too, I'll attach a patch that uses the preprocessor test given in xpcom/base/nsDebugImpl.cpp.
Updated•19 years ago
|
Attachment #195971 -
Flags: review?(brendan) → review+
Assignee | ||
Comment 2•19 years ago
|
||
Fix checked into trunk. I don't think this needs to go in on the branch.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•19 years ago
|
Flags: testcase-
You need to log in
before you can comment on or make changes to this bug.
Description
•