109685 - loading non-xml mimetype hangs synchloader

Status: RESOLVED INCOMPLETE

(Core :: XML, defect, P5)

Description

[Roland Mainz]

2001-11-10-cvs build on Solaris 7 SPARC.
Mozilla hangs in XSLT code while viewing a DocBook/XML document via a XSL
stylesheet (DocBook/XML --> XHTML stylesheel available from
http://sourceforge.net/project/showfiles.php?group_id=21935):
-- snip --
15089:	/shared/bigtmp2/mozilla/2001-11-10-cvs/objdir_ws6_xlib/dist/bin/./mozi
-----------------  lwp# 1 / thread# 4  --------------------
 fee19d70 lwp_sema_wait (fea0be78)
 fee8b2f8 _park    (fea0bdc0, fea0be78, 0, 1, feead7a8, 0) + 10c
 fee8afc0 _swtch   (5, feeaca1c, fea0be54, fea0be50, fea0be4c, fea0be48) + 3b4
 fee8e884 _reap_wait (feeb13d8, feeb4310, 0, 0, 1b, 0) + 34
 fee8e600 _reaper  (feeaca1c, feeb13d8, feead838, 37bc8, 1, fe400000) + 34
 fee9bcec _thread_start (0, 0, 0, 0, 0, 0) + 40
-----------------  lwp# 2 / thread# 2  --------------------
 fee19740 signotifywait ()
 fee8f780 _dynamiclwps (feeaca1c, feeae36c, 2, 0, fee35d58, 0) + 1c
 fee926d8 thr_yield (0, 0, 0, 0, 0, 0) + 8c
-----------------  lwp# 3 / thread# 3  --------------------
 fee19d70 lwp_sema_wait (feeb3b30)
 fee96f64 _tdb_agent (0, 0, feeb38c8, feeb3f54, feeb3b30, 0) + 44
 fee926d8 thr_yield (0, 0, 0, 0, 0, 0) + 8c
-----------------  lwp# 4 / thread# 5  --------------------
 fee17f8c poll     (fda719d8, 1, 88b8)
 fee9b438 poll     (0, 1, 3567e0, 1432f8, fda719d8, 0) + 34
 fefaf868 PR_Poll  (1432f8, 1, 3567e0, 143490, 1432f8, 1) + c
 fe10efc8 __1cYnsSocketTransportServiceDRun6M_I_ (13e5e0, 1432f8, 0, 6deb5b13,
fffffffc, fffffff8) + a4
 ff0eb9c8 __1cInsThreadEMain6Fpv_v_ (f83d0, 1480, ff1bb05c, 2634c, 4, 400) + 98
 fefb0ea8 _pt_root (1435c0, fec75d60, 1, feeb976c, 3ad38, 3ad38) + d0
 fee9bcec _thread_start (1435c0, 0, 0, 0, 0, 0) + 40
-----------------  lwp# 5  --------------------------------
 fee19d24 lwp_cond_wait (feeb3f78, feeb3f88, fec43cf0)
 fee117f0 _lwp_cond_timedwait (feeb3f78, feeb3f88, 0, 3bef37da, 0, 0) + 90
 fee8a94c _age     (feeaca1c, feead79c, 1, 0, 0, fe401000) + 90
 fee8a8b4 _qswtch  (0, 0, 0, 0, 0, 0) + 10c
-----------------  lwp# 6 / thread# 24  --------------------
 fee19d70 lwp_sema_wait (fbe31e78)
 fee8b2f8 _park    (fbe31dc0, fbe31e78, 0, 3, feead7a8, 37b38) + 10c
 fee8afc0 _swtch   (5, feeaca1c, fbe31e54, fbe31e50, fbe31e4c, fbe31e48) + 3b4
 fee89bbc cond_wait (0, 1437a8, f8400, 4356, feeaca1c, 800) + e0
 fee89abc pthread_cond_wait (f8400, 1437a8, 1437a8, 2634c, d4674, 400) + 8
 fefa9df8 PR_WaitCondVar (f83f8, ffffffff, 1, 0, ff1c3640, 0) + 150
 ff0ed194 __1cMnsThreadPoolKGetRequest6MpnJnsIThread__pnLnsIRunnable__ (143698,
862630, 0, fe24d0e0, ff1c3640, ff07e628) + 300
 ff0edfb4 __1cUnsThreadPoolRunnableDRun6M_I_ (6e0198, 6e01a4, 143698, 1400,
1518, 2400) + 7c
 ff0eb9c8 __1cInsThreadEMain6Fpv_v_ (862630, 1480, ff1bb05c, 2634c, 4, 400) + 98
 fefb0ea8 _pt_root (96fd28, feeb3188, 0, fbe11e50, 84fa18, 84fa18) + d0
 fee9bcec _thread_start (96fd28, 0, 0, 0, 0, 0) + 40
-----------------  lwp# 7  --------------------------------
 fee19d24 lwp_cond_wait (feeb3f78, feeb3f88, fc925cf0)
 fee117f0 _lwp_cond_timedwait (feeb3f78, feeb3f88, 0, 3bef37dc, 0, 0) + 90
 fee8a94c _age     (feeaca1c, feead79c, 1, 0, 0, fe401000) + 90
 fee8a8b4 _qswtch  (0, 0, 0, 0, 0, 0) + 10c
-----------------  lwp# 8 / thread# 1  --------------------
 fee19d70 lwp_sema_wait (37bf0)
 fee8b2f8 _park    (37b38, 37bf0, 0, 2, feead7a8, fea0bdc0) + 10c
 fee8afc0 _swtch   (5, feeaca1c, 37bcc, 37bc8, 37bc4, 37bc0) + 3b4
 fee89bbc cond_wait (0, aba230, 9f3808, 4356, feeaca1c, 800) + e0
 fee89abc pthread_cond_wait (9f3808, aba230, feeaca1c, 0, 0, 25610) + 8
 fefa9df8 PR_WaitCondVar (9f3800, ffffffff, 1, 2634c, aba1f4, 0) + 150
 fefaa528 PR_Wait  (6e0, ffffffff, fefbdfa0, abab94, d52e0, ff0e61bc) + ec
 ff0e5dfc PL_WaitForEvent (aba1f0, fb8c26a8, 0, 0, ffffffff, 0) + 84
 ff0e7f0c __1cQnsEventQdDueueImplMWaitForEvent6MppnHPLEvent__I_ (aba1c0,
ffbee348, 0, 0, 1, fe251940) + 58
 fb8674e4
__1cMnsSyncLoaderMLoadDocument6MpnGnsIURI_pnLnsIDocument_ppnOnsIDOMDocument__I_
(b05a10, 1, b05a28, ffbee47c, 80000000, b34280) + c44
 fb864c58 __1cJXMLParserSgetDocumentFromURI6MrknGString_pnIDocument_r1_5_
(80000000, ffbee520, 9eecb0, ffbee518, ffbee468, ffbee478) + 1c8
 fb871ec4 __1cOProcessorStateQretrieveDocument6MrknGString_3_pnENode__ (b01128,
b011b0, fb8c83e0, c00, f5c, fb8c5aa8) + 98
 fb876ff4
__1cNXSLTProcessorOprocessInclude6MrnGString_pnIDocument_pnOtxListIterator_pnOProcessorState__v_
(a92498, ffbee654, afc388, ffbee994, b01128, 13cc) + 118
 fb8769b4
__1cNXSLTProcessorPprocessTopLevel6MpnIDocument_pnHElement_pnOtxListIterator_pnOProcessorState__v_
(a92498, afc388, 10000, ffbee994, b01128, 1) + 46c
 fb8763f8
__1cNXSLTProcessorRprocessStylesheet6MpnIDocument_2pnOtxListIterator_pnOProcessorState__v_
(a92498, afc388, b30688, ffbee994, b01128, fb841ea0) + 13c
 fb877098
__1cNXSLTProcessorOprocessInclude6MrnGString_pnIDocument_pnOtxListIterator_pnOProcessorState__v_
(a92498, ffbee86c, afc388, ffbee994, b01128, 13cc) + 1bc
 fb876708
__1cNXSLTProcessorPprocessTopLevel6MpnIDocument_pnHElement_pnOtxListIterator_pnOProcessorState__v_
(a92498, afc388, ae993c, ffbee994, b01128, d) + 1c0
 fb8763f8
__1cNXSLTProcessorRprocessStylesheet6MpnIDocument_2pnOtxListIterator_pnOProcessorState__v_
(a92498, afc388, 9edc60, ffbee994, b01128, fb841ea0) + 13c
 fb87a450
__1cNXSLTProcessorRTransformDocument6MpnKnsIDOMNode_2pnOnsIDOMDocument_pnLnsIObserver__I_
(a92498, aa20c0, 9edc60, 9ef434, a79c04, fb8c26a8) + 3d4
 fcf870cc __1cTnsTransformMediatorOTryToTransform6M_v_ (a8d640, 9eecb4,
fd406d24, ffffffff, 0, 0) + dc
 fcf87124 __1cTnsTransformMediatorZSetStyleSheetContentModel6MpnKnsIDOMNode__I_
(a8d640, 9eecb4, fcf87114, 0, 0, 9eecb0) + 10
 fcf862f0 __1cQnsXSLContentSinkNDidBuildModel6Mi_I_ (a8dd18, 1, 9eecb0,
fd421340, fd0b5cbc, ae89e8) + a4
 fdf68300
__1cOCWellFormedDTDNDidBuildModel6MIipnJnsIParser_pnOnsIContentSink__I_ (ae99b8,
0, 1, 9ec9b0, a8dd18, fdf682b4) + 4c
 fdf5b9d8 __1cInsParserNDidBuildModel6MI_I_ (9ec9b0, 0, 3, 97c488, 3, 0) + 8c
 fdf5c664 __1cInsParserLResumeParse6Mii_I_ (9ec9b0, 1, 1, 0, 0, 804e03e8) + 208
 fdf5dc7c __1cInsParserNOnStopRequest6MpnKnsIRequest_pnLnsISupports_I_I_
(9ec9b0, 9edbe0, 0, 0, b9de0, 446) + c8
 fe18b2d4 __1cNnsFileChannelNOnStopRequest6MpnKnsIRequest_pnLnsISupports_I_I_
(0, 9f995c, 0, 0, 915f88, 145d84) + 7c
 fe0ff3ec __1cUnsOnStopRequestEventLHandleEvent6M_v_ (a9c620, 22c0, fe1fdb36,
fe24503c, 556d90, 0) + 138
 fe0fdba8 __1cXnsARequestObserverEventNHandlePLEvent6FpnHPLEvent__v_ (a9c624, 1,
1, a9c624, d54e0, ff0e55c0) + 58
 ff0e5bc8 PL_HandleEvent (a9c624, 0, 1000, 1358, 1000, 135c) + 58
 ff0e5a10 PL_ProcessPendingEvents (f6950, 0, d5714, d3400, ff1bb05c, 50) + cc
 ff0e7cb4 __1cQnsEventQdDueueImplUProcessPendingEvents6M_I_ (f6920, ff0e7c58,
ffbeeef8, 144c14, ffbeeffc, 144c94) + 5c
 fde29934 __1cRHandleQdDueueXtProc6FpvpipL_v_ (f6920, 5ad384, ffbef0b4,
fde29928, 1, 0) + c
 fddcac60 DoOtherSources (144b50, fde02000, 1, 13d584, 5dc, fde72e94) + d8
 fddcaa14 XtAppNextEvent (144b50, 0, 1, 0, ffbef184, 11) + 98
 fde29a60 __1cKnsAppShellDRun6M_I_ (f7c00, 0, 45a9c, fc700ab0, 49400, 0) + 10c
 fdeaa808 __1cRnsAppShellServiceDRun6M_I_ (13a998, ffbef2b4, fdef7e64, fdeaa7f4,
fc7012f4, fc6e44e8) + 14
 00017128 __1cFmain16FippcpnLnsISupports__I_ (ffbef2c8, 13a998, ffbef2cc,
ffbef300, ffbef2f0, ffbef314) + a14
 00017b2c main     (2, ffbef44c, ffbef458, 36c00, 0, 0) + 1d0
 000134a8 _start   (0, 0, 0, 0, 0, 0) + 108
-----------------  lwp# 9  --------------------------------
 fee19d24 lwp_cond_wait (feeb3f78, feeb3f88, fbdb5cf0)
 fee117f0 _lwp_cond_timedwait (feeb3f78, feeb3f88, 0, 3bef37da, 0, 0) + 90
 fee8a94c _age     (feeaca1c, feead79c, 1, 0, 0, fe401000) + 90
 fee8a8b4 _qswtch  (0, 0, 0, 0, 0, 0) + 10c
-----------------  lwp# 10 / thread# 11  --------------------
 fee19d70 lwp_sema_wait (feeae400)
 fee8a5e0 _co_timerset (feeae3f0, feead730, feeaca1c, feeae070, 3, feeaca1c) +
f4
 fee9bcec _thread_start (0, 0, 0, 0, 0, 0) + 40
-----------------  lwp# 11  --------------------------------
 fee17318 door     (0, 0, 0, 0, fcac5d60, 4)
 fee8becc _lwp_start (0, 0, 0, 0, 0, 0) + 14
--------------------------  thread# 6  --------------------
 fee89bbc cond_wait (0, 1438f8, f8478, 4356, feeaca1c, 800) + e0
 fee89abc pthread_cond_wait (f8478, 1438f8, fec20984, 292, fe096eb0, fe249780) +
8
 fefa9df8 PR_WaitCondVar (f8470, ffffffff, 1438f8, 2634c, 0, 0) + 150
 fe128ba4 __1cMnsDNSServicePDequeuePendingQdD6M_pnLnsDNSLookup__ (143868,
1438f8, 3, 4, cedec, 11cad8) + 4c
 fe128590 __1cMnsDNSServiceDRun6M_I_ (143868, fe24f234, fe12af94, 0, 2400, 2620)
+ 30
 ff0eb9c8 __1cInsThreadEMain6Fpv_v_ (f8498, 1480, ff1bb05c, 2634c, 4, 400) + 98
 fefb0ea8 _pt_root (1439b8, fec43d60, 1, feeb976c, 1435c0, 1435c0) + d0
 fee9bcec _thread_start (1439b8, 0, 0, 0, 0, 0) + 40
-- snip --

Comment 1

[Roland Mainz]

Attachment: Test document

Comment 2

[Roland Mainz]

Main thread, symbols translated to human readable format:
-- snip --
lwp_sema_wait == lwp_sema_wait
_park == _park
_swtch == _swtch
cond_wait == cond_wait
pthread_cond_wait == pthread_cond_wait
PR_WaitCondVar == PR_WaitCondVar
PR_Wait == PR_Wait
PL_WaitForEvent == PL_WaitForEvent
__1cQnsEventQdDueueImplMWaitForEvent6MppnHPLEvent__I_ == unsigned
nsEventQueueImpl::WaitForEvent(PLEvent**)
__1cMnsSyncLoaderMLoadDocument6MpnGnsIURI_pnLnsIDocument_ppnOnsIDOMDocument__I_
== unsigned nsSyncLoader::LoadDocument(nsIURI*,nsIDocument*,nsIDOMDocument**)
__1cJXMLParserSgetDocumentFromURI6MrknGString_pnIDocument_r1_5_ ==
Document*XMLParser::getDocumentFromURI(const String&,Document*,String&)
__1cOProcessorStateQretrieveDocument6MrknGString_3_pnENode__ ==
Node*ProcessorState::retrieveDocument(const String&,const String&)
__1cNXSLTProcessorOprocessInclude6MrnGString_pnIDocument_pnOtxListIterator_pnOProcessorState__v_
== void
XSLTProcessor::processInclude(String&,Document*,txListIterator*,ProcessorState*)
__1cNXSLTProcessorPprocessTopLevel6MpnIDocument_pnHElement_pnOtxListIterator_pnOProcessorState__v_
== void
XSLTProcessor::processTopLevel(Document*,Element*,txListIterator*,ProcessorState*)
__1cNXSLTProcessorRprocessStylesheet6MpnIDocument_2pnOtxListIterator_pnOProcessorState__v_
== void
XSLTProcessor::processStylesheet(Document*,Document*,txListIterator*,ProcessorState*)
__1cNXSLTProcessorOprocessInclude6MrnGString_pnIDocument_pnOtxListIterator_pnOProcessorState__v_
== void
XSLTProcessor::processInclude(String&,Document*,txListIterator*,ProcessorState*)
__1cNXSLTProcessorPprocessTopLevel6MpnIDocument_pnHElement_pnOtxListIterator_pnOProcessorState__v_
== void
XSLTProcessor::processTopLevel(Document*,Element*,txListIterator*,ProcessorState*)
__1cNXSLTProcessorRprocessStylesheet6MpnIDocument_2pnOtxListIterator_pnOProcessorState__v_
== void
XSLTProcessor::processStylesheet(Document*,Document*,txListIterator*,ProcessorState*)
__1cNXSLTProcessorRTransformDocument6MpnKnsIDOMNode_2pnOnsIDOMDocument_pnLnsIObserver__I_
== unsigned
XSLTProcessor::TransformDocument(nsIDOMNode*,nsIDOMNode*,nsIDOMDocument*,nsIObserver*)
__1cTnsTransformMediatorOTryToTransform6M_v_ == void
nsTransformMediator::TryToTransform()
__1cTnsTransformMediatorZSetStyleSheetContentModel6MpnKnsIDOMNode__I_ ==
unsigned nsTransformMediator::SetStyleSheetContentModel(nsIDOMNode*)
__1cQnsXSLContentSinkNDidBuildModel6Mi_I_ == unsigned
nsXSLContentSink::DidBuildModel(int)
__1cOCWellFormedDTDNDidBuildModel6MIipnJnsIParser_pnOnsIContentSink__I_ ==
unsigned CWellFormedDTD::DidBuildModel(unsigned,int,nsIParser*,nsIContentSink*)
__1cInsParserNDidBuildModel6MI_I_ == unsigned nsParser::DidBuildModel(unsigned)
__1cInsParserLResumeParse6Mii_I_ == unsigned nsParser::ResumeParse(int,int)
__1cInsParserNOnStopRequest6MpnKnsIRequest_pnLnsISupports_I_I_ == unsigned
nsParser::OnStopRequest(nsIRequest*,nsISupports*,unsigned)
__1cNnsFileChannelNOnStopRequest6MpnKnsIRequest_pnLnsISupports_I_I_ == unsigned
nsFileChannel::OnStopRequest(nsIRequest*,nsISupports*,unsigned)
__1cUnsOnStopRequestEventLHandleEvent6M_v_ == void
nsOnStopRequestEvent::HandleEvent()
__1cXnsARequestObserverEventNHandlePLEvent6FpnHPLEvent__v_ == void
nsARequestObserverEvent::HandlePLEvent(PLEvent*)
PL_HandleEvent == PL_HandleEvent
PL_ProcessPendingEvents == PL_ProcessPendingEvents
__1cQnsEventQdDueueImplUProcessPendingEvents6M_I_ == unsigned
nsEventQueueImpl::ProcessPendingEvents()
__1cRHandleQdDueueXtProc6FpvpipL_v_ == void
HandleQueueXtProc(void*,int*,unsigned long*)
DoOtherSources == DoOtherSources
XtAppNextEvent == XtAppNextEvent
__1cKnsAppShellDRun6M_I_ == unsigned nsAppShell::Run()
__1cRnsAppShellServiceDRun6M_I_ == unsigned nsAppShellService::Run()
__1cFmain16FippcpnLnsISupports__I_ == unsigned main1(int,char**,nsISupports*)
main == main
-- snip --

Comment 3

[Axel Hecht]

the docbook stylesheets include a xml file called VERSION. Which isn't a .xml,
so the file:// url assignes the wrong mimetype. This is partly a evang problem,
but we shouldn't hang, anyway.
Over to peterv

Comment 4

[Roland Mainz]

> Which isn't a .xml,
> so the file:// url assignes the wrong mimetype. This is partly a evang 
> problem,

We should not rely on the file's name to gets it's mimetype. A content-based
mimetype detection (as used in OpenGroups CDE) is far more reliable... :-)

Comment 5

[Axel Hecht]

bug 122305 has a patch that might show the way for this one.
The spec says only for document() to ignore the mimetype and force XML parsing,
but this would make sense here as well. And fixing it in nsSyncLoader is prolly
the right place

Comment 6

[scottputterman]

ADT3 per ADT triage.

Comment 7

[Axel Hecht]

*** Bug 133268 has been marked as a duplicate of this bug. ***

Comment 8

[scottputterman]

Changing nsbeta1+ [adt3] bugs to nsbeta1- on behalf of the adt.  If you have any
questions about this, please email [e-mail].  You can search for
"changing adt3 bugs" to quickly find and delete these bug mails.

Comment 9

[scottputterman]

Changing nsbeta1+ [adt3] bugs to nsbeta1- on behalf of the adt.  If you have any
questions about this, please email [e-mail].  You can search for
"changing adt3 bugs" to quickly find and delete these bug mails.

Comment 10

[Boris Zbarsky [:bzbarsky]]

*** Bug 140958 has been marked as a duplicate of this bug. ***

Comment 11

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

changing component now that the syncloader lives in content

Comment 12

[rbs]

gisburn, do you mind attaching a stack trace for bug 184159? I suspect that if
you make a stack trace for that bug, it should look similar to the one that you
attached here (modulo the fact that the sync loader now lives in content). Why
the sync loader is hanging at all is a mystery to me.

Comment 13

[Axel Hecht]

rbs, the parser doesn't report the wrong mime type as error. it just drops the
load without notifying the contentsink. and so noone tells the modal loop to
stop.

Comment 14

[rbs]

Is it something like that that is happening in bug 184159 too?

Comment 15

[rbs]

(I meant that "noone tells the modal loop to stop" due to an error or
something.)

Comment 16

[rbs]

It turns out that bug 189338 has a stack trace (attachment 111801 [details]) which is
essentially the same as in this bug. Therefore it is probable that these are the
same bugs. So the support of <script for="..."> seems to have just exposed a bug
that has been there all along.

For ease of reference, I am going to mark the dependencies between the bugs.

Comment 17

[Sylvestre Ledru [:Sylvestre]]

Decreasing the priority as no update for the last 2 years on this bug.
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage 
about the priority meaning.

Comment 18

[Andrei Purice]

Hey Axel,
Does this bug reproduce for you or can we close it?

Comment 19

[Axel Hecht]

I haven't been involved in XSLT/XML in over a decade, this isn't on me.

Comment 20

[Andrei Purice]

Marking this as Resolved > Incomplete since the reporter cannot be contacted further to confirm if the issue is still reproducible.
If anyone can reproduce this issue please re-open the bug or file a new one.