assertions when opening mail account

RESOLVED WORKSFORME

Status

RESOLVED WORKSFORME
18 years ago
5 years ago

People

(Reporter: blizzard, Unassigned)

Tracking

({assertion})

Trunk
x86
Linux
assertion

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

18 years ago
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
(Reporter)

Comment 1

18 years ago
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.
Whiteboard: critical for mozilla 0.8

Comment 2

18 years ago
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.
Assignee: chuang → putterman

Comment 3

18 years ago
reassigning to gayatrib and cc'ing bienvenu. Is this assertion the result of
anything that needs to be fixed right away?
Assignee: putterman → gayatrib

Comment 4

18 years ago
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

18 years ago
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?
(Reporter)

Comment 6

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

Comment 7

18 years ago
we certainly could. I'll attach a patch.

Comment 8

18 years ago
Created attachment 24716 [details] [diff] [review]
patch to add printfs

Comment 9

18 years ago
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.
(Reporter)

Comment 10

18 years ago
Doesn't seem to hurt anything that I can see.  OK for 0.8.
Whiteboard: critical for mozilla 0.8

Comment 11

18 years ago
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

18 years ago
Created attachment 25030 [details]
mail assertions from winNT custom 20000211 ~9am on 0.8 branch

Comment 13

18 years ago
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

18 years ago
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

18 years ago
reassigning to naving
Assignee: gayatrib → naving
QA Contact: esther → stephend

Comment 16

17 years ago
This is over a year old. Are the assertions still there?
mass re-assign.
Assignee: naving → sspitzer
Product: Browser → Seamonkey

Updated

14 years ago
Assignee: sspitzer → mail
This seems more like a general account manager or main mail window & not an
address book problem. Changing the component.
Component: Address Book → MailNews: Main Mail Window
QA Contact: stephend
Assignee: mail → nobody
Keywords: assertion
QA Contact: message-display

Comment 19

5 years ago
No errors in SeaMonkey/2.23a2 any more.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.