40462 - crashes at CountTotalMimeAttachments

Status: VERIFIED INVALID

(MailNews Core :: MIME, defect, P3)

Description

[Shiva Thirumazhusai]

Crash reported through Talkback System.

Latest Build Information :-

   Build: 2000052209 CrashDate: 2000-05-24 

User Comments :-

   None Available (I am trying to get re-producible steps).

Stack Trace :- 

CountTotalMimeAttachments 
[d:\builds\seamonkey\mozilla\mailnews\mime\src\mimemoz2.cpp, line 207] 
CountTotalMimeAttachments 
[d:\builds\seamonkey\mozilla\mailnews\mime\src\mimemoz2.cpp, line 207] 
CountTotalMimeAttachments 
[d:\builds\seamonkey\mozilla\mailnews\mime\src\mimemoz2.cpp, line 207] 
MimeGetAttachmentList 
[d:\builds\seamonkey\mozilla\mailnews\mime\src\mimemoz2.cpp, line 419] 
mime_display_stream_complete 
[d:\builds\seamonkey\mozilla\mailnews\mime\src\mimemoz2.cpp, line 773] 
nsStreamConverter::OnStopRequest 
[d:\builds\seamonkey\mozilla\mailnews\mime\src\nsStreamConverter.cpp, line 846] 
nsDocumentOpenInfo::OnStopRequest 
[d:\builds\seamonkey\mozilla\uriloader\base\nsURILoader.cpp, line 204] 
nsMsgProtocol::OnStopRequest 
[d:\builds\seamonkey\mozilla\mailnews\base\util\nsMsgProtocol.cpp, line 229] 
nsMailboxProtocol::OnStopRequest 
[d:\builds\seamonkey\mozilla\mailnews\local\src\nsMailboxProtocol.cpp, line 200] 
nsFileChannel::OnStopRequest 
[d:\builds\seamonkey\mozilla\netwerk\protocol\file\src\nsFileChannel.cpp, line 
627] 
nsOnStopRequestEvent::HandleEvent 
[d:\builds\seamonkey\mozilla\netwerk\base\src\nsAsyncStreamListener.cpp, line 
307] 
nsStreamListenerEvent::HandlePLEvent 
[d:\builds\seamonkey\mozilla\netwerk\base\src\nsAsyncStreamListener.cpp, line 
106] 
PL_HandleEvent [d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c, line 576] 
PL_ProcessPendingEvents [d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c, 
line 539] 
_md_EventReceiverProc [d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c, line 
1032] 
KERNEL32.DLL + 0x363b (0xbff7363b) 
KERNEL32.DLL + 0x242e7 (0xbff942e7) 
0x00688b4a 

Code Around the Crash :-
 195 rhp      1.70  PRInt32
 196                CountTotalMimeAttachments(MimeContainer *aObj)
 197 rhp      1.34  {
 198 rhp      1.70    PRInt32     i;
 199                  PRInt32     rc = 0;
 200 rhp      1.34  
 201 rhp      1.70    if ( (!aObj) || (!aObj->children) || (aObj->nchildren <= 
0) )
 202 rhp      1.34      return 0;
 203                
 204 rhp      1.70    for (i=0; i<aObj->nchildren; i++)
 205                    rc += CountTotalMimeAttachments((MimeContainer 
*)aObj->children[i]) + 1;
 206                
 207                  return rc;
 208                }
 209

Comment 1

[Shiva Thirumazhusai]

Adding keywords crash, topcrash and nsbeta2.

Comment 2

[rhp (gone)]

Uh...not really possible to fix this without some reproducable case.

- rhp

Comment 3

[lchiang]

Peter - when you have time, perhaps you can take a look at some of the talkback
logs to see if anyone had comments on what s/he was doing at the time of the
crash.  I hope that info will be available.

Rich - do you think this could be the problem with displaying more than 19
attachments?  I'm just guessing based on the name "CountTotalMimeAttachments"

Comment 4

[Shiva Thirumazhusai]

Last three days talkback received around 9 crashes. But, none of them have 
comments to re-produce this crash. I will monitor talkback data for this crash 
and update this bug. If needed you can mark it has worksforme/invalid and
we can re-open it when I have re-producible case.

Comment 5

[rhp (gone)]

I'm going to mark it as invalid for a moral victory 
for now. 

CountTotalMimeAttachments() gets called for every email with attachments and it 
certainly would have happened in that case of 19 or more email messages.

- rhp

Comment 6

[lchiang]

Ok, I'm going to mark this verified then.  The bug with 19 attachments was
recently fixed so we'll see if we get more talkback reports from people using a
build later than today, for example.