52147 - Crash sending msg when there's a mailing list i Collected Addresses

Status: VERIFIED FIXED

(MailNews Core :: Composition, defect, P2)

Description

[Suresh]

steps:
compose a msg and Click send. Crashes.

Build and platform:
Today's linux commercial build.

note:
1. I used *.tar.gz build.
2. Tested on a pop account. 
3. Haven't tested on Win32 and Mac.

Comment 1

[Suresh]

Adding keywords crash, dogfood.

Console reads the following on Linux: 

SendMessage from XUL
GenericSendMessage from XUL
Identity = [nsIMsgIdentity: id1]
attachments = 
DetermineHTMLAction: noHtmlRecipients are [e-mail]

Stack trace: (Talkback ID: 17231635)
   
   nsAddrDatabase::NotifyCardEntryChange() 
                                                 
     
   nsAddrDatabase::CreateNewCardAndAddToDB() 
                                                 
     
   nsAbCardProperty::AddCardToDatabase() 
                                                 
     
   nsAbAddressCollecter::CollectAddress() 
                                                 
     
   nsMsgComposeAndSend::DeliverFileAsMail() 
                                                 
     
   nsMsgComposeAndSend::DeliverMessage() 
                                                 
     
   nsMsgComposeAndSend::GatherMimeAttachments() 
                                                 
     
   nsMsgAttachmentHandler::UrlExit() 
                                                 
     
   FetcherURLDoneCallback() 
                                                 
     
   nsURLFetcher::OnStopRequest() 
                                                 
     
   nsDocumentOpenInfo::OnStopRequest() 
                                                 
     
   nsFileChannel::OnStopRequest() 
                                                 
     
   nsOnStopRequestEvent::HandleEvent() 
                                                 
     
   nsStreamListenerEvent::HandlePLEvent() 
                                                 
     
   PL_HandleEvent() 
                                                 
     
   PL_ProcessPendingEvents() 
                                                 
     
   nsEventQueueImpl::ProcessPendingEvents() 
                                                 
     
   event_processor_callback() 
                                                 
     
   our_gdk_io_invoke() 
                                                 
     
   libglib-1.2.so.0 + 0xe52a (0x4053752a) 
                                                 
     
   libglib-1.2.so.0 + 0xfbe6 (0x40538be6) 
                                                 
     
   libglib-1.2.so.0 + 0x101a1 (0x405391a1) 
                                                 
     
   libglib-1.2.so.0 + 0x10341 (0x40539341) 
                                                 
     
   libgtk-1.2.so.0 + 0x8c209 (0x40463209) 
                                                 
     
   nsAppShell::Run() 
                                                 
     
   nsAppShellService::Run() 
                                                 
     
   main1() 
                                                 
     
   main() 
                                                 
     
   libc.so.6 + 0x181eb (0x402531eb) 
                                                 

Comment 2

[Suresh]

hmm... some more updates:

1. I can reproduce this bug in only one profile (which has two pop accounts)
2. Not able to reproduce this in an IMAP account and also on Win32.
3. Will try to narrow down this problem.

Comment 3

[R.K.Aa.]

fresh profile with 2 POP accounts on linux 2000091108 does not crash when
sending or replying to mail. There is some oddity in the menu asking for html or
text only, however - a little green box saying "1 2 3" (vertically)
Never seen that before.

Comment 4

[Jim Roskind]

Marking dogfood-plus

Comment 5

[Jean-Francois Ducarroz]

Suresh, can you try to see if you can recreate this problem using a new profile. 
Also, can you reproduce it on Mac or Window? I use to see this crash a long time 
go but I haven't see it recently!

Candice, do you have any idea?

Comment 6

[chuang]

I can't think of anything right away.  Suresh, please email me the history.mab 
from the profile which crashed.

Comment 7

[Suresh]

Attachment: history.mab file for the profile that crashed.

Comment 8

[Suresh]

Ducarroz, I couldn't reproduce this bug if I create a new profile with those
same two pop accounts. I am NOT seeing this on Win32 and Mac. 

Comment 9

[chuang]

Suresh, please try rename or move the history.mab from the profile directory and 
run again.  Let me know if the crash happens or not.  Thanks.

Comment 10

[Suresh]

That does the trick!! I renamed history.mab file. No crash. :)

Comment 11

[chuang]

Here's what I found out.  There is a mailing list in Suresh's collected address 
book.  I can reproduce it by adding a mailing list in the collected address 
book,  then I get the crash when I restart the application on my NT machine.  I 
don't have a fix yet, I'll dig into it.

Comment 12

[Suresh]

Candice, fyi, I've a mailing list in Personal AB (Not in CAB) for that profile.

Comment 13

[chuang]

The crash happened when trying to add the new recipient into the collected 
address book.  Mailing list in PAB doesn't really related to the crash here.

Comment 14

[lchiang]

Updating summary to be more precise.

Comment 15

[esther]

reassgining qa to suresh since he's been commenting and is the reporter.

Comment 16

[chuang]

Remove degfood+.  The crash only occurred when there's a mailing list in the 
Collected Addresses.  User has to manually add the mailing list into the 
Collected Addresses which is not happened frequently.   

Comment 17

[Jim Roskind]

Putting on the dogfood-minus radar in view of above comment.
Adding nsbeta3 nomination, as this is probably a common enough crash to warrant
a need to fix for beta3.

Comment 18

[selmer (gone)]

nsbeta3-, this shouldn't be hit frequently.  Adding relnote3 to get a release 
note though.

Comment 19

[laurel]

removing relnote3 keyword after talking to Suresh more about this bug.

Comment 20

[chuang]

Add mail2 keyword

Comment 21

[scottputterman]

Instead of fixing the crash, why don't we disable the ability to create a
mailing list in the Collected Address book and prevent the ability to move a
mailing list into the collected address book.  In fact, I wouldn't mind
disabling the ability to create a new card in it either.

Comment 22

[chuang]

This looks like a dup for bug 54530 which was fixed for rtm.  I'll mark it fixed 
unless it's still crash.  I tried it on NT and it works.

Comment 23

[timeless]

It makes sense to me that the collected addressbook shouldn't have lists or 
manual additions.

Should we go as far as forcing changes to collected addresses to end up in a 
normal addessbook?

I guess we should file a new bug for this.

Comment 24

[fenella]

Mark it verified/dup.