Closed Bug 38104 Opened 25 years ago Closed 24 years ago

crash in nsMsgStatusFeedback::NotifyStopMeteors

Categories

(MailNews Core :: Backend, defect, P3)

x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

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?
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
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.
mass fixing up crash reports (adding crash kw, raising severity). please fix any errors
Severity: normal → critical
Keywords: crash
Mail Review recommends beta2 stopper. Adding nsbeta2 keyword.
Keywords: nsbeta2
Putting on [nsbeta2+] radar for beta2 fix.
Whiteboard: [nsbeta2+]
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
build 2000-07-19-11-m17 win98, mac8.6 and linux it doesnt crash.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.