8770 - Crash when Prefs50.js references "pop" instead of "pop3"

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect, P1)

Description

[Ninoschka Baca]

Build 1999062208M8 on Win32/NT and Linux/Redhat 5.2
Incident ID: 10308097

Messenger crashes when the prefs50.js file references "pop" instead of "pop3" on
the `mail.server.server1.type` line. I know that this shouldn't matter once
prefs are working but wanted to document the problem.

Steps to Reproduce:
1. Open a prefs50.js file which has a POP3 account defined.
2. Change the `user_pref("mail.server.server1.type", "pop3");` statement to
reference "pop" instead of "pop3". Save the changes.
3. Start apprunner
4. Select Tasks/Messenger. The Messenger window begins to display but then
crashes.

Expected Results: It should behave similar to the Mac where it does not crash
and shows a generic Accounts icon.

Stack Trace:

nsMsgAccountManager::isUnique
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgAccountManager.cpp, line
405]
nsMsgAccountManager::AddAccount
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgAccountManager.cpp, line
263]
nsMsgAccountManager::LoadAccounts
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgAccountManager.cpp, line
674]
nsMsgMailSession::nsMsgMailSession
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgMailSession.cpp, line 51]
nsMsgFactory::CreateInstance
[d:\builds\seamonkey\mozilla\mailnews\base\build\nsMsgFactory.cpp, line 211]
nsComponentManagerImpl::CreateInstance
[d:\builds\seamonkey\mozilla\xpcom\components\nsComponentManager.cpp, line 1127]
nsComponentManager::CreateInstance
[d:\builds\seamonkey\mozilla\xpcom\components\nsRepository.cpp, line 68]
nsServiceManagerImpl::GetService
[d:\builds\seamonkey\mozilla\xpcom\components\nsServiceManager.cpp, line 244]
nsServiceManager::GetService
[d:\builds\seamonkey\mozilla\xpcom\components\nsServiceManager.cpp, line 446]
nsMessenger::InitializeFolderRoot
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMessenger.cpp, line 392]
nsMessenger::nsMessenger
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMessenger.cpp, line 241]
NS_NewMessenger
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMessenger.cpp, line 322]
nsMsgFactory::CreateInstance
[d:\builds\seamonkey\mozilla\mailnews\base\build\nsMsgFactory.cpp, line 224]
sComponentManagerImpl::CreateInstance
[d:\builds\seamonkey\mozilla\xpcom\components\nsComponentManager.cpp, line 1127]
nsComponentManager::CreateInstance
[d:\builds\seamonkey\mozilla\xpcom\components\nsRepository.cpp, line 68]
CIDCreateInstanceScriptable::Call
[d:\builds\seamonkey\mozilla\js\src\xpconnect\src\xpcjsid.cpp, line 362]
WrappedNative_Call
[d:\builds\seamonkey\mozilla\js\src\xpconnect\src\xpcwrappednativeclass.cpp,
line 1227]
js_Invoke
[d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 656]
js_Interpret
[d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 2207]
js_Execute
[d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 822]
JS_EvaluateUCScriptForPrincipals
[d:\builds\seamonkey\mozilla\js\src\jsapi.c, line 2509]
nsJSContext::EvaluateString
[d:\builds\seamonkey\mozilla\dom\src\base\nsJSEnvironment.cpp, line 140]
XULContentSinkImpl::EvaluateScript
[d:\builds\seamonkey\mozilla\rdf\datasource\src\nsXULContentSink.cpp, line 1553]
XULContentSinkImpl::DoneLoadingScript
[d:\builds\seamonkey\mozilla\rdf\datasource\src\nsXULContentSink.cpp, line 1511]
nsUnicharStreamLoader::OnStopBinding
[d:\builds\seamonkey\mozilla\network\module\nsNetStreamLoader.cpp, line 158]
nsDocumentBindInfo::OnStopBinding
[d:\builds\seamonkey\mozilla\webshell\src\nsDocLoader.cpp, line 1601]
OnStopBindingProxyEvent::HandleEvent
[d:\builds\seamonkey\mozilla\network\module\nsNetThread.cpp, line 594]
StreamListenerProxyEvent::HandlePLEvent
[d:\builds\seamonkey\mozilla\network\module\nsNetThread.cpp, line 474]
PL_HandleEvent [plevent.c, line 492]
PL_ProcessPendingEvents [plevent.c, line 453]
_md_EventReceiverProc [plevent.c, line 881]
USER32.dll + 0x1250 (0x77e71250)
nsappshell.dll + 0x183f (0x0160183f)
apprunner.exe + 0x22c1 (0x004022c1)
KERNEL32.dll + 0x1b304 (0x77f1b304)

Comment 1

[Phil Peterson]

Reassign to alecf

Comment 2

[Alec Flett]

oh man, that's bad. I think I know what's going on.

Comment 3

[lchiang]

<updated qa contact name>

Comment 4

[Alec Flett]

fixed.

Comment 5

[Alec Flett]

*** Bug 8758 has been marked as a duplicate of this bug. ***

Comment 6

[Ninoschka Baca]

1999062508M8 Win32/NT4, Linux Redhat 5.2, Mac 8.5.1
Verified Fixed.

Changed the prefs50.js file to state "pop", launched messenger and the folder
shows a generic server name called "Accounts". It no longer crashes. If I change
the type back to "pop3" then it works as expected showing the server name.