Closed Bug 2537 Opened 26 years ago Closed 26 years ago

NECKO: [PP]included css file missing causes crash parsing xml

Categories

(Core :: DOM: HTML Parser, defect, P2)

PowerPC
Mac System 8.5
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: mikepinkerton, Assigned: gagan)

References

Details

This snippet: <?xml version="1.0"?> <?xml-stylesheet href="xul.css" type="text/css"?> <!DOCTYPE window> <window xmlns:xul ="http://www.mozilla.org/keymaster/gatekeeper/ there.is.only.xul"> Kick me <xul:checkbox name="kickme" value="buttocks"/>Buttocks <xul:checkbox name="kickme" value="hamhock"/>Hamhock </window> causes a crash if put into a file and loaded in viewer. If you provide a file called xul.css, it doesn't crash anymore.
Status: NEW → ASSIGNED
Mike -- I'm unable to reproduce this bug. Can you call me to discuss other things that exist in your context (but not mine) that may cause this bug to show up for you but not me? Thanks!
Mike -- I'm unable to reproduce this bug. Can you call me to discuss other things that exist in your context (but not mine) that may cause this bug to show up for you but not me? Thanks.
Summary: included css file missing causes crashe parsing xml → included css file missing causes crash parsing xml
Here's the stack trace from the mac. Looks more like a netlib problem. Back chain ISA Caller 00000000 PPC 02A844C8 03468B40 PPC 02A6C5FC main+0008C 03468AF0 PPC 02A6BD5C nsNativeViewerApp::Run()+00034 03468AB0 PPC 067BF928 nsAppShell::Run()+00108 034689D0 PPC 067C016C nsMacMessagePump::DoMessagePump()+001D4 03468960 PPC 06970A60 Repeater::DoRepeaters(const EventRecord&)+00058 03468900 PPC 0696A98C TimerPeriodical::RepeatAction(const EventRecord&)+ 0007C 034688A0 PPC 069698D8 TimerImpl::Fire()+00030 03468860 PPC 06931078 timer_callback(nsITimer*, void*)+00028 03468820 PPC 068B1700 net_process_net_timer_callback+0004C 034687E0 PPC 068AC550 NET_ProcessNet+00520 03468440 PPC 068BB0F0 net_ProcessFile+000F4 034683F0 PPC 068BA6C0 net_read_file_chunk+00294 034683A0 PPC 068DCF3C stub_put_block(_NET_StreamClass*, const char*, long)+000D8 03468350 PPC 0641AE64 nsDocumentBindInfo::OnDataAvailable(nsIURL*, nsIInputStream*, un signed int)+000BC 034682F0 PPC 0676BF94 nsParser::OnDataAvailable(nsIURL*, nsIInputStream*, unsigned int )+00250 03468280 PPC 0676B908 nsParser::ResumeParse(nsIDTD*)+0008C 03468230 PPC 0676BA54 nsParser::BuildModel()+0006C 034681E0 PPC 067773C4 CWellFormedDTD::BuildModel(nsIParser*, nsITokenizer* )+00080 03468180 PPC 067779DC CWellFormedDTD::HandleToken(CToken*, nsIParser*)+ 000E8 034680A0 PPC 066D18D4 nsXMLContentSink::AddProcessingInstruction(const nsIParserNode&) +005EC 03467DF0 PPC 068F17DC NS_NewUnicharStreamLoader(nsIUnicharStreamLoader**, nsIURL*, voi d (*)(nsIUnicharStreamLoader*, nsString&, void*, unsigned int), void*)+0008C 03467DA0 PPC 068F0F30 nsUnicharStreamLoader::nsUnicharStreamLoader(nsIURL* , void (*)(n sIUnicharStreamLoader*, nsString&, void*, unsigned int), void*)+000B4 03467D50 PPC 068D912C NS_OpenURL(nsIURL*, nsIStreamListener*)+00070 03467D00 PPC 06419164 nsDocLoaderImpl::OpenStream(nsIURL*, nsIStreamListener*)+00178 03467C90 PPC 0641A67C nsDocumentBindInfo::Bind(nsIURL*, nsIStreamListener* )+000A0 03467C40 PPC 068D740C nsNetlibService::OpenStream(nsIURL*, nsIStreamListener*)+00278 03467BC0 PPC 068ABFA0 NET_GetURL+01A84 034676B0 PPC 068A99E0 net_CallExitRoutine+000F0 03467660 PPC 068D97F4 bam_exit_routine(URL_Struct_*, int, MWContext_*)+ 000E8
Assignee: rickg → gagan
Status: ASSIGNED → NEW
On advice from Pink, this appears to be a netlib problem. I'm reassigning it to you.
Status: NEW → ASSIGNED
Investigating, no problem on windows though. Will run it as soon as I can get one successful Mac build going...
Setting all current Open/Normal to M4.
setting paulmac as QA contact for all gagan's bugs (sorry for the spam)
Assignee: gagan → pinkerton
Status: ASSIGNED → NEW
Mike could you confirm for me if this is still a crash? (and if so assign it back to me) Thx.
Assignee: pinkerton → gagan
Mike is out today. It still crashes, unless there is a xul.css file. I reproduced it reliably using today's optimized build. reassigning to gagan
Mike is out today. It still crashes, unless there is a xul.css file. I reproduced it reliably using today's optimized build. reassigning to gagan
Target Milestone: M4 → M6
Also crashes for me on Mac by copying this snippet into a file called "crash.xml" then opening locally on 4/11 builds. Doesn't seem like an M4 stopper though, changing to M6 unless someone screams.
Vera - Please add this to the M4 release notes. On Mac, if an included css file is missing, the application will crash. Thanks - Hubie
Summary: included css file missing causes crash parsing xml → [PP]included css file missing causes crash parsing xml
Deffered till Necko lands...
Deferring till Necko lands...
This looks like bug 5206.
Per DP's suggestion marking these till M8. Though Necko lands with M7, we will be able to verify it for M8.
*** Bug 5206 has been marked as a duplicate of this bug. ***
*** Bug 6757 has been marked as a duplicate of this bug. ***
Re-tested on 5/24 build. It looks like it isn't crashing anymore. Please resolve as appropriate.
*** Bug 2621 has been marked as a duplicate of this bug. ***
*** Bug 6699 has been marked as a duplicate of this bug. ***
I'm moving this to target M9, Necko will be enabled somewhere during late M8 or early M9. We will need to get on this and it cannot be postponed past the M9 milestone.
Summary: [PP]included css file missing causes crash parsing xml → NECKO: [PP]included css file missing causes crash parsing xml
Pl. verify with Necko.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
not crashing anymore, necko for president
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.