Closed Bug 21302 Opened 25 years ago Closed 25 years ago

[CRASH] fox news, followed by another page crashes

Categories

(Core :: Networking, defect, P3)

PowerPC
Mac System 8.5
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: mikepinkerton, Assigned: gordon)

Details

go to www.foxnews.com
click on any link on that page

crash

 Calling chain using A6/R1 links
  Back chain  ISA  Caller
  00000000    PPC  0AC33368
  0B80B080    PPC  0AC2DB40  main+00198
  0B80AFF0    PPC  0AC2C51C  main1(int, char**)+006AC
  0B80AF10    PPC  0DD3B500  nsAppShellService::Run()+00020
  0B80AED0    PPC  0DACBD80  nsAppShell::Run()+00050
  0B80AE50    PPC  0DACC680  nsMacMessagePump::DoMessagePump()+00044
  0B80AE00    PPC  0DACC9A4  nsMacMessagePump::DispatchEvent(int, EventRecord*)+
001A4
  0B80ADB0    PPC  0F053C1C  Repeater::DoRepeaters(const EventRecord&)+00034
  0B80AD70    PPC  0DAA7E58  nsMacNSPREventQueueHandler::RepeatAction(const
EventRecord&)+000
2C
  0B80AD30    PPC  0DB86D28  nsEventQueueServiceImpl::ProcessEvents()+00070
  0B80ACE0    PPC  0DBA1BD4  nsEventQueueImpl::ProcessPendingEvents()+00018
  0B80ACA0    PPC  0E62A8A0  PL_ProcessPendingEvents+0007C
  0B80AC50    PPC  0E62AA18  PL_HandleEvent+00094
  0B80AC10    PPC  0E62AAA4  PL_DestroyEvent+00048
  0B80ABD0    PPC  0DD063C0  nsStreamListenerEvent::DestroyPLEvent(PLEvent*)+
00064
 Closing log
Severity: major → critical
Bulk move of all Necko (to be deleted component) bugs to new Networking

component.
Assignee: gagan → gordon
Target Milestone: M13
-> gordon
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
We weren't ref counting the nsDNSListener that was passed into the DNS lookup,
which could cause a crash if it had been deallocated before the lookup completed.
I made the mListener field of nsDNSLookup a COMPtr, and no longer crash this way.
verified - 2000012008
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.