Closed Bug 97240 Opened 24 years ago Closed 24 years ago

Mail asserts several times on Quit

Categories

(MailNews Core :: Backend, defect)

PowerPC
Mac System 8.5
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: sfraser_bugs, Assigned: Bienvenu)

Details

Attachments

(2 files)

When I quit the browser, having run mail in a session, I get a number of assertions: ###!!! ASSERTION: NS_ENSURE_TRUE(NS_SUCCEEDED(rv)) failed: '(!((rv) & 0x80000000))', file nsMsgIncomingServer.cpp, line 1144 ###!!! ASSERTION: NS_ENSURE_TRUE(NS_SUCCEEDED(rv)) failed: '(!((rv) & 0x80000000))', file nsMsgIncomingServer.cpp, line 1060 ###!!! ASSERTION: NS_ENSURE_TRUE(NS_SUCCEEDED(rv)) failed: '(!((rv) & 0x80000000))', file nsMsgIncomingServer.cpp, line 1144 ###!!! ASSERTION: NS_ENSURE_TRUE(NS_SUCCEEDED(rv)) failed: '(!((rv) & 0x80000000))', file nsMsgIncomingServer.cpp, line 1060 ###!!! ASSERTION: NS_ENSURE_TRUE(NS_SUCCEEDED(rv)) failed: '(!((rv) & 0x80000000))', file nsMsgIncomingServer.cpp, line 1144 ###!!! ASSERTION: NS_ENSURE_TRUE(NS_SUCCEEDED(rv)) failed: '(!((rv) & 0x80000000))', file nsMsgIncomingServer.cpp, line 1060 ###!!! ASSERTION: NS_ENSURE_TRUE(NS_SUCCEEDED(rv)) failed: '(!((rv) & 0x80000000))', file nsMsgIncomingServer.cpp, line 1144 ###!!! ASSERTION: NS_ENSURE_TRUE(NS_SUCCEEDED(rv)) failed: '(!((rv) & 0x80000000))', file nsMsgIncomingServer.cpp, line 1060 These appear to happen because in querying whehter a server supports Biff, we try to do a do_GetService, and this fails because the service manager is in the process of shutting down.
Biff --> Bhuvan.
Assignee: mscott → racham
I'll take this since it's been annoying me for a while.
Assignee: racham → bienvenu
Attached patch proposed fixSplinter Review
Fix is simply to remove check for doBiff - removebiff handles the case where the server is not biff'ed, so that check was not needed.
Status: NEW → ASSIGNED
Can I get a review, Bhuvan? Thanks.
r=bhuvan
fix checked into trunk, no need for branch.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
QA Contact: esther → stephend
I'm not seeing this assert anymore on a debug win32 build.
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.

Attachment

General

Created:
Updated:
Size: