I believe this amounts to adding "delete data.unwrapBKey" here: http://hg.mozilla.org/mozilla-central/annotate/7146e89a7b83/services/fxaccounts/FxAccounts.jsm#l651 and adding some tests to make sure it gets deleted after a successful key fetch.
Created attachment 8437097 [details] [diff] [review] delete unwrapBKey after use, plus tests This should be everything.
Created attachment 8437111 [details] [diff] [review] 0001-Bug-1021950-delete-unwrapBKey.patch Ah, this format of patch should be slightly easier to mark checkin-needed.
Attachment #8437111 - Flags: review?(ckarlof) → review+
There are two unrelated tests which fail when I run the full suite, but those same tests fail when I run an unmodified git checkout (1f1507a8dac30b1bbf934a801d48a684cf375ae0). So I think this is ready for checkin.
Please provide a Try link when requesting checkin. If you need some recommendations on what to run, you can take a look at the below: https://wiki.mozilla.org/Sheriffing/How:To:Recommended_Try_Practices
Thanks Nick! Ryan: is that sufficient? (sorry, I'm new at using checkin-needed).
Whiteboard: checkin-needed → [fixed-in-fx-team]
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in before you can comment on or make changes to this bug.