68019 - assertions when opening mail account

Status: RESOLVED WORKSFORME

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Christopher Blizzard (:blizzard)]

Build is 2001 02 07.  When I open a mail account I'm getting truckloads of
assertions that look like this:

###!!! ASSERTION: couldn't get names: 'names', file
../../../../../mozilla/mailnews/base/search/src/nsMsgSearchTerm.cpp, line 929
###!!! Break: at file
../../../../../mozilla/mailnews/base/search/src/nsMsgSearchTerm.cpp, line 929
###!!! ASSERTION: couldn't get addresses: 'addresses', file
../../../../../mozilla/mailnews/base/search/src/nsMsgSearchTerm.cpp, line 930
###!!! Break: at file
../../../../../mozilla/mailnews/base/search/src/nsMsgSearchTerm.cpp, line 930

Comment 1

[Christopher Blizzard (:blizzard)]

I'm marking this critical for 0.8 since it might indicate another problem.  If
it turns out not to be then feel free to remove it.

Comment 2

[chuang]

Scott,  can you reassign to some one in this area?  I don't get the assertion on 
today's winNT debug build.  This is coming from search and the "names" is 
returned by ParseHeaderAddresses which is in mime.

Comment 3

[scottputterman]

reassigning to gayatrib and cc'ing bienvenu. Is this assertion the result of
anything that needs to be fixed right away?

Comment 4

[David :Bienvenu]

If this is related to Chris's problem with mail filters not firing (which I bet
it is), then I'd guess it is critical. I'm not seeing these assertions myself.

Comment 5

[David :Bienvenu]

This looks like a bug in the rfc822 message header parser.
nsMsgHeaderParser::ParseHeaderAddresses - I wonder if some i18n code changed
(though I don't know why I'm not seeing this). Chris, can you run in the
debugger and see what the string and charset passed in to ParseHeaderAddresses is?

Comment 6

[Christopher Blizzard (:blizzard)]

Can we do this with debugging printf()'s?  I don't have a working build and
debugger right now.

Comment 7

[David :Bienvenu]

we certainly could. I'll attach a patch.

Comment 8

[David :Bienvenu]

Attachment: patch to add printfs

Comment 9

[David :Bienvenu]

Chris, can you apply this patch and let me know if anything is printed out? You
don't need me to check this in, do you?

thanks.

Comment 10

[Christopher Blizzard (:blizzard)]

Doesn't seem to hurt anything that I can see.  OK for 0.8.

Comment 11

[David :Bienvenu]

are you seeing any printf's? Are you still seeing the assertions? I don't see
any reason to check this in for .8 since it's just for diagnosis, and I haven't
heard anyone else having this problem.

Comment 12

[Mack Earnhardt]

Attachment: mail assertions from winNT custom 20000211 ~9am on 0.8 branch

Comment 13

[Mack Earnhardt]

I should point out that the assertions I attached are when I open mail/news from
the browser. Perhaps this is a different problem?

Comment 14

[David :Bienvenu]

Mack, those are definitely different assertions, and have to do with your
account setup. It's messed up somehow, though beyond that I couldn't say.

Comment 15

[scottputterman]

reassigning to naving

Comment 16

[R.K.Aa.]

This is over a year old. Are the assertions still there?

Comment 17

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

mass re-assign.

Comment 18

[Mark Banner (:standard8)]

This seems more like a general account manager or main mail window & not an
address book problem. Changing the component.

Comment 19

[:Hb]

No errors in SeaMonkey/2.23a2 any more.