Closed Bug 9105 Opened 25 years ago Closed 25 years ago

[CRASH] when prefs50.js statement is not terminated

Categories

(MailNews Core :: Backend, defect, P3)

All
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: nbaca, Assigned: Bienvenu)

Details

1999063008M8: Win32/NT4, Linux Redhat 5.2, Mac 8.5.1 Incident: 10601073. Crash occurs when the prefs50.js `user_pref("mail.identity.id1.smtp_server"..` entry is not terminated. Step to Reproduce: 1. Open the prefs50.js file. 2. Go to the user_pref mail.identity.id1.smtp_server statement and remove one of the quotes from the server name. For instance: It should be in this form: user_pref("mail.identity.id1.smtp_server", "dredd"); but remove the last quote after dredd: user_pref("mail.identity.id1.smtp_server", "dredd); 3. Save the changes 4. Launch appruner and it reports `An error occured reading the startup configuration file. Please contact your administrator. line 43: unterminated string literal. user_pref("mail.identity.id1.smtp_server","dredd),` and press OK. 5. A Netscape Warning appears stating `Error in preference file (prefs.js). Default preferences will be used` and press OK. 6. The Browser launches. 7. Select Tasks/Messenger and an "Application Error in Apprunner.exe" appears and it crashes. Workaround: Edit the mail.identity.id1.smtp_server statement so that dredd is surrounded by quotes (i.e. "dredd"). Expected Results: Using a default preference is fine but it should not crash. Call Stack: (Signature = nsLocalURI2Server 407d8b6a) nsLocalURI2Server [d:\builds\seamonkey\mozilla\mailnews\local\src\nsLocalUtils.cpp, line 128] nsLocalURI2Path [d:\builds\seamonkey\mozilla\mailnews\local\src\nsLocalUtils.cpp, line 156] nsMsgLocalMailFolder::GetPath [d:\builds\seamonkey\mozilla\mailnews\local\src\nsLocalMailFolder.cpp, line 1042] nsMsgLocalMailFolder::GetSubFolders [d:\builds\seamonkey\mozilla\mailnews\local\src\nsLocalMailFolder.cpp, line 298] nsMsgFolderDataSource::createFolderChildNode [d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgFolderDataSource.cpp, line 842] nsMsgFolderDataSource::createFolderNode [d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgFolderDataSource.cpp, line 693] nsMsgFolderDataSource::GetTarget [d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgFolderDataSource.cpp, line 215] CompositeDataSourceImpl::GetTarget [d:\builds\seamonkey\mozilla\rdf\base\src\nsCompositeDataSource.cpp, line 735] RDFGenericBuilderImpl::IsContainer [d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFGenericBuilder.cpp, line 2268] RDFMenuBuilderImpl::AddWidgetItem [d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFMenuBuilder.cpp, line 228] RDFGenericBuilderImpl::CreateWidgetItem [d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFGenericBuilder.cpp, line 1265] RDFGenericBuilderImpl::CreateContents [d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFGenericBuilder.cpp, line 598] XULDocumentImpl::CreateContents [d:\builds\seamonkey\mozilla\rdf\content\src\nsXULDocument.cpp, line 2549] RDFElementImpl::EnsureContentsGenerated [d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2575] RDFElementImpl::ChildCount [d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 1495] RDFElementImpl::GetChildNodes [d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 612] RDFElementImpl::GetElementsByAttribute [d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2775] RDFElementImpl::GetElementsByAttribute [d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2814] RDFElementImpl::GetElementsByAttribute [d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2814] RDFElementImpl::GetElementsByAttribute [d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 2814] RDFElementImpl::GetElementsByAttribute [d:\builds\seamonkey\mozilla\rdf\content\src\nsRDFElement.cpp, line 1030]
QA Contact: lchiang → nbaca
Assignee: phil → bienvenu
Target Milestone: M10
I guess I'll take this since Alec is on vacation.
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
works for me now - I didn't do anything to fix it, but someone seems to have.
Status: RESOLVED → VERIFIED
Build 1999072208M9: Win32/NT4, Linux Redhat 5.2, Mac 8.5.1 Verified Fixed. The crash no longer occurs.
I'll bet this was fixed by some of the bulletproofing I added to the account system just before I left...
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.