Last Comment Bug 664123 - "ASSERTION: Why don't we have a document type node?" and null deref
: "ASSERTION: Why don't we have a document type node?" and null deref
Status: RESOLVED FIXED
: assertion, crash, regression, testcase
Product: Core
Classification: Components
Component: DOM: Core & HTML (show other bugs)
: Trunk
: x86_64 Mac OS X
: -- critical (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
:
: Andrew Overholt [:overholt]
Mentors:
Depends on:
Blocks: 326633 659053
  Show dependency treegraph
 
Reported: 2011-06-14 06:06 PDT by Jesse Ruderman
Modified: 2011-06-15 14:20 PDT (History)
7 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
testcase (crashes Firefox when loaded) (64 bytes, text/html)
2011-06-14 06:06 PDT, Jesse Ruderman
no flags Details
stack trace for the assertion (2.06 KB, text/plain)
2011-06-14 06:12 PDT, Jesse Ruderman
no flags Details

Description Jesse Ruderman 2011-06-14 06:06:20 PDT
Created attachment 539182 [details]
testcase (crashes Firefox when loaded)

###!!! ASSERTION: Why don't we have a document type node?: 'docType1 && docType2', file content/base/src/nsGenericElement.cpp, line 905

Regression from somewhere in here:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=4e1b20229d4d&tochange=1b11c64ffcd
Comment 1 Jesse Ruderman 2011-06-14 06:12:16 PDT
Created attachment 539183 [details]
stack trace for the assertion
Comment 2 :Ms2ger (⌚ UTC+1/+2) 2011-06-14 07:18:33 PDT
http://hg.mozilla.org/mozilla-central/rev/f4ff8d137c1f

   5.110 +        nsCOMPtr<nsIDOMDocumentType> docType1 = do_QueryInterface(this);
   5.111 +        nsCOMPtr<nsIDOMDocumentType> docType2 = do_QueryInterface(aOther);

This is bogus...
Comment 3 Jonas Sicking (:sicking) No longer reading bugmail consistently 2011-06-14 08:40:19 PDT
Why was this cause for backing out all of the original patches?
Comment 4 Boris Zbarsky [:bz] (still a bit busy) 2011-06-15 13:22:55 PDT
So this is fixed now, right?
Comment 5 Jonas Sicking (:sicking) No longer reading bugmail consistently 2011-06-15 14:20:53 PDT
Yes, by updated patches in bug 659053

Note You need to log in before you can comment on or make changes to this bug.