Closed
Bug 180995
Opened 22 years ago
Closed 22 years ago
nsStatusBarBiffManager::~nsStatusBarBiffManager() unconditionally releases a null member
Categories
(SeaMonkey :: MailNews: Message Display, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: timeless, Assigned: timeless)
References
Details
(Keywords: crash)
Attachments
(1 file)
593 bytes,
patch
|
sspitzer
:
review+
bzbarsky
:
superreview+
|
Details | Diff | Splinter Review |
+ kBiffStateAtom 0x00000000 NS_RELEASE(kBiffStateAtom); nsStatusBarBiffManager::~nsStatusBarBiffManager() line 76 + 6 bytes nsStatusBarBiffManager::`scalar deleting destructor'(unsigned int 1) + 15 bytes nsStatusBarBiffManager::Release(nsStatusBarBiffManager * const 0x03d0aca0) line 64 + 179 bytes nsCOMPtr_base::assign_assuming_AddRef(nsISupports * 0x00000000) line 436 nsCOMPtr_base::assign_with_AddRef(nsISupports * 0x00000000) line 74 nsCOMPtr<nsISupports>::operator=(nsISupports * 0x00000000) line 796 FreeServiceContractIDEntryEnumerate(PLDHashTable * 0x004a676c, PLDHashEntryHdr * 0x010514b4, unsigned int 319, void * 0x00000000) line 1926 PL_DHashTableEnumerate(PLDHashTable * 0x004a676c, int (PLDHashTable *, PLDHashEntryHdr *, unsigned int, void *)* 0x0029b1f0 FreeServiceContractIDEntryEnumerate(PLDHashTable *, PLDHashEntryHdr *, unsigned int, void *), void * 0x00000000) line 603 + 34 bytes nsComponentManagerImpl::FreeServices() line 1938 + 19 bytes NS_ShutdownXPCOM(nsIServiceManager * 0x00000000) line 723 main(int 2, char * * 0x004a4424) line 949 + 8 bytes mainCRTStartup() line 338 + 17 bytes KERNEL32! 77e87903()
Comment 1•22 years ago
|
||
*** Bug 180998 has been marked as a duplicate of this bug. ***
Attachment #106860 -
Flags: superreview?(bzbarsky)
Attachment #106860 -
Flags: review?(suresh)
Updated•22 years ago
|
Attachment #106860 -
Flags: superreview?(bzbarsky) → superreview+
Attachment #106860 -
Flags: review?(suresh) → review?(sspitzer)
Comment 3•22 years ago
|
||
Comment on attachment 106860 [details] [diff] [review] patch r=sspitzer, this looks fine. but how are you hitting this? this the factory creates this service and calls Init() from nsMsgFactory.cpp: NS_GENERIC_FACTORY_CONSTRUCTOR_INIT(nsStatusBarBiffManager, Init) is the call to NS_NewAtom() failing, and then we crash at shutdown?
Attachment #106860 -
Flags: review?(sspitzer) → review+
Comment 4•22 years ago
|
||
version 1.25 was checked in, why is this not marked Resolved Fixed?
because i'm not very organized.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Timeless, this was a crash, no?
I don't see any crashes in http://climate (internal Talkback server), but then again, that doesn't mean much, as this was probably a very difficult bug to reproduce. I'm verifying based on running biff for the past week on all platforms, and checking Talkback at: http://climate/reports/VeryFastSearchStackSigNEW.cfm?stacksig=nsStatusBarBiffManager%3A%3A%7EnsStatusBarBiffManager (also, lxr verification)
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: Browser → Seamonkey
Component: MailNews: Notification → MailNews: Message Display
QA Contact: stephend → search
You need to log in
before you can comment on or make changes to this bug.
Description
•