Closed
Bug 124252
Opened 23 years ago
Closed 23 years ago
nsDocument::Init() gets called twice, giving assertion.
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla0.9.9
People
(Reporter: pj, Assigned: jst)
Details
(Whiteboard: [HAVE FIX])
Attachments
(2 files)
135 bytes,
text/html
|
Details | |
1.06 KB,
patch
|
john
:
review+
vidur
:
superreview+
|
Details | Diff | Splinter Review |
In nsDocument.cpp:395 the NS_ASSURE_TRUE(rv) gets triggered due to calling nsDocument::Init() twice with the testcase. (Will attach). This might be related to bug 117693 or 92577 (Pointer corruption, crashes happens for me at random places).
Reporter | ||
Comment 1•23 years ago
|
||
Causes nsDocument::Init() to return NS_ERROR_ALREADY_DEFINED.
Reporter | ||
Comment 2•23 years ago
|
||
Of course I mean NS_ERROR_ALREADY_INITIALIZED and not NS_ERROR_ALREADY_DEFINED, sorry.
Assignee | ||
Comment 3•23 years ago
|
||
Confirming and accepting. Patch coming up.
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Whiteboard: [HAVE FIX]
Target Milestone: --- → mozilla0.9.9
Assignee | ||
Comment 4•23 years ago
|
||
Comment 5•23 years ago
|
||
Comment on attachment 68451 [details] [diff] [review] Proposed fix. sr=vidur
Attachment #68451 -
Flags: superreview+
Comment 6•23 years ago
|
||
Comment on attachment 68451 [details] [diff] [review] Proposed fix. sr=vidur
Comment 7•23 years ago
|
||
Comment on attachment 68451 [details] [diff] [review] Proposed fix. r=jkeiser
Attachment #68451 -
Flags: review+
Assignee | ||
Comment 8•23 years ago
|
||
Fixed. I highly doubt this has anything to do with a crash. Marking FIXED.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Component: DOM: HTML → DOM: Core & HTML
QA Contact: stummala → general
You need to log in
before you can comment on or make changes to this bug.
Description
•