Closed
Bug 38104
Opened 25 years ago
Closed 24 years ago
crash in nsMsgStatusFeedback::NotifyStopMeteors
Categories
(MailNews Core :: Backend, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
M17
People
(Reporter: warrensomebody, Assigned: mscott)
Details
(Keywords: crash, Whiteboard: [nsbeta2+])
Closing the mail window while message were still downloading:
nsMsgStatusFeedback::NotifyStopMeteors(nsITimer * 0x046d3190) line 355 + 17
bytes
nsMsgStatusFeedback::notifyStopMeteors(nsITimer * 0x046d3190, void * 0x0399f2c0)
line 320
nsTimer::Fire() line 194 + 17 bytes
FireTimeout(HWND__ * 0x00000000, unsigned int 0x00000113, unsigned int
0x0000789e, unsigned long 0x50984880) line 78
USER32! 77e7185c()
nsAppShellService::Run(nsAppShellService * const 0x01471f30) line 366
main1(int 0x00000001, char * * 0x00c44450, nsISplashScreen * 0x00000000) line
765 + 32 bytes
main(int 0x00000001, char * * 0x00c44450) line 998 + 17 bytes
mainCRTStartup() line 338 + 17 bytes
KERNEL32! 77f1ba06()
void
nsMsgStatusFeedback::NotifyStopMeteors(nsITimer* aTimer)
{
mQueuedMeteorStops--;
// meteors not spinning
if (!m_meteorsSpinning) return;
// there is at least one more timer firing soon
if (mQueuedMeteorStarts > 0) return;
// actually stop the meteors
if (mStatusFeedback)
=> mStatusFeedback->StopMeteors();
m_meteorsSpinning = PR_FALSE;
}
mStatusFeedback is 0xdddddddd (deleted). Refcount problem?
Is this similar to http://bugzilla.mozilla.org/show_bug.cgi?id=31411?
Assignee | ||
Comment 2•25 years ago
|
||
I think the stack trace is significantly different from that other bug. We used
to have this crash but then I fixed it. It looks like it is back again =(.
Status: NEW → ASSIGNED
Target Milestone: --- → M17
Reporter | ||
Comment 3•25 years ago
|
||
Not sure. That one looks like it happens in an pl event callback. This one's in
the timer code. But it could be do to the same underlying problem.
Comment 4•25 years ago
|
||
mass fixing up crash reports (adding crash kw, raising severity). please fix
any errors
Severity: normal → critical
Keywords: crash
Comment 5•25 years ago
|
||
Mail Review recommends beta2 stopper. Adding nsbeta2 keyword.
Keywords: nsbeta2
Assignee | ||
Comment 7•24 years ago
|
||
I added some code just before the tree shut down that should protect against
this case.
I just tried it several times on my machine and could not trigger the crash. We
cleanly handle the fact that the window has gone away while trying to download a
message.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Luis - pls verify. Send yourself a huge mail message. Get mail and select it.
Before the message finishes loading, close the 3pane mail window. Make sure
there are no crashes. Please try on all platforms. Thanks.
Anothe case to try: Open that message in a standalone window (I can show you if
you want). Close the standalone message window before the message finishes
loading.
QA Contact: lchiang → luis
Comment 9•24 years ago
|
||
build 2000-07-19-11-m17
win98, mac8.6 and linux
it doesnt crash.
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•