Closed
Bug 189779
Opened 22 years ago
Closed 22 years ago
crashes quitting [@ nsDiskCacheOutputStream::~nsDiskCacheOutputStream]
Categories
(Core :: Networking: Cache, defect, P1)
Tracking
()
VERIFIED
FIXED
mozilla1.3beta
People
(Reporter: dbaron, Assigned: darin.moz)
References
Details
(Keywords: crash, regression, topcrash)
Crash Data
Attachments
(1 file)
1.61 KB,
patch
|
KaiE
:
review+
bzbarsky
:
superreview+
|
Details | Diff | Splinter Review |
Crashes at nsDiskCacheOutputStream::~nsDiskCacheOutputStream have started showing up in talkback starting on 2003-01-18. I suspect this is related to darin's landing. User comments seem to show the crashes are occuring on quit, at least in some cases. Here's some summary info, although it only includes half the crashes that are mentioned in the -detailed report. nsDiskCacheOutputStream::~nsDiskCacheOutputStream 5 BBID range: 16338868 - 16346156 Min/Max Seconds since last crash: 47 - 9572 Min/Max Runtime: 47 - 18641 Crash data range: 2003-01-18 to 2003-01-18 Build ID range: 2003011808 to 2003011808 Keyword List : Stack Trace: nsDiskCacheOutputStream::~nsDiskCacheOutputStream [c:/builds/seamonkey/mozilla/netwerk/cache/src/nsDiskCacheStreams.cpp line 217] nsDiskCacheOutputStream::`scalar deleting destructor' nsSOCKSSocketInfo::Release [c:/builds/seamonkey/mozilla/netwerk/socket/base/nsSOCKSIOLayer.cpp line 93] nsCOMPtr_base::~nsCOMPtr_base [c:/builds/seamonkey/mozilla/xpcom/glue/nsCOMPtr.cpp line 65] nsCacheEntryDescriptor::nsOutputStreamWrapper::`scalar deleting destructor' nsCacheEntryDescriptor::nsOutputStreamWrapper::Release [c:/builds/seamonkey/mozilla/netwerk/cache/src/nsCacheEntryDescriptor.cpp line 561] nsCOMPtr_base::~nsCOMPtr_base [c:/builds/seamonkey/mozilla/xpcom/glue/nsCOMPtr.cpp line 65] nsWyciwygChannel::~nsWyciwygChannel [c:/builds/seamonkey/mozilla/content/html/document/src/nsWyciwygChannel.cpp line 56] nsWyciwygChannel::Release [c:/builds/seamonkey/mozilla/content/html/document/src/nsWyciwygChannel.cpp line 56] nsCOMPtr_base::~nsCOMPtr_base [c:/builds/seamonkey/mozilla/xpcom/glue/nsCOMPtr.cpp line 65] nsHTMLDocument::~nsHTMLDocument [c:/builds/seamonkey/mozilla/content/html/document/src/nsHTMLDocument.cpp line 364] nsDocument::Release [c:/builds/seamonkey/mozilla/content/base/src/nsDocument.cpp line 669] SubDocClearEntry [c:/builds/seamonkey/mozilla/content/base/src/nsDocument.cpp line 1232] PL_DHashTableFinish [c:/builds/seamonkey/mozilla/xpcom/ds/pldhash.c line 324] PL_DHashTableDestroy [c:/builds/seamonkey/mozilla/xpcom/ds/pldhash.c line 165] nsXULDocument::~nsXULDocument [c:/builds/seamonkey/mozilla/content/xul/document/src/nsXULDocument.cpp line 462] nsXULDocument::`scalar deleting destructor' nsXULDocument::Release [c:/builds/seamonkey/mozilla/content/xul/document/src/nsXULDocument.cpp line 558] nsCOMPtr_base::~nsCOMPtr_base [c:/builds/seamonkey/mozilla/xpcom/glue/nsCOMPtr.cpp line 65] nsEventStateManager::~nsEventStateManager [c:/builds/seamonkey/mozilla/content/events/src/nsEventStateManager.cpp line 265] nsEventStateManager::`scalar deleting destructor' nsEventStateManager::Release [c:/builds/seamonkey/mozilla/content/events/src/nsEventStateManager.cpp line 300] nsCOMPtr_base::~nsCOMPtr_base [c:/builds/seamonkey/mozilla/xpcom/glue/nsCOMPtr.cpp line 65] nsPresContext::~nsPresContext [c:/builds/seamonkey/mozilla/layout/base/src/nsPresContext.cpp line 127] GalleyContext::`scalar deleting destructor' nsPresContext::Release [c:/builds/seamonkey/mozilla/layout/base/src/nsPresContext.cpp line 245] nsDOMEvent::~nsDOMEvent [c:/builds/seamonkey/mozilla/content/events/src/nsDOMEvent.cpp line 266] nsDOMEvent::`scalar deleting destructor' nsDOMEvent::Release [c:/builds/seamonkey/mozilla/content/events/src/nsDOMEvent.cpp line 284] XPCJSRuntime::GCCallback [c:/builds/seamonkey/mozilla/js/src/xpconnect/src/xpcjsruntime.cpp line 539] js_GC [c:/builds/seamonkey/mozilla/js/src/jsgc.c line 1408] js_ForceGC [c:/builds/seamonkey/mozilla/js/src/jsgc.c line 994] nsCOMPtr_base::assign_from_helper [c:/builds/seamonkey/mozilla/xpcom/glue/nsCOMPtr.cpp line 78] nsDOMSOFactory::Observe [c:/builds/seamonkey/mozilla/dom/src/build/nsDOMFactory.cpp line 230] nsObserverService::NotifyObservers [c:/builds/seamonkey/mozilla/xpcom/ds/nsObserverService.cpp line 212] NS_ShutdownXPCOM [c:/builds/seamonkey/mozilla/xpcom/build/nsXPComInit.cpp line 691] GRE_Shutdown [c:/builds/seamonkey/mozilla/xpcom/glue/standalone/nsXPCOMGlue.cpp line 254] main [c:/builds/seamonkey/mozilla/xpfe/bootstrap/nsAppRunner.cpp line 1941] WinMain [c:/builds/seamonkey/mozilla/xpfe/bootstrap/nsAppRunner.cpp line 1951] WinMainCRTStartup() kernel32.dll + 0x214c7 (0x77e814c7) Source File : c:/builds/seamonkey/mozilla/netwerk/cache/src/nsDiskCacheStreams.cpp line : 217 (16346156) Comments: Shuting browser down (16342741) Comments: quit mail via "X" button
Reporter | ||
Updated•22 years ago
|
Flags: blocking1.3b+
Assignee | ||
Updated•22 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → mozilla1.3beta
Comment 1•22 years ago
|
||
Adding regression and zt4newcrash keywords. This is a new crash introduced in the past 72 hours...and either needs to be fixed or the checkin that caused it should be backed out.
Comment 2•22 years ago
|
||
*** Bug 189874 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 3•22 years ago
|
||
dumb mistake on my part when i was making changes to the cache. i did not completely convert from nsCOMPtr<nsDiskCacheStreamIO> to nsDiskCacheStreamIO* :-(
Comment 4•22 years ago
|
||
Comment on attachment 112203 [details] [diff] [review] v1 patch This patch works very well for me, I can't make it crash with this fix applied. r=kaie
Updated•22 years ago
|
Attachment #112203 -
Flags: review+
Assignee | ||
Comment 5•22 years ago
|
||
for reference, kai sent me a 100% reproducible testcase. this caused profile switching to crash and burn if initiated while a page was still loading. requires the latest patch in bug 97622 to repro.
Comment 6•22 years ago
|
||
Comment on attachment 112203 [details] [diff] [review] v1 patch + NS_ASSERTION(mStreamIO == nsnull, "leak"); Maybe !mStreamIO instead of the comparison to nsnull?
Attachment #112203 -
Flags: superreview+
Assignee | ||
Updated•22 years ago
|
Attachment #112203 -
Flags: superreview?(bzbarsky)
Assignee | ||
Updated•22 years ago
|
Attachment #112203 -
Flags: superreview?(bzbarsky)
Updated•22 years ago
|
Blocks: profile-switching
Assignee | ||
Comment 7•22 years ago
|
||
patch checked in yesterday; marking FIXED.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 8•22 years ago
|
||
*** Bug 190028 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 9•22 years ago
|
||
Marking verified based on lack of talkback reports with this signature for 3 days.
Status: RESOLVED → VERIFIED
Comment 10•21 years ago
|
||
*** Bug 190142 has been marked as a duplicate of this bug. ***
Updated•13 years ago
|
Crash Signature: [@ nsDiskCacheOutputStream::~nsDiskCacheOutputStream]
You need to log in
before you can comment on or make changes to this bug.
Description
•