36831 - assertions when using imap in nsSocketTransport.cpp (no mPrintHost)

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect, P3)

Description

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

logging in and using imap on the tip, you'll get a billion of these asserts.

NTDLL! 77f7629c()
nsDebug::Assertion(const char * 0x01df14d4, const char * 0x01df14d0, const char 
* 0x01df1494, int 2332) line 191 + 13 bytes
nsSocketTransport::fireStatus(unsigned int 5) line 2332 + 35 bytes
nsSocketTransport::Process(short 3) line 620
nsSocketTransportService::Run(nsSocketTransportService * const 0x0267e0e4) line 
417 + 13 bytes
nsThread::Main(void * 0x0267f7b0) line 84 + 26 bytes
_PR_NativeRunThread(void * 0x0267f640) line 399 + 13 bytes
_threadstartex(void * 0x0267f490) line 212 + 13 bytes
KERNEL32! 77f04ee8()

Comment 1

[Scott MacGregor]

Gagan, this is making it very very difficult to use mail in the debug builds.
Can you please look at this ASAP.

If we can't get this fixed today I'm going to debug_gagan the assertion failure
so you can look into it on your own if that's okay with you.

Remember to run imap as a smoketest if you are making nsSocketTransport changes =).

Comment 2

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

wrapped the assertion with #ifdef DEBUG_gagan until gagan fixes the real 
problem.

Comment 3

[Scott MacGregor]

Thanks for checking this change in Seth. Gagan, I don't know what the
ramifications of the assert were but mailnews isn't the only one that triggers
it on. If you look at the smoketest logs on tinderbox you'll see that it was
hitting your assertion a lot too for loading web pages and ftp I think....

Comment 4

[lchiang]

Marking dogfood for debug builds.

Comment 5

[leger]

Putting on [dogfood+] radar.

Comment 6

[Gagan]

ooops... this was an accidental checkin. I will remove the assertion (its safe
to do so) as soon as the tree opens. Sorry for all the assertion noise!

Comment 7

[Gagan]

checked in...

Comment 8

[lchiang]

marking verified.