272588 - Crash viewing the source of the page http://www.molos.ch/cgi-bin/livreor/livreor.cgi with the extension HTML Validator [@ nstidy.dll + 0x38f5]

Status: RESOLVED FIXED

(Toolkit :: View Source, defect)

Description

[Xavier Robin]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; fr-FR; rv:1.7.5) Gecko/20041108 Firefox/1.0
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; fr-FR; rv:1.7.5) Gecko/20041108 Firefox/1.0

In the page http://www.molos.ch/cgi-bin/livreor/livreor.cgi when I try to view
the source (with extension HTML Validator installed), firefox crashes before
displaying the source code.

Reproducible: Always
Steps to Reproduce:
I could not reproduce it on another profile.
1. go to the page http://www.molos.ch/cgi-bin/livreor/livreor.cgi
2. view the source

Actual Results:  
Firefox show the view-source window and crashes before displaying the code.

Expected Results:  
Display the source.

I reported the crash with the following talkback : TB2290270M.
I strongly suspect HTML validator (see http://users.skynet.be/mgueury/mozilla/)
to cause the crash but I could not reproduce it with only it.
I will try to install others extensions in a clean profile to try to reproduce.

I use the extension HTML validator and several others (TBE for example). I use
the QuteRetro theme.

Comment 1

[Xavier Robin]

Attachment: Saved page

The crash also appears when viewing this saved page.

Comment 2

[timeless]

Stack Signature	nstidy.dll + 0x38f5 (0x019038f5) 93d43b16
Product ID	Firefox10
Build ID	2004110812
Trigger Time	2004-12-01 06:10:08.0
Platform	Win32
Operating System	Windows NT 5.1 build 2600
Module	nstidy.dll + (000038f5)
URL visited	http://www.molos.ch/cgi-bin/livreor/livreor.cgi
User Comments	I was opening the "view source" from the page (with extension HTML
validator based on tidy"
Since Last Crash	1932 sec
Total Uptime	356589 sec
Trigger Reason	Access violation
Source File, Line No.	N/A
Stack Trace 	
nstidy.dll + 0x38f5 (0x019038f5)
nstidy.dll + 0x16bd (0x019016bd)
XPTC_InvokeByIndex 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/xpcom/reflect/xptcall/src/md/win32/xptcinvoke.cpp,
line 102]
XPCWrappedNative::CallMethod 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/js/src/xpconnect/src/xpcwrappednative.cpp,
line 2034]
XPC_WN_CallMethod 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/js/src/xpconnect/src/xpcwrappednativejsops.cpp,
line 1287]
js_Invoke 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/js/src/jsinterp.c,
line 941]
js_Interpret 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/js/src/jsinterp.c,
line 2978]
js_Invoke 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/js/src/jsinterp.c,
line 958]
nsXPCWrappedJSClass::CallMethod 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/js/src/xpconnect/src/xpcwrappedjsclass.cpp,
line 1339]
nsXPCWrappedJS::CallMethod 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/js/src/xpconnect/src/xpcwrappedjs.cpp,
line 450]
SharedStub 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/xpcom/reflect/xptcall/src/md/win32/xptcstubs.cpp,
line 147]
nsEventListenerManager::HandleEventSubType 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/content/events/src/nsEventListenerManager.cpp,
line 1436]
nsEventListenerManager::HandleEvent 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/content/events/src/nsEventListenerManager.cpp,
line 1516]
nsXULElement::HandleDOMEvent 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/content/xul/content/src/nsXULElement.cpp,
line 2841]
nsXULElement::HandleChromeEvent 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/content/xul/content/src/nsXULElement.cpp,
line 3988]
GlobalWindowImpl::HandleDOMEvent 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/dom/src/base/nsGlobalWindow.cpp,
line 916]
DocumentViewerImpl::LoadComplete 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/content/base/src/nsDocumentViewer.cpp,
line 917]
nsDocShell::EndPageLoad 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/docshell/base/nsDocShell.cpp,
line 4441]
nsWebShell::EndPageLoad 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/docshell/base/nsWebShell.cpp,
line 755]
nsDocShell::OnStateChange 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/docshell/base/nsDocShell.cpp,
line 4375]
nsDocLoaderImpl::FireOnStateChange 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/uriloader/base/nsDocLoader.cpp,
line 1247]
nsDocLoaderImpl::doStopDocumentLoad 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/uriloader/base/nsDocLoader.cpp,
line 868]
nsDocLoaderImpl::OnStopRequest 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/uriloader/base/nsDocLoader.cpp,
line 696]
nsLoadGroup::RemoveRequest 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/netwerk/base/src/nsLoadGroup.cpp,
line 695]
PresShell::RemoveDummyLayoutRequest 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/layout/html/base/src/nsPresShell.cpp,
line 6581]
PresShell::ProcessReflowCommands 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/layout/html/base/src/nsPresShell.cpp,
line 6454]
ReflowEvent::HandleEvent 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/layout/html/base/src/nsPresShell.cpp,
line 6226]
PL_HandleEvent 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/xpcom/threads/plevent.c,
line 674]
0x778b0c24
nsWyciwygChannel::GetStatus 
[c:/builds/tinderbox/firefox-aviarybranch-l10n/WINNT_5.1_Clobber/mozilla/content/html/document/src/nsWyciwygChannel.cpp,
line 90]
0xac02a302

Comment 3

[mgueury]

Currently the site is down and reporting a HTTP-500 server unavailable.
I tried to reproduce with my dev version I have that is based
on a later version of tidy. But I can not see it.

I have uploaded the last version 0.32. I will release it in some days.

Some questions
- which version do you use now ? 
- what is the accessibility level that you use ? 
  -> go in extension / choose the HTML validator / options
  -> does it happen with none ?
- can try with the new version and tell if it is solved.

Thanks by advance

Comment 4

[mgueury]

Attachment: HTML Validator (based on Tidy) 0.3.2

Comment 5

[mgueury]

I have finally reproduced the issue:
- it happens with Accessibility level 2 and 3 only.
- it is due to a bug in tidy:
  -> I have submitted a bug in tidy with a fix for review:
   
http://sourceforge.net/tracker/index.php?func=detail&aid=1077430&group_id=27659&atid=390963

- I will upload a new version of the validator with the fix. 

Please double-check. Then I will close the bug as fixed.

Thanks by advance.

Comment 6

[mgueury]

Attachment: HTML Validator (based on Tidy) 0.3.2 with fix

Comment 7

[Xavier Robin]

It works fine now and the source code is correctly shown. You can close it.
Thank you.

Comment 8

[mgueury]

Version 0.4.0 has been submitted to update.mozilla.org
You can find it here also:

> http://users.skynet.be/mgueury/mozilla

There is an additional fix in it.

Thanks