Split read and write cipher specs

RESOLVED FIXED in Future

Status

NSS
Libraries
P2
normal
RESOLVED FIXED
8 months ago
7 months ago

People

(Reporter: mt, Assigned: mt)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 2 obsolete attachments)

(Assignee)

Description

8 months ago
There's a lot of duplication in these structs and that isn't suitable any longer.  TLS 1.3 maintains separate copies for reading and writing.

In TLS 1.2 and earlier, this will increase the overhead slightly - by a couple of pointers - but the code gets a fair bit simpler as a result.

Updated

8 months ago
Priority: -- → P2
(Assignee)

Updated

7 months ago
Assignee: nobody → martin.thomson

Updated

7 months ago
Attachment #8906498 - Attachment is obsolete: true

Updated

7 months ago
Attachment #8906499 - Attachment is obsolete: true
Comment on attachment 8919545 [details]
Bug 1398679 - Move much of the cipher spec code into a dedicated file, r?ekr

Eric Rescorla (:ekr) has approved the revision.

https://phabricator.services.mozilla.com/D137#3604
Attachment #8919545 - Flags: review+
Comment on attachment 8919546 [details]
Bug 1398679 - Make cipher specs properly directional, r?ekr

Eric Rescorla (:ekr) has approved the revision.

https://phabricator.services.mozilla.com/D138#4260
Attachment #8919546 - Flags: review+
(Assignee)

Comment 4

7 months ago
9d1576864bf1
Status: NEW → RESOLVED
Last Resolved: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → Future
You need to log in before you can comment on or make changes to this bug.