Closed
Bug 85225
Opened 23 years ago
Closed 23 years ago
Mail crashes at startup M091 [@ nsMsgStatusFeedback::OnStateChange]
Categories
(SeaMonkey :: MailNews: Message Display, defect, P1)
SeaMonkey
MailNews: Message Display
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.2
People
(Reporter: namachi, Assigned: sspitzer)
Details
(Keywords: crash, topcrash, Whiteboard: PDT+; fix eta, 6-21-01, r=varada, sr=alecf, a=chofmann critical for 0.9.2)
Crash Data
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
Early M091 topcrash report
Stack Trace:
nsMsgStatusFeedback::OnStateChange
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgStatusFeedback.cpp line
131]
nsDocLoaderImpl::FireOnStateChange
[d:\builds\seamonkey\mozilla\uriloader\base\nsDocLoader.cpp line 1096]
nsDocLoaderImpl::doStopDocumentLoad
[d:\builds\seamonkey\mozilla\uriloader\base\nsDocLoader.cpp line 736]
nsDocLoaderImpl::DocLoaderIsEmpty
[d:\builds\seamonkey\mozilla\uriloader\base\nsDocLoader.cpp line 633]
nsDocLoaderImpl::OnStopRequest
[d:\builds\seamonkey\mozilla\uriloader\base\nsDocLoader.cpp line 564]
nsLoadGroup::RemoveRequest
[d:\builds\seamonkey\mozilla\netwerk\base\src\nsLoadGroup.cpp line 517]
imgRequestProxy::OnStopRequest
[d:\builds\seamonkey\mozilla\modules\libpr0n\src\imgRequestProxy.cpp line 381]
imgRequest::OnStopRequest
[d:\builds\seamonkey\mozilla\modules\libpr0n\src\imgRequest.cpp line 639]
ProxyListener::OnStopRequest
[d:\builds\seamonkey\mozilla\modules\libpr0n\src\imgLoader.cpp line 373]
nsStreamListenerTee::OnStopRequest
[d:\builds\seamonkey\mozilla\netwerk\base\src\nsStreamListenerTee.cpp line 25]
nsHttpChannel::OnStopRequest
[d:\builds\seamonkey\mozilla\netwerk\protocol\http\src\nsHttpChannel.cpp line
2068]
nsOnStopRequestEvent::HandleEvent
[d:\builds\seamonkey\mozilla\netwerk\base\src\nsRequestObserverProxy.cpp line
159]
PL_HandleEvent [d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c
line 591]
PL_ProcessPendingEvents
[d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c line 524]
_md_EventReceiverProc
[d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c line 1072]
KERNEL32.DLL + 0x24407 (0xbff94407)
0x00688b5e
Source File :
http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/mailnews/base/src/nsMsgStatu
sFeedback.cpp line : 131
(31565628) Comments: Clicked on the messenger icon in the browser window.
Note many text aren't displayed in the GUI (menu items
(31554279) Comments: I was trying to start the message center
(31529427) Comments: Loading the e-mail part of mozillaI had just applied
the modern theme and could not see any options for mail+news preferences in the
preferences window. The controls were (radio buttons etc..) there but the text
was missing.
(31517482) Comments: I tried opening the mail-client by clicking on the
icon two times
(31513972) Comments: Started Mozilla as Administrator on Win2000 wit 'run
as....'some entries of the menues did not appear....clickung around resulted in
the access violation
(31510635) Comments: Tried to Configure E-Mail. E-Mail is already
configured in Netscape 6.01 and I use a different Profile with Mozilla
(31483197) Comments: Opened Mozilla Mail and News for the second time
(31483126) Comments: I opened Mozilla Mail and News for the very first time
(31477520) Comments: Started up mail client
(31468819) Comments: Visited the Mozilla 0.9.1 milestone release site
Reporter | ||
Updated•23 years ago
|
I believe I am experiencing the same bug on the MacIntosh. I am using the Nightly
Mozilla build from June 13 (using the Modern Theme) on a G3 Mac with OS 8.6. Here
is what happens:
1. Launch Mozilla
2. Use the Mac menu bar to select mail under tasks and select Mail.
3. Mozilla crashes before another screen appears.
I believe the paltform should be changed to include the MacIntosh.
(Sorry if this isn't the same bug.)
Comment 2•23 years ago
|
||
Updated•23 years ago
|
Keywords: nsenterprise
Comment 3•23 years ago
|
||
Putting in 0.9.2 This is showing up in some trunk builds as well. One of the
linux comments from 0.9.1 has:
"0x80004005 (NS_ERROR_FAILURE)"
location: "JS frame ::
XStringBundle.stringBundle (getter) ::
onget :: line 5"
data: no] Failed to get
stringbundle: [Exception... "Component
returned failure code: 0x80004005
(NS_ERROR_FAILURE)
Priority: -- → P1
Target Milestone: --- → mozilla0.9.2
Comment 5•23 years ago
|
||
Please update the status whiteboard with an eta.
Whiteboard: PDT+ → PDT+; need eta
Assignee | ||
Comment 6•23 years ago
|
||
varada and I working on a fix...
Status: NEW → ASSIGNED
OS: Windows NT → All
Hardware: PC → All
Whiteboard: PDT+; need eta → PDT+; bullet proof fix eta, 6-21-01
Assignee | ||
Comment 7•23 years ago
|
||
I think the problem is mBundle is a static nsCOMPtr.
there's a bug (or document) about why not to use static nsCOMPtrs, I'll go look
for it.
Whiteboard: PDT+; bullet proof fix eta, 6-21-01 → PDT+; fix eta, 6-21-01
Assignee | ||
Comment 8•23 years ago
|
||
Assignee | ||
Comment 9•23 years ago
|
||
r=varada
Whiteboard: PDT+; fix eta, 6-21-01 → PDT+; fix eta, 6-21-01, r=varada, sr=?, a=?
Comment 10•23 years ago
|
||
+ NS_ENSURE_TRUE(mBundle, NS_ERROR_NULL_POINTER);
should be NS_ENSURE_ARG_POINTER(mBundle); (which automatically returns
ns_error_null_pointer if mBundle would be null)
after that, r=hwaara too.
Assignee | ||
Comment 11•23 years ago
|
||
I only use NS_ENSURE_ARG_POINTER(foo) if foo was an ARG.
Comment 12•23 years ago
|
||
sr=alecf
Assignee | ||
Updated•23 years ago
|
Whiteboard: PDT+; fix eta, 6-21-01, r=varada, sr=?, a=? → PDT+; fix eta, 6-21-01, r=varada, sr=alecf, a=?
Comment 13•23 years ago
|
||
a=chofmann
Assignee | ||
Comment 14•23 years ago
|
||
fixed.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Whiteboard: PDT+; fix eta, 6-21-01, r=varada, sr=alecf, a=? → PDT+; fix eta, 6-21-01, r=varada, sr=alecf, a=chofmann
Updated•23 years ago
|
Whiteboard: PDT+; fix eta, 6-21-01, r=varada, sr=alecf, a=chofmann → PDT+; fix eta, 6-21-01, r=varada, sr=alecf, a=chofmann critical for 0.9.2
Assignee | ||
Comment 15•23 years ago
|
||
it's bad if we occasionally fail to get the string bundle.
that kind of sounds familiar, I remember ben tweaking the profile manager
javascript to handle the case where we'd fail to get the string bundle.
ben, do you remember that bug? do you remember what the problem was?
I couldn't find any other bugs about string bundles failing, perhaps this rings
a bell for tao or valeski.
Comment 16•23 years ago
|
||
Using build 2001-06-28 on win, mac and linux, I don't crash starting up mail. I
read through the talkback comments and didn't see anything special scenario to
cause this crash. I also tested the scenario in the comment on 2001-06-13.
Verfied, if this comes back or is still showing up in talkback reports reopen
and give steps to reproduce.
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 17•23 years ago
|
||
I've got a reproducable case where we assert (before, we would have crashed.)
see bug #88876
Comment 18•23 years ago
|
||
The number one cause of failing stringbundle is an unsuccessful chrome URL
conversion - wrong provider type/name or wrong packaging destination.
Seth might be referring to the bundle failure introduced by the obsolete
stringbundle fallback mechanism.
It'd be interesting to see if there are other misusage of the static nsCOMPtrs.
Updated•20 years ago
|
Product: Browser → Seamonkey
Updated•13 years ago
|
Crash Signature: [@ nsMsgStatusFeedback::OnStateChange]
You need to log in
before you can comment on or make changes to this bug.
Description
•