Last Comment Bug 699347 - Crash when viewing source of an XML file that has an XML declaration without the encoding pseudo-attribute [@ nsCharTraits<wchar_t>::length(wchar_t const*) ]
: Crash when viewing source of an XML file that has an XML declaration without ...
: crash
Product: Core
Classification: Components
Component: HTML: Parser (show other bugs)
: Trunk
: All All
-- critical (vote)
: mozilla10
Assigned To: Henri Sivonen (:hsivonen)
: Andrew Overholt [:overholt]
Depends on:
  Show dependency treegraph
Reported: 2011-11-03 01:37 PDT by Ekanan Ketunuti
Modified: 2013-07-09 13:03 PDT (History)
3 users (show)
hsivonen: in‑testsuite+
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Null-check the encoding obtained from expat (4.36 KB, patch)
2011-11-03 02:51 PDT, Henri Sivonen (:hsivonen)
bugs: review+
Details | Diff | Splinter Review

Description User image Ekanan Ketunuti 2011-11-03 01:37:59 PDT
1. start latest nightly build
2. open about:addons or about:memory or about:config or about:permissions or chrome://browser/content/places/places.xul or chrome://global/content/console.xul
3. press ctrl+u


0 	xul.dll 	nsCharTraits<wchar_t>::length 	obj-firefox/dist/include/nsCharTraits.h:378
1 	xul.dll 	nsDependentString::nsDependentString 	obj-firefox/dist/include/nsTDependentString.h:90
2 	xul.dll 	nsHtml5StreamParser::MaybeSetEncodingFromExpat 	parser/html/nsHtml5StreamParser.cpp:411
3 	xul.dll 	HandleXMLDeclaration 	parser/html/nsHtml5StreamParser.cpp:440
4 	xul.dll 	processXmlDecl 	parser/expat/lib/xmlparse.c:3491
5 	xul.dll 	doProlog 	parser/expat/lib/xmlparse.c:3855
6 	xul.dll 	prologProcessor 	parser/expat/lib/xmlparse.c:3759
7 	xul.dll 	prologInitProcessor 	parser/expat/lib/xmlparse.c:3575
8 	xul.dll 	MOZ_XML_Parse 	parser/expat/lib/xmlparse.c:1520
9 	xul.dll 	nsHtml5StreamParser::FinalizeSniffing 	
10 	xul.dll 	nsHtml5StreamParser::SniffStreamBytes 	parser/html/nsHtml5StreamParser.cpp:709
11 	xul.dll 	nsHtml5StreamParser::DoDataAvailable 	parser/html/nsHtml5StreamParser.cpp:1019
12 	xul.dll 	nsHtml5DataAvailable::Run 	parser/html/nsHtml5StreamParser.cpp:1063
13 	xul.dll 	nsThread::ProcessNextEvent 	xpcom/threads/nsThread.cpp:631
14 	xul.dll 	nsThreadStartupEvent::Run 	xpcom/threads/nsThread.cpp:201
15 	nspr4.dll 	_PR_NativeRunThread 	nsprpub/pr/src/threads/combined/pruthr.c:426
16 	nspr4.dll 	pr_root 	nsprpub/pr/src/md/windows/w95thred.c:122
17 	msvcr80.dll 	msvcr80.dll@0x29ba 	
18 	msvcr80.dll 	msvcr80.dll@0x2a46 	
19 	ntdll.dll 	__RtlUserThreadStart 	
20 	ntdll.dll 	_RtlUserThreadStart 

last good
first bad Kyle Huey — Merge m-i to m-c. 

last good
first bad


Bug 482921 - [HTML5] Re-implement view source syntax highlighting using the HTML5 parser
Comment 1 User image Henri Sivonen (:hsivonen) 2011-11-03 02:51:58 PDT
Created attachment 571581 [details] [diff] [review]
Null-check the encoding obtained from expat
Comment 2 User image Henri Sivonen (:hsivonen) 2011-11-03 07:34:52 PDT
Comment 3 User image Ed Morley [:emorley] 2011-11-03 13:06:58 PDT

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