19396 - talkback crash in nsLocalMailFolder::GetIncomingServerType

Status: VERIFIED DUPLICATE of bug 18075

(MailNews Core :: Backend, defect, P3)

Description

[Scott MacGregor]

Warren pointed this out. One of the top M11 crashers is actually a mailnews
crash. Here's the stack trace:

nsUnescapeCount [d:\builds\seamonkey\mozilla\xpcom\io\nsEscape.cpp, line 149]
nsUnescape [d:\builds\seamonkey\mozilla\xpcom\io\nsEscape.cpp, line 139]
nsMsgLocalMailFolder::GetIncomingServerType
[d:\builds\seamonkey\mozilla\mailnews\local\src\nsLocalMailFolder.cpp, line
1849]
nsMsgFolder::Init
[d:\builds\seamonkey\mozilla\mailnews\base\util\nsMsgFolder.cpp, line 181]


I'm looking into fixing it right now.

Comment 1

[Scott MacGregor]

I added a url which shows you the entire full circle stack trace.

Comment 2

[lchiang]

user comments in the talkback reports shows the crash occuring when setting up
accounts.

Comment 3

[lchiang]

cc'ing [e-mail] since he reported this crash in Talkback.  Now,
you know when this bug will get fixed and you can use mail again.  Thanks.

Comment 4

[Alec Flett]

mscott - I think this is fixed in M12

Comment 5

[Alec Flett]

the problem was that I had cut&pasted the check to see if userName was null or
not, I was accidentally checking if hostName was null before unescaping
userName.

Comment 6

[Scott MacGregor]

I'm going to agree with Alec that this problem has been fixed. I tried
simulating different return values for the PreHost call (such as an empty
string, null, etc) and wasn't able to produce the crash.

Marking as fixed.

Comment 7

[lchiang]

alec, was this a dup of some other bug?

Comment 8

[Alec Flett]

yes, reported by tony robinson

Comment 9

[lchiang]

perhaps http://bugzilla.mozilla.org/show_bug.cgi?id=16075.

Comment 10

[Alec Flett]

oh, I just realized that there was more to tony's bug, so I hadn't closed it
yet. It's 18075

Comment 11

[Alec Flett]

*** This bug has been marked as a duplicate of 18075 ***

Comment 12

[lchiang]

thanks. marking verified as duplicate.