The default bug view has changed. See this FAQ.

After using Firefox migrator, the password manager sometimes can't decrypt passwords

VERIFIED FIXED in Firefox 12

Status

()

Firefox
Migration
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: MattN, Assigned: MattN)

Tracking

12 Branch
Firefox 13
Points:
---

Firefox Tracking Flags

(firefox12 fixed)

Details

(Whiteboard: [qa!])

Attachments

(1 attachment)

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.
Created attachment 591641 [details] [diff] [review]
v.1 delay the doStartup call

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)
Comment on attachment 591641 [details] [diff] [review]
v.1 delay the doStartup call

Review ping
Attachment #591641 - Flags: feedback?(mak77)
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)
https://hg.mozilla.org/integration/mozilla-inbound/rev/18fda283b571
status-firefox12: --- → affected
Target Milestone: --- → Firefox 13
https://hg.mozilla.org/mozilla-central/rev/18fda283b571
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
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

5 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+
https://hg.mozilla.org/releases/mozilla-aurora/rev/17f98701eca2
status-firefox12: affected → fixed

Comment 9

5 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.