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)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: scottputterman, Assigned: scottputterman)
Details
(Whiteboard: [dogfood+][nsbeta2-])
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•25 years ago
|
||
Assignee | ||
Comment 2•25 years ago
|
||
marking nsbeta2 and dogfood.
Assignee | ||
Updated•25 years ago
|
Severity: normal → blocker
Is this because you can't click on the Finish button (ie. nothing happens)?
Assignee | ||
Comment 5•25 years ago
|
||
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.
Assignee | ||
Comment 7•25 years ago
|
||
a resource url is getting passed in.
Sorry that I didn't read the bug description carefully... Yes, please fix it.
Assignee | ||
Comment 9•25 years ago
|
||
fix checked in.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Comment 10•25 years ago
|
||
thanks for fixing this - I think warren and I were both sitting on similar
fixes.
Comment 11•25 years ago
|
||
Scott - ok for you now? We never really saw this on our release builds. If ok,
pls mark verified. Thanks.
Assignee | ||
Comment 12•25 years ago
|
||
ok. I'll verify that it works for me now!
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•