31913 - assertions in morkConfig.cpp

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect, P3)

Description

[(not reading, please use [e-mail] instead)]

jefft knows all about this already.  see bug #31904

NTDLL! 77f7629c()
nsDebug::Assertion(const char * 0x0295cbb0, const char * 0x0295cabc, const char 
* 0x0295ca8c, int 78) line 189 + 13 bytes
mork_assertion_signal(const char * 0x0295cbb0) line 78 + 31 bytes
morkEnv::NewError(const char * 0x0392b280) line 369 + 19 bytes
morkFile::NewFileErrnoError(morkEnv * 0x0393daa0) line 272
morkStdioFile::new_stdio_file_fault(morkEnv * 0x0393daa0) line 668
morkStdioFile::OpenStdio(morkEnv * 0x0393daa0, const char * 0x0393d510, const 
char * 0x0295cdf8) line 739
morkStdioFile::morkStdioFile(morkEnv * 0x0393daa0, const morkUsage & {...}, 
nsIMdbHeap * 0x022c5dc8, nsIMdbHeap * 0x022c5dc8, const char * 0x0393d510, const 
char * 0x0295cdf8) line 693
morkStdioFile::OpenOldStdioFile(morkEnv * 0x0393daa0, nsIMdbHeap * 0x022c5dc8, 
const char * 0x0393d510, unsigned char 0) line 371 + 62 bytes
morkFile::OpenOldFile(morkEnv * 0x0393daa0, nsIMdbHeap * 0x022c5dc8, const char 
* 0x0393d510, unsigned char 0) line 178 + 21 bytes
orkinFactory::OpenOldFile(nsIMdbEnv * 0x0393d5c8, nsIMdbHeap * 0x022c5dc8, const 
char * 0x0393d510, unsigned char 0, nsIMdbFile * * 0x0012f33c) line 312 + 21 
bytes
nsMsgDatabase::OpenMDB(nsMsgDatabase * const 0x0393db10, const char * 
0x0393d968, int 0) line 752 + 34 bytes
nsImapMailDatabase::Open(nsImapMailDatabase * const 0x0393fa40, nsIFileSpec * 
0x0393dc00, int 0, int 0, nsIMsgDatabase * * 0x0012f528) line 93 + 29 bytes
nsImapMailFolder::GetDBFolderInfoAndDB(nsImapMailFolder * const 0x0393fc1c, 
nsIDBFolderInfo * * 0x0012f66c, nsIMsgDatabase * * 0x0012f5b4) line 1274 + 49 
bytes
nsImapMailFolder::AddSubfolder(nsImapMailFolder * const 0x03351d5c, nsAutoString 
* 0x0012f898, nsIMsgFolder * * 0x0012f934) line 258 + 95 bytes
nsImapMailFolder::CreateClientSubfolderInfo(nsImapMailFolder * const 0x03351dec, 
const char * 0x0012fbc0, unsigned short 47) line 658 + 62 bytes
nsImapIncomingServer::PossibleImapMailbox(nsImapIncomingServer * const 
0x0334d940, const char * 0x03938290, unsigned short 47, int 64) line 697
XPTC_InvokeByIndex(nsISupports * 0x0334d940, unsigned int 3, unsigned int 3, 
nsXPTCVariant * 0x0393c5e0) line 139
EventHandler(PLEvent * 0x0393c640) line 481 + 41 bytes
PL_HandleEvent(PLEvent * 0x0393c640) line 563 + 10 bytes
PL_ProcessPendingEvents(PLEventQueue * 0x00ff1b90) line 508 + 9 bytes
_md_EventReceiverProc(HWND__ * 0x030c0336, unsigned int 49374, unsigned int 0, 
long 16718736) line 1018 + 9 bytes
USER32! 77e71820()
00ff1b90()

Comment 1

[(not reading, please use [e-mail] instead)]

fix summary

Comment 2

[jefft]

I don't see this stack trace anymore. May be the wrong stack trace. Should be 
the other stated in bug 31904, I think. Accepting....

Comment 3

[David :Bienvenu]

I will lose my mind if this isn't fixed before m16 - I'll take it.

Comment 4

[David :Bienvenu]

fix checked in - the problem was that Jeff was asking for the flags before 
SetPath had been called on the folder. to fix this, I had to change 
AddSubfolders to AddSubfoldersWithPath and pass in the db path so I can SetPath 
before looking for the folder in the cache. Seth might have to verify this, 
since he was getting asserts, and you only get asserts in a debug build. I'm not 
getting asserts anymore, thank goodness.

Comment 5

[(not reading, please use [e-mail] instead)]

thanks for fixing this. 

Comment 6

[[e-mail] (gone - use [e-mail] instead)]

I migrated an IMAP profile (from bug 31904) and only got another, unrelated
nsPrefMigration.cppp assert (failed to copy file, etc).  Marking verified FIXED.