Closed
Bug 721242
Opened 13 years ago
Closed 13 years ago
After using Firefox migrator, the password manager sometimes can't decrypt passwords
Categories
(Firefox :: Migration, defect)
Tracking
()
VERIFIED
FIXED
Firefox 13
Tracking | Status | |
---|---|---|
firefox12 | --- | fixed |
People
(Reporter: MattN, Assigned: MattN)
References
Details
(Whiteboard: [qa!])
Attachments
(1 file)
(deleted),
patch
|
mak
:
review+
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
My theory is that PSM? is initialized before the key3.db file is copied to the destination profile. Password manager outputs "Initializing key3.db with default blank password."
Mano wants to remove the ability to call DoStartup in bug 718586 so simply delaying the startup like was traditionally done in these situations would block that.
Assignee | ||
Comment 1•13 years ago
|
||
I personally think we should take this patch for now to avoid having broken password migration on Mozilla 12. We can address the removal of the startup argument in bug 718586 after branching so that there's more time to find a workaround to this problem, if necessary.
Attachment #591641 -
Flags: review?(mano)
Assignee | ||
Comment 2•13 years ago
|
||
Comment on attachment 591641 [details] [diff] [review]
v.1 delay the doStartup call
Review ping
Attachment #591641 -
Flags: feedback?(mak77)
Comment 3•13 years ago
|
||
Comment on attachment 591641 [details] [diff] [review]
v.1 delay the doStartup call
Review of attachment 591641 [details] [diff] [review]:
-----------------------------------------------------------------
It's ok, for the sake of having this working in the 12-13 releases, though please add a comment explaning the problem in bug 718586 (if you didn't yet), so that it is somehow tracked for when that bug will be addressed.
and sigh, we really need to move on with making some tests :( As it is this may break at any time.
::: browser/components/migration/src/FirefoxProfileMigrator.js
@@ +233,5 @@
>
> if (aItems & MIGRATOR.PASSWORDS)
> this._migratePasswords();
>
> + // The password manager encryption key must be copied before startup
nit: end with period
Attachment #591641 -
Flags: review?(mano)
Attachment #591641 -
Flags: review+
Attachment #591641 -
Flags: feedback?(mak77)
Assignee | ||
Comment 4•13 years ago
|
||
status-firefox12:
--- → affected
Target Milestone: --- → Firefox 13
Comment 5•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 6•13 years ago
|
||
Comment on attachment 591641 [details] [diff] [review]
v.1 delay the doStartup call
[Approval Request Comment]
Regression caused by (bug #): Not a regression, new code.
User impact if declined: password migration fails from an existing Firefox profile. There is no UI to activate the feature in 12 but I will make an extension for SUMO to use.
Testing completed (on m-c, etc.): Tested on m-c for a few days.
Risk to taking this patch (and alternatives if risky): Low risk, trivial, tiny patch.
String changes made by this patch: None
Attachment #591641 -
Flags: approval-mozilla-aurora?
Comment 7•13 years ago
|
||
Comment on attachment 591641 [details] [diff] [review]
v.1 delay the doStartup call
[Triage Comment]
Low risk and fixes a regression in a new FF12 feature. Approving for Aurora.
Attachment #591641 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 8•13 years ago
|
||
Comment 9•13 years ago
|
||
Verified as fixed on:
Mozilla/5.0 (Windows NT 6.1; rv:13.0) Gecko/20120318 Firefox/13.0a2
Mozilla/5.0 (X11; Linux i686; rv:13.0) Gecko/20120319 Firefox/13.0a2
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:13.0) Gecko/20120318 Firefox/13.0a2
Status: RESOLVED → VERIFIED
Whiteboard: [qa!]
You need to log in
before you can comment on or make changes to this bug.
Description
•