Closed Bug 40483 Opened 25 years ago Closed 25 years ago

Can't open Account Wizard dialog when creating a new profile

Categories

(SeaMonkey :: MailNews: Account Configuration, defect, P3)

x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: scottputterman, Assigned: scottputterman)

Details

(Whiteboard: [dogfood+][nsbeta2-])

Attachments

(1 file)

If I create a new profile on my Windows debug build, I can't create a new account through the Account Wizard. I have a patch that I will attach shortly. This is occurring in nsMsgServiceProvider's GetConvertedChromeURL This was happening because urlSpec was being assigned to fileURL.GetURLString which is just a ptr to fileURL's member variable. When we go out of that scope, fileURL gets destroyed and so does the member variable making urlSpec garbage which causes problems on the last line of the function. The fix we've chosen here is to move fileURL out of the if statement.
Attached patch fix for this bug. (deleted) — Splinter Review
marking nsbeta2 and dogfood.
Keywords: dogfood, nsbeta2
Severity: normal → blocker
dogfood+
Whiteboard: [dogfood+][nsbeta2-]
Is this because you can't click on the Finish button (ie. nothing happens)?
I can't even get a dialog on my debug build without this fix.
What are the type of URL you are getting if it is not of "file"? I intentionally insert the "if statement" to make sure the URL type is what we expected. But, feel free to make any change to patch abnormalty for now as long as Alec reviewed it. Also note that the code in question will be obsolete once we adopted the new localizability solution.
a resource url is getting passed in.
Sorry that I didn't read the bug description carefully... Yes, please fix it.
fix checked in.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
thanks for fixing this - I think warren and I were both sitting on similar fixes.
Scott - ok for you now? We never really saw this on our release builds. If ok, pls mark verified. Thanks.
ok. I'll verify that it works for me now!
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: