Closed
Bug 959766
Opened 11 years ago
Closed 11 years ago
Remove an unnecessary ssl_FindSocket call in ssl_ImportFD
Categories
(NSS :: Libraries, defect, P2)
NSS
Libraries
Tracking
(Not tracked)
RESOLVED
FIXED
3.16
People
(Reporter: wtc, Assigned: wtc)
Details
Attachments
(1 file)
|
812 bytes,
patch
|
ryan.sleevi
:
review+
|
Details | Diff | Splinter Review |
The patch replaces an unnecessary ssl_FindSocket call in ssl_ImportFD with
an assertion. If I understand the code correctly, this assertion is not
necessary. In a future NSS release, we can remove this assertion.
The patch was originally reviewed in bug 807250 comment 3. The other changes
in that patch are wrong. So I wrote a separate patch for this change.
Attachment #8359959 -
Flags: review?(ryan.sleevi)
|
||
Updated•11 years ago
|
Attachment #8359959 -
Flags: review?(ryan.sleevi) → review+
|
Assignee | |
Comment 1•11 years ago
|
||
Patch checked in: https://hg.mozilla.org/projects/nss/rev/1b1eb6d77728
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•