44 bytes, text/x-phabricator-request
|Details | Review|
All cipher specs currently have a field for storing the master secret outside of PKCS#11 called msItem. This field has a lot of code around it, but that code is never actually used (maybe it was once). That code can all go.
Hmm, I should rename this to reflect that it's not everything that I'm removing, only the SECItem stuff. I discovered that the code doesn't even reference ssl3KeyMaterial.write_key_item and the other _key_item fields, so they are going too. https://phabricator.services.mozilla.com/D46
Assignee: nobody → martin.thomson
Summary: Remove unused fields from cipher specs → Remove raw key fields from cipher specs
Comment on attachment 8906436 [details] Bug 1398647 - Remove the SECItems used for "storing" keys in cipher specs, r?ttaubert Tim Taubert [:ttaubert] has approved the revision. https://phabricator.services.mozilla.com/D46#2652
Attachment #8906436 - Flags: review+
Status: NEW → RESOLVED
Last Resolved: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → Future
You need to log in before you can comment on or make changes to this bug.