Closed Bug 721242 Opened 8 years ago Closed 8 years ago

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

Categories

(Firefox :: Migration, defect)

12 Branch
defect
Not set

Tracking

()

VERIFIED FIXED
Firefox 13
Tracking Status
firefox12 --- fixed

People

(Reporter: MattN, Assigned: MattN)

References

Details

(Whiteboard: [qa!])

Attachments

(1 file)

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.
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/mozilla-central/rev/18fda283b571
Status: ASSIGNED → RESOLVED
Closed: 8 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 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+
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.